repro original issue

Author: EricCousineau-TRI

tests/CMakeLists.txt

@@ -94,43 +94,45 @@ endif()
 
 # Full set of test files (you can override these; see below)
 set(PYBIND11_TEST_FILES
-    test_async.cpp
-    test_buffers.cpp
-    test_builtin_casters.cpp
-    test_call_policies.cpp
-    test_callbacks.cpp
-    test_chrono.cpp
-    test_class.cpp
-    test_constants_and_functions.cpp
-    test_copy_move.cpp
-    test_custom_type_casters.cpp
-    test_docstring_options.cpp
-    test_eigen.cpp
-    test_enum.cpp
-    test_eval.cpp
-    test_exceptions.cpp
-    test_factory_constructors.cpp
-    test_gil_scoped.cpp
-    test_iostream.cpp
-    test_kwargs_and_defaults.cpp
-    test_local_bindings.cpp
-    test_methods_and_attributes.cpp
-    test_modules.cpp
-    test_multiple_inheritance.cpp
-    test_numpy_array.cpp
-    test_numpy_dtypes.cpp
-    test_numpy_vectorize.cpp
-    test_opaque_types.cpp
-    test_operator_overloading.cpp
-    test_pickling.cpp
-    test_pytypes.cpp
-    test_sequences_and_iterators.cpp
-    test_smart_ptr.cpp
-    test_stl.cpp
-    test_stl_binders.cpp
-    test_tagbased_polymorphic.cpp
-    test_union.cpp
-    test_virtual_functions.cpp)
+    test_issue1552.cpp
+    # test_async.cpp
+    # test_buffers.cpp
+    # test_builtin_casters.cpp
+    # test_call_policies.cpp
+    # test_callbacks.cpp
+    # test_chrono.cpp
+    # test_class.cpp
+    # test_constants_and_functions.cpp
+    # test_copy_move.cpp
+    # test_custom_type_casters.cpp
+    # test_docstring_options.cpp
+    # test_eigen.cpp
+    # test_enum.cpp
+    # test_eval.cpp
+    # test_exceptions.cpp
+    # test_factory_constructors.cpp
+    # test_gil_scoped.cpp
+    # test_iostream.cpp
+    # test_kwargs_and_defaults.cpp
+    # test_local_bindings.cpp
+    # test_methods_and_attributes.cpp
+    # test_modules.cpp
+    # test_multiple_inheritance.cpp
+    # test_numpy_array.cpp
+    # test_numpy_dtypes.cpp
+    # test_numpy_vectorize.cpp
+    # test_opaque_types.cpp
+    # test_operator_overloading.cpp
+    # test_pickling.cpp
+    # test_pytypes.cpp
+    # test_sequences_and_iterators.cpp
+    # test_smart_ptr.cpp
+    # test_stl.cpp
+    # test_stl_binders.cpp
+    # test_tagbased_polymorphic.cpp
+    # test_union.cpp
+    # test_virtual_functions.cpp)
+    )
 
 # Invoking cmake with something like:
 #     cmake -DPYBIND11_TEST_OVERRIDE="test_callbacks.cpp;test_pickling.cpp" ..

tests/test_issue1552.cpp

@@ -0,0 +1,66 @@
+// https://github.com/pybind/pybind11/issues/1552
+#include <iostream>
+
+#include "pybind11_tests.h"
+
+class Dispatcher;
+
+class Client
+{
+public:
+    Client(Dispatcher* disp): PtrD(disp)
+    {
+        std::cout << "In Client::Client\n";
+    }
+    virtual ~Client(){};
+    virtual void ProcessEvent()
+    {
+        std::cout << "THIS SHOULDN'T HAPPEN --In Client::ProcessEvent\n";
+    }
+    Dispatcher* PtrD;
+};
+
+class Dispatcher
+{
+public:
+    Dispatcher()
+    {
+        std::cout << "In Dispatcher::Dispatcher\n";
+    }
+    virtual ~Dispatcher(){};
+
+    void Dispatch(Client* client)
+    {
+        std::cout << "Dispatcher::Dispatch called by " << client << std::endl;
+        client->ProcessEvent();
+    }
+};
+
+class DispatcherTrampoline : public Dispatcher
+{
+public:
+    using Dispatcher::Dispatcher;
+};
+
+class ClientTrampoline : public Client
+{
+public:
+    using Client::Client;
+
+    void ProcessEvent() override
+    {
+        PYBIND11_OVERLOAD(void,Client,ProcessEvent,);
+    }
+};
+
+TEST_SUBMODULE(issue1552, m)
+{
+    py::class_<Client,ClientTrampoline> cli(m,"Client");
+    cli.def(py::init<Dispatcher* >());
+    cli.def("ProcessEvent",&Client::ProcessEvent);
+    cli.def_readwrite("PtrD",&Client::PtrD);
+
+    py::class_<Dispatcher,DispatcherTrampoline> dsp(m,"Dispatcher");
+    dsp.def(py::init< >());
+    dsp.def("Dispatch",&Dispatcher::Dispatch);
+}

tests/test_issue1552.py

@@ -0,0 +1,21 @@
+# -*- coding: utf-8 -*-
+import pytest
+
+from pybind11_tests import issue1552 as m
+
+
+class SomeClient(m.Client):
+    def __init__(self,d):
+        print("In SomeClient::__init__")
+        super().__init__(d);
+
+    def ProcessEvent(self):
+        print("in SomeClient::ProcessEvent,about to call self.ProcessEvent")
+        self.PtrD.Dispatch(self);
+
+
+def test_main():
+    # https://github.com/pybind/pybind11/issues/1552
+    dd = m.Dispatcher()
+    cl = SomeClient(dd)
+    dd.Dispatch(cl)