boostorg/unordered
1
0
Fork 1
mirror of https://github.com/boostorg/unordered.git synced 2025-07-30 03:17:15 +02:00
Code Issues Packages Projects Releases Wiki Activity

Remove the optimization for std::pair with a key reference. It'll be too much hassle to get a very unusual use case to work on all compilers.

Browse Source 
[SVN r56461]
This commit is contained in:
Daniel James
2009-09-28 23:06:03 +00:00
parent 31cd8f4e16
commit 2b8680d2c4
2 changed files with 3 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
include/boost/unordered/detail/extract_key.hpp
View File

@ -107,20 +107,6 @@ namespace unordered_detail {
return v.first; return v.first;
} }
template <class Second>
static key_type const& extract(
std::pair<key_type&, Second> const& v)
{
return v.first;
}
template <class Second>
static key_type const& extract(
std::pair<key_type const&, Second> const& v)
{
return v.first;
}
#if defined(BOOST_UNORDERED_STD_FORWARD) #if defined(BOOST_UNORDERED_STD_FORWARD)
template <class Arg1, class... Args> template <class Arg1, class... Args>
static key_type const& extract(key_type const& k, static key_type const& extract(key_type const& k,

5
test/unordered/insert_range_tests.cpp
View File

@ -29,12 +29,13 @@ UNORDERED_AUTO_TEST(needless_copies_2) {
test::object_count count = test::global_object_count; test::object_count count = test::global_object_count;
dst.emplace(src); dst.emplace(src);
BOOST_TEST(test::global_object_count.instances == count.instances + 1); BOOST_TEST_EQ(test::global_object_count.instances, count.instances + 1);
BOOST_TEST_EQ(test::global_object_count.constructions, count.constructions + 1); BOOST_TEST_EQ(test::global_object_count.constructions, count.constructions + 1);
count = test::global_object_count; count = test::global_object_count;
dst.emplace(src); dst.emplace(src);
BOOST_TEST_EQ(test::global_object_count, count); BOOST_TEST_EQ(test::global_object_count.instances, count.instances);
BOOST_TEST(test::global_object_count.constructions <= count.constructions + 1);
} }
RUN_TESTS() RUN_TESTS()