Boost C++ Libraries: Ticket #10071: [container] flat_map::insert() is ambiguous if used with initializer lists

Ion Gaztañaga — Mon, 23 Jun 2014 20:58:44 GMT

Merging issue #10135. Copying the issue text:

[container] flat_map::insert() is ambiguous if used with pair that requires conversion

Reported by: Sebastian Karlsson Version: Boost 1.55.0

The following is a no go it seems:

boost::container::flat_map< int, int > fm; fm.insert( std::make_pair( 0u, 0 ) );

Because the pair has to be converted, but it can be converted to either std::pair<> or container::container_details::pair<>.

Ion Gaztañaga — Mon, 23 Jun 2014 21:22:18 GMT

Fixing this correctly requires a bit of surgery since it should avoid using boost::container::container_detail::pair and make std::pair moveable on compilers without movable std::pair. As a first step, in compilers with rvalue reference we could avoid boost::container::container_detail::pair but we shouldn't break old code.

viboes — Sun, 06 Dec 2015 09:11:08 GMT

What is wrong documenting the limitation on c++03 and requesting the user to use boost::make_pair?

anonymous — Sat, 09 Jan 2016 18:35:02 GMT

As a first-time user of flat_map, I was pretty confused why

insert( std::make_pair( x, y ) )

wasn't compiling, especially since the docs suggest that the Boost containers are drop-in replacements, and since flat_map::value_type is std::pair. At any rate, I just recommend including a note on proper usage in the descriptions for the insert() family.

P.S. For anybody stumbling across this bug report after a Google search, you can use

emplace( std::make_pair(x, y ) )

Still not { x, y }, though.