Boost C++ Libraries: Ticket #11032: boost::offset_ptr needs explicit ctor

Ion Gaztañaga — Wed, 11 Mar 2015 20:35:07 GMT

I can't see the problem. offset_ptr has a constructor that allows conversions:

//!Constructor from other offset_ptr. If pointers of pointee types are
//!convertible, offset_ptrs will be convertibles. Never throws.
template<class T2>
offset_ptr( const offset_ptr<T2, DifferenceType, OffsetType, OffsetAlignment> &ptr
          #ifndef BOOST_INTERPROCESS_DOXYGEN_INVOKED
          , typename ipcdetail::enable_if_c< ipcdetail::is_convertible<T2*, PointedType*>::value
            && ipcdetail::offset_ptr_maintains_address<T2, PointedType>::value
          >::type * = 0
          #endif
          )
   : internal(static_cast<OffsetType>
         (ipcdetail::offset_ptr_to_offset_from_other<0>(this, &ptr, ptr.get_offset())))
{}
//!Constructor from other offset_ptr. If pointers of pointee types are
//!convertible, offset_ptrs will be convertibles. Never throws.
template<class T2>
offset_ptr( const offset_ptr<T2, DifferenceType, OffsetType, OffsetAlignment> &ptr
            , typename ipcdetail::enable_if_c< ipcdetail::is_convertible<T2*, PointedType*>::value
            && !ipcdetail::offset_ptr_maintains_address<T2, PointedType>::value
            >::type * = 0)
   : internal(static_cast<OffsetType>
      (ipcdetail::offset_ptr_to_offset<0>(static_cast<PointedType*>(ptr.get()), this)))
{}

this allows upcasting (from derived to base). Did you mean that offset_ptr does not support downcasting (from base to derived)? It does that for good safety reasons as raw pointers don't allow those conversions. Do you want this to compile?

offset_ptr<Base> pb = ...; offset_ptr<Derived>pd(p);

whereas this is a compilation error

Base *pb = ... Derived *pd = pb;

stheophil@… — Thu, 12 Mar 2015 09:21:24 GMT

Yes, I want it to support downcasting. Sorry.

offset_ptr<Base> pb = ...; offset_ptr<Derived>pd(pb);

Does not have to compile because, as you say, raw pointers do not support this conversion, but

offset_ptr<Base> pb = ...; offset_ptr<Derived>pd = static_cast<offset_ptr<Derived>>(pb);

should compile and by consequence

offset_ptr<Base> pb = ...; offset_ptr<Derived>pd = (offset_ptr<Derived>)pb;

too, which just does a static_cast if I understand correctly. Both are used a lot in std::list and std::map implementations of Clang and MSVC.

Tue, 17 Nov 2015 14:29:08 GMT

Please see the table here: http://en.cppreference.com/w/cpp/concept/Allocator

The C++11 standard requires static_cast<A::pointer>(vptr) to succeed.

I have also run into this problem when trying to use my own container, which uses void_pointer to store child pointers, with Boost.Interprocess.