iterator value_type possibly cv-qualified

[Shunsuke Sogame <pstade.mb@…>]

boost::ptr_vector<int const>::iterator::value_type

is int const now, but it should be int.

[Thorsten Ottosen]

AFAICT, you can't even use boost::ptr_vector<int const>, because the whole library has not been designed/tested with this in mind.

Standard containers don't support this. Can you describe why it is useful?

-Thorsten

[Shunsuke Sogame <pstade.mb@…>]

Replying to nesotto:

the whole library has not been designed/tested with this in mind.

I didn't know that. Sorry for the noise.

-- Shunsuke Sogame

[Shunsuke Sogame <pstade.mb@…>]

Replying to nesotto:

AFAICT, you can't even use boost::ptr_vector<int const>, because the whole library has not been designed/tested with this in mind.

Sorry again.

See this:

#include <boost/ptr_container/ptr_vector.hpp>
#include <boost/range/const_iterator.hpp>
#include <boost/mpl/assert.hpp>

int main()
{
    typedef boost::range_const_iterator< boost::ptr_vector<int> >::type citer_t;
    BOOST_MPL_ASSERT((boost::is_same<citer_t::value_type, int const>)); // pass
    BOOST_MPL_ASSERT((boost::is_same<citer_t::value_type, int>)); // failed
}

This seems a bug.

--

Shunsuke Sogame

[Thorsten Ottosen]

Should be fixed...