with raw pointers it does work

Author: aoloe

scripter-api/README.md

@@ -34,6 +34,8 @@ $ ./scripting
 
 ## notes
 
+- if we ever want to check how it works with `unique_ptr`s, this is a good article on how to add a `unique_ptr` as a private member in a class: <https://katyscode.wordpress.com/2012/10/04/c11-using-stdunique_ptr-as-a-class-member-initialization-move-semantics-and-custom-deleters/>
+
 
 i'm now looking for a data structure that would allow me to pass my document. if i use "naked" pointers i can get there:
 

scripter-api/python/set-bar.py

@@ -6,5 +6,5 @@
 print(Sample.a)
 document = Sample.document
 print(document.a)
-# print(Sample.document.a)
+print(Sample.document.a)
 print(Sample.a)

scripter-api/src/main.cpp

@@ -13,7 +13,7 @@ int main()
 {
     Sample::Document document;
 
-    Scripter scripter{document};
+    Scripter scripter{&document};
     scripter.runFile("../python/set-bar.py");
     // std::cout << "document a " << document.a << std::endl;
     std::cout << "document top " << document.margin.top << std::endl;

scripter-api/src/scripter.cpp

@@ -9,10 +9,9 @@ void Scripter::runFile(std::string fileName)
 {
     // Sample::Document document;
     //Sample::Margin saMargin;
-    auto document = std::make_unique<Sample::Document>();
 
     ScripterAPI::ScripterAPI scripterAPI;
-    scripterAPI.setDocument(std::move(document));
+    scripterAPI.setDocument(document);
 
 
     auto module = py::module::import("scripterapi");

scripter-api/src/scripter.h

@@ -12,11 +12,11 @@ namespace py = pybind11;
 class Scripter
 {
     public:
-        Scripter(Sample::Document &document) :
+        Scripter(Sample::Document const *document) :
             document{document} {};
         void runFile(std::string fileName);
     private:
-        Sample::Document &document;
+        const Sample::Document *document;
 
         py::scoped_interpreter guard{};
 };

scripter-api/src/scripterAPI/document.cpp

@@ -4,9 +4,9 @@
 
 namespace py = pybind11;
 
-PYBIND11_MODULE(documentapi, m) {
+void init_DocumentAPI(py::module &m) {
     py::class_<ScripterAPI::Document>(m, "Document")
         .def(py::init<>())
-        .def_readwrite("document", &ScripterAPI::Document::a)
+        .def_readwrite("a", &ScripterAPI::Document::a)
         ;
 }

scripter-api/src/scripterAPI/document.h

@@ -7,12 +7,11 @@
 namespace ScripterAPI {
     class Document {
         public:
-            // void setDocument(std::unique_ptr<Sample::Document> d) {/* document = std::move(d);*/}
-            // std::unique_ptr<Sample::Document> getDocument() {/*return std::move(document);*/}
+            Document() {}
+            void setDocument(const Sample::Document* d) { document = d; }
 			int a{42};
         private:
-            // std::unique_ptr<Sample::Document> document;
+            const Sample::Document* document;
     };
 }
 #endif
-

scripter-api/src/scripterAPI/scripterAPI.cpp

@@ -4,10 +4,18 @@
 
 namespace py = pybind11;
 
-PYBIND11_MODULE(scripterapi, m) {
+void init_ScripterAPI(py::module &m) {
     py::class_<ScripterAPI::ScripterAPI>(m, "Sample")
         .def(py::init<>())
         .def_readwrite("a", &ScripterAPI::ScripterAPI::a)
         .def_readwrite("document", &ScripterAPI::ScripterAPI::documentAPI)
-        ;
+    ;
+}
+
+void init_DocumentAPI(py::module &);
+
+PYBIND11_MODULE(scripterapi, m) {
+        m.doc() = "Python interface the Sample application";
+        init_ScripterAPI(m);
+        init_DocumentAPI(m);
 }

scripter-api/src/scripterAPI/scripterAPI.h

@@ -8,9 +8,9 @@
 namespace ScripterAPI {
     class ScripterAPI {
         public:
+            ScripterAPI() {}
             int a = 1;
-            void setDocument(std::unique_ptr<Sample::Document> d) {/* documentAPI.setDocument(std::move(d));*/}
-            // std::unique_ptr<Sample::Document> getDocument() {return std::move(documentAPI.getDocument());}
+            void setDocument(const Sample::Document* d) { documentAPI.setDocument(d);}
             Document documentAPI;
     };
 }