bug fix + regression test

[SVN r17002]

test/Jamfile

@@ -95,6 +95,8 @@ bpl-test back_reference ;
 bpl-test implicit ;
 bpl-test data_members ;
 
+bpl-test ben_scott1 ;
+
 bpl-test bienstman1 ;
 bpl-test bienstman2 ;
 bpl-test bienstman3 ;

test/ben_scott1.cpp

@@ -0,0 +1,51 @@
+#include <boost/python.hpp>
+#include <iostream>
+using namespace boost::python;
+using namespace boost;
+
+struct Product {};
+typedef shared_ptr<Product> ProductPtr;
+
+
+struct Creator
+{
+   virtual ~Creator() {}
+   virtual ProductPtr create() = 0;
+};
+
+
+struct Factory
+{
+   void reg(Creator* c) { mC = c; }
+   ProductPtr create()
+   {
+      std::cout << "Name: " << (typeid(*mC)).name() << std::endl;
+      return mC->create();
+   }
+
+private:
+   Creator* mC;
+};
+
+struct CreatorWrap : public Creator
+{
+   CreatorWrap(PyObject* self) : mSelf(self) {}
+   ProductPtr create() { return call_method<ProductPtr>(mSelf, "create"); }
+   PyObject* mSelf;
+};
+
+BOOST_PYTHON_MODULE(ben_scott1_ext)
+{
+   class_<Product, ProductPtr>("Product");
+
+   class_<Creator, CreatorWrap, noncopyable>("Creator")
+       .def("create", &CreatorWrap::create)
+       ;
+
+   class_<Factory>("Factory")
+      .def("reg", &Factory::reg, with_custodian_and_ward<1,2>())
+      .def("create", &Factory::create)
+       ;
+}
+
+#include "../test/module_tail.cpp"

test/ben_scott1.py

@@ -0,0 +1,14 @@
+# This regression test checks that call_method<T>(...) where T is a
+# non-reference, non-pointer type that happens to be held inside the
+# result object (and thus is found as an lvalue) works.
+from ben_scott1_ext import *
+
+class CreatorImpl(Creator):
+   def create(self):
+      return Product()
+
+factory = Factory()
+c = CreatorImpl()
+factory.reg(c)
+
+a = factory.create()