On 02/04/2012 08:23 PM, Adam Preble wrote:
On Sat, Feb 4, 2012 at 10:46 AM, Jim Bosch <tallji...@gmail.com
<mailto:tallji...@gmail.com>> wrote:
If I understand your intend correctly, I think your class_
definition needs to be:
class_<Communicable,shared___ptr<CommunicableWrapper>,__noncopyable>
(right now you have class_<CommunicableWrapper,...__>)
Something like this?
class_<Communicatable, boost::shared_ptr<CommunicatableWrapper>,
boost::noncopyable>("Communicatable")
I get burnt by the compiler when I try that:
d:\coding\boost_1_47_0\boost\python\object\pointer_holder.hpp(217):
error C2664: 'CommunicatableWrapper::CommunicatableWrapper(const
CommunicatableWrapper &)' : cannot convert parameter 1 from 'PyObject *'
to 'const CommunicatableWrapper &'
Reason: cannot convert from 'PyObject *' to 'const
CommunicatableWrapper'
No constructor could take the source type, or constructor
overload resolution was ambiguous
It sounds like if I had a constructor that would take a PyObject *, that
would shut it up, but I have no idea what to make of such a thing.
Oh, you're right. I was confusing the manual way of doing Python-side
polymorphism (derived-class holders) with the more automatic (and
better) way you're doing it (with wrapper). Your original code was fine
in that regard.
Anyhow, looking closer, here's what you need to do:
- Put your code back to what it was before I told you to change it.
- Add another line, after the class wrappers:
register_ptr_to_python< boost::shared_ptr<Communicatable> >();
That explicitly registers a shared_ptr converter for Communicatable.
I'm not sure why Boost.Python doesn't automatically do that when you
register converters for its derived classes (those are registered by
putting the shared_pr types in the template parameters to class_). It
might just be an oversight, or it might be that doing the base class
registration automatically would cause problems in some contexts.
Jim
_______________________________________________
Cplusplus-sig mailing list
Cplusplus-sig@python.org
http://mail.python.org/mailman/listinfo/cplusplus-sig
