From a90181962d3f9eca3e00f665465bbd66bdd79927 Mon Sep 17 00:00:00 2001
From: David Robillard <d@drobilla.net>
Date: Sat, 9 Sep 2006 21:42:33 +0000
Subject: Canvas cleanups, bugfixes, more robustness, etc.

git-svn-id: http://svn.drobilla.net/lad/ingen@124 a436a847-0d15-0410-975c-d299462d15a1
---
 src/progs/ingenuity/PortController.cpp | 69 ++++++++++++----------------------
 1 file changed, 25 insertions(+), 44 deletions(-)

(limited to 'src/progs/ingenuity/PortController.cpp')

diff --git a/src/progs/ingenuity/PortController.cpp b/src/progs/ingenuity/PortController.cpp
index 0668a9e0..6a23169f 100644
--- a/src/progs/ingenuity/PortController.cpp
+++ b/src/progs/ingenuity/PortController.cpp
@@ -18,7 +18,7 @@
 #include "OmFlowCanvas.h"
 #include "OmModule.h"
 #include "PortModel.h"
-#include "ControlPanel.h"
+#include "PatchModel.h"
 #include "OmPort.h"
 #include "OmPatchPort.h"
 #include "Store.h"
@@ -28,9 +28,9 @@ namespace Ingenuity {
 
 PortController::PortController(CountedPtr<PortModel> model)
 : GtkObjectController(model),
+  m_patch_port(NULL),
   m_module(NULL),
   m_port(NULL)
-  //m_control_panel(NULL)
 {
 	assert(model);
 	assert(model->parent());
@@ -47,9 +47,6 @@ PortController::destroy()
 	NodeController* parent = (NodeController*)m_model->parent()->controller();
 	assert(parent != NULL);
 
-	//if (m_control_panel != NULL)
-	//	m_control_panel->remove_port(path());
-
 	parent->remove_port(path(), false);
 }
 
@@ -70,35 +67,38 @@ PortController::create_module(OmFlowCanvas* canvas)
 
 	assert(canvas);
 	assert(port_model());
+
+	if (m_module)
+		delete m_module;
+
 	m_module = new OmPortModule(canvas, this, x, y);
 	
-	// FIXME: leak
-	m_patch_port = new OmPatchPort(m_module, port_model());
-	m_module->add_port(m_patch_port, false);
+	if (CountedPtr<PatchModel>(port_model()->parent())) {
+		if (m_patch_port)
+			delete m_patch_port;
+
+		m_patch_port = new OmPatchPort(m_module, port_model());
+	}
 	
+	m_module->resize();
+
 	m_module->move_to(x, y); // FIXME: redundant (?)
 }
 
 
+void
+PortController::destroy_module()
+{
+	delete m_module;
+	m_module = NULL;
+}
+
+
 void
 PortController::metadata_update(const string& key, const string& value)
 {
-	// FIXME: double lookups
-	
-	//cerr << path() << ": " << key << " = " << value << endl;
-
-/* Panel now listens to model signals..
-
-	if (key == "user-min") {
-		port_model()->user_min(atof(value.c_str()));
-		if (m_control_panel != NULL)
-			m_control_panel->set_range_min(m_model->path(), atof(value.c_str()));
-	} else if (key == "user-max") {
-		port_model()->user_max(atof(value.c_str()));
-		if (m_control_panel != NULL)
-			m_control_panel->set_range_max(m_model->path(), atof(value.c_str()));
-	}
-*/
+	//cerr << "Metadata " << path() << ": " << key << " = " << value << endl;
+
 	if (m_module != NULL) {
 		if (key == "module-x") {
 			float x = atof(value.c_str());
@@ -114,21 +114,6 @@ PortController::metadata_update(const string& key, const string& value)
 	GtkObjectController::metadata_update(key, value);
 }
 
-
-/** "Register" a control panel that is monitoring this port.
- *
- * The OmPort will handle notifying the ControlPanel when state
- * changes occur, etc.
- */
-/*
-void
-PortController::set_control_panel(ControlPanel* cp)
-{
-	assert(m_control_panel == NULL);
-	m_control_panel = cp;
-}
-*/
-
 void
 PortController::set_path(const Path& new_path)
 {
@@ -136,9 +121,6 @@ PortController::set_path(const Path& new_path)
 	if (m_port != NULL)
 		m_port->set_name(new_path.name());
 
-	//if (m_control_panel != NULL)
-	//	m_control_panel->rename_port(m_model->path(), new_path);
-
 	m_model->set_path(new_path);
 }
 
@@ -152,8 +134,7 @@ PortController::create_port(OmModule* module)
 {
 	assert(module != NULL);
 
-	m_port = new OmPort(module, port_model());
-	module->add_port(m_port, false);
+	new OmPort(module, port_model());
 }
 
 
-- 
cgit v1.2.1