Boost C++ Libraries: Ticket #6089: Non const & used in ptr_map::insert

attachment set

jdoliner@… — Thu, 03 Nov 2011 18:25:23 GMT

attachment → patch

A patch which fixes the problem.

Fri, 05 Apr 2013 03:27:17 GMT

still exists in 1.53.0

anonymous — Wed, 02 Nov 2016 19:44:40 GMT

Still there is 1.62.0 and causes the most basic examples of ptr_map to fail to compile. For instance:

    boost::ptr_map<int, int> m;
    m.insert(0, new int(5));

It's literally a one word change to fix this.

m.champlon@… — Tue, 17 Jan 2017 22:15:34 GMT

I believe this is actually a feature, see www.boost.org/libs/ptr_container/doc/faq.html#why-does-ptr-map-t-insert-replace-take-two-arguments-the-key-and-the-pointer-instead-of-one-std-pair-and-why-is-the-key-passed-by-non-const-reference for a rationale.