Fix memory leake by passing parent to FlowScene (paceholder#151) (paceholder#155)

Author: paceholder

examples/calculator/main.cpp

@@ -100,7 +100,7 @@ main(int argc, char *argv[])
   QVBoxLayout *l = new QVBoxLayout(&mainWidget);
 
   l->addWidget(menuBar);
-  auto scene = new FlowScene(registerDataModels());
+  auto scene = new FlowScene(registerDataModels(), &mainWidget);
   l->addWidget(new FlowView(scene));
   l->setContentsMargins(0, 0, 0, 0);
   l->setSpacing(0);

include/nodes/internal/FlowScene.hpp

@@ -31,8 +31,10 @@ class NODE_EDITOR_PUBLIC FlowScene
   Q_OBJECT
 public:
 
-  FlowScene(std::shared_ptr<DataModelRegistry> registry =
-              std::make_shared<DataModelRegistry>());
+  FlowScene(std::shared_ptr<DataModelRegistry> registry,
+            QObject * parent = Q_NULLPTR);
+
+  FlowScene(QObject * parent = Q_NULLPTR);
 
   ~FlowScene();
 

src/FlowScene.cpp

@@ -36,13 +36,22 @@ using QtNodes::PortType;
 using QtNodes::PortIndex;
 using QtNodes::TypeConverter;
 
+
 FlowScene::
-FlowScene(std::shared_ptr<DataModelRegistry> registry)
-  : _registry(registry)
+FlowScene(std::shared_ptr<DataModelRegistry> registry,
+          QObject * parent)
+  : QGraphicsScene(parent)
+  , _registry(registry)
 {
   setItemIndexMethod(QGraphicsScene::NoIndex);
 }
 
+FlowScene::
+FlowScene(QObject * parent)
+  : FlowScene(std::make_shared<DataModelRegistry>(),
+              parent)
+{}
+
 
 FlowScene::
 ~FlowScene()