From b15864870d34a1188eda93ad215734275037278e Mon Sep 17 00:00:00 2001 From: David Robillard Date: Mon, 11 Sep 2006 11:10:35 +0000 Subject: Switched homebrew CountedPtr to boost::shared_ptr. Factories for patch windows, controller. Robustness updated in many places. Tons of cleanups, rewrites, bugfixes, etc. git-svn-id: http://svn.drobilla.net/lad/ingen@128 a436a847-0d15-0410-975c-d299462d15a1 --- src/progs/Makefile.am | 2 +- src/progs/demolition/DemolitionClientInterface.cpp | 4 +- src/progs/demolition/DemolitionClientInterface.h | 10 +- src/progs/demolition/DemolitionModel.cpp | 2 +- src/progs/demolition/DemolitionModel.h | 2 +- src/progs/ingenuity/App.cpp | 5 +- src/progs/ingenuity/App.h | 8 +- src/progs/ingenuity/BreadCrumbBox.cpp | 39 ++- src/progs/ingenuity/BreadCrumbBox.h | 3 + src/progs/ingenuity/ConnectWindow.cpp | 8 +- src/progs/ingenuity/ControlGroups.cpp | 12 +- src/progs/ingenuity/ControlPanel.cpp | 19 +- src/progs/ingenuity/ControlPanel.h | 2 +- src/progs/ingenuity/ControllerFactory.cpp | 72 ++++++ src/progs/ingenuity/ControllerFactory.h | 32 +++ src/progs/ingenuity/DSSIModule.cpp | 4 +- src/progs/ingenuity/GtkObjectController.cpp | 2 - src/progs/ingenuity/LoadPatchWindow.cpp | 5 +- src/progs/ingenuity/LoadPatchWindow.h | 8 +- src/progs/ingenuity/LoadPluginWindow.cpp | 13 +- src/progs/ingenuity/LoadPluginWindow.h | 4 +- src/progs/ingenuity/LoadSubpatchWindow.cpp | 11 +- src/progs/ingenuity/LoadSubpatchWindow.h | 6 +- src/progs/ingenuity/Loader.cpp | 4 +- src/progs/ingenuity/Loader.h | 24 +- src/progs/ingenuity/Makefile.am | 4 + src/progs/ingenuity/NewSubpatchWindow.cpp | 10 +- src/progs/ingenuity/NewSubpatchWindow.h | 7 +- src/progs/ingenuity/NodeController.cpp | 59 ++--- src/progs/ingenuity/NodeController.h | 18 +- src/progs/ingenuity/NodePropertiesWindow.cpp | 1 - src/progs/ingenuity/OmFlowCanvas.cpp | 27 +- src/progs/ingenuity/OmModule.cpp | 4 +- src/progs/ingenuity/OmModule.h | 1 + src/progs/ingenuity/OmPatchPort.cpp | 2 +- src/progs/ingenuity/OmPort.cpp | 2 +- src/progs/ingenuity/PatchController.cpp | 281 +++++++-------------- src/progs/ingenuity/PatchController.h | 35 +-- src/progs/ingenuity/PatchPropertiesWindow.cpp | 1 - src/progs/ingenuity/PatchTreeWindow.cpp | 29 +-- src/progs/ingenuity/PatchTreeWindow.h | 8 +- src/progs/ingenuity/PatchView.cpp | 29 ++- src/progs/ingenuity/PatchView.h | 5 +- src/progs/ingenuity/PatchWindow.cpp | 154 ++--------- src/progs/ingenuity/PatchWindow.h | 27 +- src/progs/ingenuity/PortController.cpp | 9 +- src/progs/ingenuity/PortController.h | 6 +- src/progs/ingenuity/RenameWindow.cpp | 2 +- src/progs/ingenuity/SubpatchModule.cpp | 41 ++- src/progs/ingenuity/SubpatchModule.h | 8 +- src/progs/ingenuity/WindowFactory.cpp | 108 ++++++++ src/progs/ingenuity/WindowFactory.h | 42 +++ src/progs/ingenuity/ingenuity.glade | 7 +- src/progs/patch_loader/patch_loader.cpp | 25 +- src/progs/server/Makefile.am | 2 +- src/progs/server/main.cpp | 2 +- 56 files changed, 669 insertions(+), 588 deletions(-) create mode 100644 src/progs/ingenuity/ControllerFactory.cpp create mode 100644 src/progs/ingenuity/ControllerFactory.h create mode 100644 src/progs/ingenuity/WindowFactory.cpp create mode 100644 src/progs/ingenuity/WindowFactory.h (limited to 'src/progs') diff --git a/src/progs/Makefile.am b/src/progs/Makefile.am index a1a8c911..01791a1b 100644 --- a/src/progs/Makefile.am +++ b/src/progs/Makefile.am @@ -5,7 +5,7 @@ DIST_SUBDIRS = python supercollider SUBDIRS = server if BUILD_CONSOLE_CLIENTS -SUBDIRS += patch_loader demolition +SUBDIRS += patch_loader #demolition endif if BUILD_GTK_CLIENT diff --git a/src/progs/demolition/DemolitionClientInterface.cpp b/src/progs/demolition/DemolitionClientInterface.cpp index a0704028..23f5733f 100644 --- a/src/progs/demolition/DemolitionClientInterface.cpp +++ b/src/progs/demolition/DemolitionClientInterface.cpp @@ -36,7 +36,7 @@ DemolitionClientInterface::error(string msg) void -DemolitionClientInterface::new_patch_model(PatchModel* pm) +DemolitionClientInterface::new_patch_model(CountedPtr pm) { m_model->add_patch(pm); } @@ -68,7 +68,7 @@ DemolitionClientInterface::patch_disabled(string path) void -DemolitionClientInterface::new_node_model(NodeModel* nm) +DemolitionClientInterface::new_node_model(CountedPtr nm) { m_model->add_node(nm); } diff --git a/src/progs/demolition/DemolitionClientInterface.h b/src/progs/demolition/DemolitionClientInterface.h index 83539afc..e1093411 100644 --- a/src/progs/demolition/DemolitionClientInterface.h +++ b/src/progs/demolition/DemolitionClientInterface.h @@ -56,19 +56,19 @@ public: void new_plugin(string type, string uri, string name) {} - void new_patch_model(PatchModel* const pm); - void new_port_model(PortModel* const port_model); + void new_patch_model(CountedPtr pm); + void new_port_model(CountedPtr port_model); void object_destroyed(string path); void patch_enabled(string path); void patch_disabled(string path); void patch_cleared(string path) { throw; } - void new_node_model(NodeModel* const nm); + void new_node_model(CountedPtr nm); void object_renamed(string old_path, string new_path); - void connection_model(ConnectionModel* const cm); + void connection_model(CountedPtr cm); void disconnection(string src_port_path, string dst_port_path); void metadata_update(string path, string key, string value) {} void control_change(string port_path, float value); - void new_plugin_model(PluginModel* const pi); + void new_plugin_model(CountedPtr pi); void program_add(string path, uint32_t bank, uint32_t program, string name) {}; void program_remove(string path, uint32_t bank, uint32_t program) {}; diff --git a/src/progs/demolition/DemolitionModel.cpp b/src/progs/demolition/DemolitionModel.cpp index b91b461a..8ac10195 100644 --- a/src/progs/demolition/DemolitionModel.cpp +++ b/src/progs/demolition/DemolitionModel.cpp @@ -189,7 +189,7 @@ DemolitionModel::remove_object(const Path& path) void -DemolitionModel::add_node(NodeModel* nm) +DemolitionModel::add_node(CountedPtr nm) { PatchModel* parent = patch(nm->path().parent()); if (parent == NULL) { diff --git a/src/progs/demolition/DemolitionModel.h b/src/progs/demolition/DemolitionModel.h index 67eb1005..6ae8c0ec 100644 --- a/src/progs/demolition/DemolitionModel.h +++ b/src/progs/demolition/DemolitionModel.h @@ -39,7 +39,7 @@ public: NodeModel* node(const Path& path); void add_patch(PatchModel* pm) { m_patches.push_back(pm); } - void add_node(NodeModel* nm); + void add_node(CountedPtr nm); void add_port(PortModel* pm); void remove_object(const Path& path); void add_connection(ConnectionModel* cm); diff --git a/src/progs/ingenuity/App.cpp b/src/progs/ingenuity/App.cpp index ecf94a92..ae452f27 100644 --- a/src/progs/ingenuity/App.cpp +++ b/src/progs/ingenuity/App.cpp @@ -22,7 +22,6 @@ #include #include #include -#include "PatchView.h" #include "OmModule.h" #include "ControlPanel.h" #include "SubpatchModule.h" @@ -44,6 +43,7 @@ #include "ConnectWindow.h" #include "Store.h" #include "Loader.h" +#include "WindowFactory.h" #ifdef HAVE_LASH #include "LashController.h" #endif @@ -66,6 +66,7 @@ App::App() _loader(NULL), _configuration(new Configuration()), _about_dialog(NULL), + _window_factory(new WindowFactory()), _enable_signal(true) { Glib::RefPtr glade_xml = GladeFactory::new_glade_reference(); @@ -94,7 +95,7 @@ App::instantiate() void -App::attach(CountedPtr& engine, CountedPtr& client) +App::attach(const CountedPtr& engine, const CountedPtr& client) { assert( ! _engine); assert( ! _client); diff --git a/src/progs/ingenuity/App.h b/src/progs/ingenuity/App.h index 06d52ca3..25e01ff2 100644 --- a/src/progs/ingenuity/App.h +++ b/src/progs/ingenuity/App.h @@ -61,6 +61,7 @@ class PatchTreeWindow; class ConnectWindow; class Configuration; class Loader; +class WindowFactory; /** Singleton master class most everything is contained within. @@ -85,7 +86,8 @@ public: int num_open_patch_windows(); - void attach(CountedPtr& engine, CountedPtr& client); + void attach(const CountedPtr& engine, + const CountedPtr& client); ConnectWindow* connect_window() const { return _connect_window; } Gtk::Dialog* about_dialog() const { return _about_dialog; } @@ -95,7 +97,8 @@ public: Configuration* configuration() const { return _configuration; } Store* store() const { return _store; } Loader* loader() const { return _loader; } - + WindowFactory* window_factory() const { return _window_factory; } + const CountedPtr& engine() const { return _engine; } const CountedPtr& client() const { return _client; } @@ -121,6 +124,7 @@ protected: PatchTreeWindow* _patch_tree_window; ConfigWindow* _config_window; Gtk::Dialog* _about_dialog; + WindowFactory* _window_factory; /** Used to avoid feedback loops with (eg) process checkbutton * FIXME: Maybe this should be globally implemented at the Controller level, diff --git a/src/progs/ingenuity/BreadCrumbBox.cpp b/src/progs/ingenuity/BreadCrumbBox.cpp index d2bf7f97..aea1cdf0 100644 --- a/src/progs/ingenuity/BreadCrumbBox.cpp +++ b/src/progs/ingenuity/BreadCrumbBox.cpp @@ -28,9 +28,10 @@ BreadCrumbBox::BreadCrumbBox() } -/** Destroys current breadcrumbs and rebuilds from scratch. - * - * (Needs to be called when a patch is cleared to eliminate children crumbs) +/** Sets up the crumbs to display a @a path. + * + * If @a path is already part of the shown path, it will be selected and the + * children preserved. */ void BreadCrumbBox::build(Path path) @@ -38,8 +39,8 @@ BreadCrumbBox::build(Path path) bool old_enable_signal = _enable_signal; _enable_signal = false; - // Moving to a parent path, just switch the active button - if (path.length() < _full_path.length() && _full_path.substr(0, path.length()) == path) { + // Moving to a path we already contain, just switch the active button + if (_breadcrumbs.size() > 0 && (path.is_parent_of(_full_path) || path == _full_path)) { for (std::list::iterator i = _breadcrumbs.begin(); i != _breadcrumbs.end(); ++i) (*i)->set_active( ((*i)->path() == path) ); @@ -102,5 +103,33 @@ BreadCrumbBox::breadcrumb_clicked(BreadCrumb* crumb) } } + +void +BreadCrumbBox::object_removed(const Path& path) +{ + for (std::list::iterator i = _breadcrumbs.begin(); i != _breadcrumbs.end(); ++i) { + if ((*i)->path() == path) { + // Remove all crumbs after the removed one (inclusive) + for (std::list::iterator j = i; j != _breadcrumbs.end(); ) { + BreadCrumb* bc = *j; + j = _breadcrumbs.erase(j); + remove(*bc); + } + break; + } + } +} + + +void +BreadCrumbBox::object_renamed(const Path& old_path, const Path& new_path) +{ + for (std::list::iterator i = _breadcrumbs.begin(); i != _breadcrumbs.end(); ++i) { + if ((*i)->path() == old_path) + (*i)->set_path(new_path); + } +} + + } // namespace Ingenuity diff --git a/src/progs/ingenuity/BreadCrumbBox.h b/src/progs/ingenuity/BreadCrumbBox.h index aedfade0..35215e43 100644 --- a/src/progs/ingenuity/BreadCrumbBox.h +++ b/src/progs/ingenuity/BreadCrumbBox.h @@ -44,6 +44,9 @@ public: private: void breadcrumb_clicked(BreadCrumb* crumb); + + void object_removed(const Path& path); + void object_renamed(const Path& old_path, const Path& new_path); Path _active_path; Path _full_path; diff --git a/src/progs/ingenuity/ConnectWindow.cpp b/src/progs/ingenuity/ConnectWindow.cpp index 16b2adea..df95f3c3 100644 --- a/src/progs/ingenuity/ConnectWindow.cpp +++ b/src/progs/ingenuity/ConnectWindow.cpp @@ -25,9 +25,11 @@ #include "OSCClientReceiver.h" #include "ThreadedSigClientInterface.h" #include "Store.h" +#include "ControllerFactory.h" #include "PatchController.h" #include "PatchModel.h" #include "App.h" +#include "WindowFactory.h" #ifdef MONOLITHIC_INGENUITY #include "engine/QueuedEngineInterface.h" #include "engine/Engine.h" @@ -349,10 +351,10 @@ ConnectWindow::gtk_callback() ++stage; } else if (stage == 7) { if (App::instance().store()->num_objects() > 0) { - CountedPtr root = App::instance().store()->object("/"); + CountedPtr root = PtrCast(App::instance().store()->object("/")); assert(root); - PatchController* root_controller = new PatchController(root); - root_controller->show_patch_window(); + CountedPtr root_c = PtrCast(ControllerFactory::get_controller(root)); + App::instance().window_factory()->present(root_c); ++stage; } } else if (stage == 8) { diff --git a/src/progs/ingenuity/ControlGroups.cpp b/src/progs/ingenuity/ControlGroups.cpp index 93a1cb5b..45218bdf 100644 --- a/src/progs/ingenuity/ControlGroups.cpp +++ b/src/progs/ingenuity/ControlGroups.cpp @@ -69,7 +69,7 @@ SliderControlGroup::SliderControlGroup(ControlPanel* panel, CountedPtrname(), 0.0, 0.0), + m_name_label(pm->path().name(), 0.0, 0.0), m_range_box(false, 0), m_range_label("Range: "), m_min_spinner(1.0, (pm->is_integer() ? 0 : 4)), // climb rate, digits @@ -90,7 +90,7 @@ SliderControlGroup::SliderControlGroup(ControlPanel* panel, CountedPtrname()); + set_name(pm->path().name()); m_name_label.property_use_markup() = true; m_range_label.property_use_markup() = true; @@ -305,10 +305,10 @@ IntegerControlGroup::IntegerControlGroup(ControlPanel* panel, CountedPtrname()), + m_name_label(pm->path().name()), m_spinner(1.0, 0) { - set_name(pm->name()); + set_name(pm->path().name()); m_spinner.set_range(-99999, 99999); m_spinner.set_value(m_port_model->value()); @@ -379,9 +379,9 @@ ToggleControlGroup::ToggleControlGroup(ControlPanel* panel, CountedPtrname()) + m_name_label(pm->path().name()) { - set_name(pm->name()); + set_name(pm->path().name()); set_value(m_port_model->value()); m_checkbutton.signal_toggled().connect( diff --git a/src/progs/ingenuity/ControlPanel.cpp b/src/progs/ingenuity/ControlPanel.cpp index 57ac7824..90d17c8a 100644 --- a/src/progs/ingenuity/ControlPanel.cpp +++ b/src/progs/ingenuity/ControlPanel.cpp @@ -67,14 +67,7 @@ ControlPanel::init(NodeController* node, size_t poly) for (PortModelList::const_iterator i = node_model->ports().begin(); i != node_model->ports().end(); ++i) { - // FIXME: - if (*i) { - PortController* pc = (PortController*)((*i)->controller()); - assert(pc != NULL); - add_port(pc); - } else { - cerr << "WTF?\n"; - } + add_port(*i); } m_callback_enabled = true; @@ -95,13 +88,9 @@ ControlPanel::find_port(const Path& path) const /** Add a control to the panel for the given port. */ void -ControlPanel::add_port(PortController* port) +ControlPanel::add_port(CountedPtr pm) { - assert(port); - assert(port->model()); - //assert(port->control_panel() == NULL); - - const CountedPtr pm = port->port_model(); + assert(pm); // Already have port, don't add another if (find_port(pm->path()) != NULL) @@ -131,8 +120,6 @@ ControlPanel::add_port(PortController* port) // pm->disconnection_sig.connect(bind(sigc::mem_fun(this, &ControlPanel::disconnection), pm)) } - //port->set_control_panel(this); - Gtk::Requisition controls_size; m_control_box->size_request(controls_size); m_ideal_size.first = controls_size.width; diff --git a/src/progs/ingenuity/ControlPanel.h b/src/progs/ingenuity/ControlPanel.h index a0cc819d..025468d5 100644 --- a/src/progs/ingenuity/ControlPanel.h +++ b/src/progs/ingenuity/ControlPanel.h @@ -59,7 +59,7 @@ public: ControlGroup* find_port(const Path& path) const; - void add_port(PortController* port); + void add_port(CountedPtr port); void remove_port(const Path& path); //void rename_port(const Path& old_path, const Path& new_path); diff --git a/src/progs/ingenuity/ControllerFactory.cpp b/src/progs/ingenuity/ControllerFactory.cpp new file mode 100644 index 00000000..b95bef5b --- /dev/null +++ b/src/progs/ingenuity/ControllerFactory.cpp @@ -0,0 +1,72 @@ +/* This file is part of Ingen. Copyright (C) 2006 Dave Robillard. + * + * Ingen is free software; you can redistribute it and/or modify it under the + * terms of the GNU General Public License as published by the Free Software + * Foundation; either version 2 of the License, or (at your option) any later + * version. + * + * Ingen is distributed in the hope that it will be useful, but WITHOUT ANY + * WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS + * FOR A PARTICULAR PURPOSE. See the GNU General Public License for details. + * + * You should have received a copy of the GNU General Public License along + * with this program; if not, write to the Free Software Foundation, Inc., + * 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA + */ + +#include "ControllerFactory.h" +#include "NodeModel.h" +#include "PatchModel.h" +#include "PortModel.h" +#include "NodeController.h" +#include "PatchController.h" +#include "PortController.h" +#include "DSSIController.h" + +namespace Ingenuity { + + +CountedPtr +ControllerFactory::get_controller(CountedPtr model) +{ + CountedPtr patch_m = PtrCast(model); + if (patch_m && patch_m->controller()) { + assert(dynamic_cast(patch_m->controller().get())); + return PtrCast(patch_m->controller()); + } else if (patch_m) { + CountedPtr pc(new PatchController(patch_m)); + patch_m->set_controller(pc); + return pc; + } + + CountedPtr node_m = PtrCast(model); + if (node_m && node_m->controller()) { + assert(dynamic_cast(node_m->controller().get())); + return PtrCast(node_m->controller()); + } else if (node_m) { + if (node_m->plugin()->type() == PluginModel::DSSI) { + CountedPtr nc(new DSSIController(node_m)); + node_m->set_controller(nc); + return nc; + } else { + CountedPtr nc(new NodeController(node_m)); + node_m->set_controller(nc); + return nc; + } + } + + CountedPtr port_m = PtrCast(model); + if (port_m && port_m->controller()) { + assert(dynamic_cast(port_m->controller().get())); + return PtrCast(port_m->controller()); + } else if (port_m) { + CountedPtr pc(new PortController(port_m)); + port_m->set_controller(pc); + return pc; + } + + return CountedPtr(); +} + + +} // namespace Ingenuity diff --git a/src/progs/ingenuity/ControllerFactory.h b/src/progs/ingenuity/ControllerFactory.h new file mode 100644 index 00000000..7f48c262 --- /dev/null +++ b/src/progs/ingenuity/ControllerFactory.h @@ -0,0 +1,32 @@ +/* This file is part of Ingen. Copyright (C) 2006 Dave Robillard. + * + * Ingen is free software; you can redistribute it and/or modify it under the + * terms of the GNU General Public License as published by the Free Software + * Foundation; either version 2 of the License, or (at your option) any later + * version. + * + * Ingen is distributed in the hope that it will be useful, but WITHOUT ANY + * WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS + * FOR A PARTICULAR PURPOSE. See the GNU General Public License for details. + * + * You should have received a copy of the GNU General Public License along + * with this program; if not, write to the Free Software Foundation, Inc., + * 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA + */ + +#ifndef CONTROLLER_FACTORY_H +#define CONTROLLER_FACTORY_H + +#include "util/CountedPtr.h" +#include "GtkObjectController.h" + +namespace Ingenuity { + +class ControllerFactory { +public: + static CountedPtr get_controller(CountedPtr model); +}; + +} + +#endif // CONTROLLER_FACTORY_H diff --git a/src/progs/ingenuity/DSSIModule.cpp b/src/progs/ingenuity/DSSIModule.cpp index ebcd114c..3d91e69b 100644 --- a/src/progs/ingenuity/DSSIModule.cpp +++ b/src/progs/ingenuity/DSSIModule.cpp @@ -29,8 +29,8 @@ DSSIModule::DSSIModule(OmFlowCanvas* canvas, DSSIController* node) void DSSIModule::on_double_click(GdkEventButton* ev) { - DSSIController* const dc = static_cast(m_node); - if (!dc->attempt_to_show_gui()) + DSSIController* dc = dynamic_cast(m_node); + if (!dc || ! dc->attempt_to_show_gui()) show_control_window(); } diff --git a/src/progs/ingenuity/GtkObjectController.cpp b/src/progs/ingenuity/GtkObjectController.cpp index 798dc1f4..ccaa3ca5 100644 --- a/src/progs/ingenuity/GtkObjectController.cpp +++ b/src/progs/ingenuity/GtkObjectController.cpp @@ -28,8 +28,6 @@ GtkObjectController::GtkObjectController(CountedPtr model) GtkObjectController::~GtkObjectController() { - assert(m_model->controller() == this); - m_model->set_controller(NULL); } } // namespace Ingenuity diff --git a/src/progs/ingenuity/LoadPatchWindow.cpp b/src/progs/ingenuity/LoadPatchWindow.cpp index a847e0fd..dcc08fff 100644 --- a/src/progs/ingenuity/LoadPatchWindow.cpp +++ b/src/progs/ingenuity/LoadPatchWindow.cpp @@ -29,7 +29,6 @@ namespace Ingenuity { LoadPatchWindow::LoadPatchWindow(BaseObjectType* cobject, const Glib::RefPtr& xml) : Gtk::FileChooserDialog(cobject), - m_patch_controller(NULL), m_replace(true) { xml->get_widget("load_patch_poly_from_current_radio", m_poly_from_current_radio); @@ -66,7 +65,7 @@ LoadPatchWindow::LoadPatchWindow(BaseObjectType* cobject, const Glib::RefPtr pc) { m_patch_controller = pc; } @@ -111,7 +110,7 @@ LoadPatchWindow::ok_clicked() if (m_replace) App::instance().engine()->clear_patch(m_patch_controller->model()->path()); - PatchModel* pm = new PatchModel(m_patch_controller->model()->path(), poly); + CountedPtr pm(new PatchModel(m_patch_controller->model()->path(), poly)); pm->filename(get_filename()); pm->set_metadata("filename", get_filename()); pm->set_parent(m_patch_controller->patch_model()->parent()); diff --git a/src/progs/ingenuity/LoadPatchWindow.h b/src/progs/ingenuity/LoadPatchWindow.h index 4f72c6c1..e22b1c1a 100644 --- a/src/progs/ingenuity/LoadPatchWindow.h +++ b/src/progs/ingenuity/LoadPatchWindow.h @@ -21,6 +21,8 @@ #include #include +#include "util/CountedPtr.h" +#include "PatchController.h" namespace Ingenuity { @@ -42,7 +44,7 @@ class LoadPatchWindow : public Gtk::FileChooserDialog public: LoadPatchWindow(BaseObjectType* cobject, const Glib::RefPtr& xml); - void patch_controller(PatchController* pc); + void set_patch(CountedPtr pc); void set_replace() { m_replace = true; } void set_merge() { m_replace = false; } @@ -56,8 +58,8 @@ private: void ok_clicked(); void cancel_clicked(); - PatchController* m_patch_controller; - bool m_replace; + CountedPtr m_patch_controller; + bool m_replace; Gtk::RadioButton* m_poly_from_current_radio; Gtk::RadioButton* m_poly_from_file_radio; diff --git a/src/progs/ingenuity/LoadPluginWindow.cpp b/src/progs/ingenuity/LoadPluginWindow.cpp index 4b4290c4..2b59efef 100644 --- a/src/progs/ingenuity/LoadPluginWindow.cpp +++ b/src/progs/ingenuity/LoadPluginWindow.cpp @@ -20,7 +20,6 @@ #include #include #include "PatchController.h" -#include "PatchView.h" #include "NodeModel.h" #include "App.h" #include "PatchWindow.h" @@ -28,6 +27,8 @@ #include "PatchModel.h" #include "Store.h" #include "ModelEngineInterface.h" +#include "PatchView.h" +#include "OmFlowCanvas.h" using std::cout; using std::cerr; using std::endl; @@ -35,7 +36,6 @@ namespace Ingenuity { LoadPluginWindow::LoadPluginWindow(BaseObjectType* cobject, const Glib::RefPtr& xml) : Gtk::Window(cobject), - m_patch_controller(NULL), m_has_shown(false), m_plugin_name_offset(0), m_new_module_x(0), @@ -135,7 +135,7 @@ LoadPluginWindow::name_changed() * This function MUST be called before using the window in any way! */ void -LoadPluginWindow::patch_controller(PatchController* pc) +LoadPluginWindow::set_patch(CountedPtr pc) { m_patch_controller = pc; @@ -190,9 +190,8 @@ LoadPluginWindow::set_plugin_list(const std::map { m_plugins_liststore->clear(); - CountedPtr plugin = NULL; for (std::map >::const_iterator i = m.begin(); i != m.end(); ++i) { - plugin = (*i).second; + CountedPtr plugin = (*i).second; Gtk::TreeModel::iterator iter = m_plugins_liststore->append(); Gtk::TreeModel::Row row = *iter; @@ -307,12 +306,12 @@ LoadPluginWindow::add_clicked() dialog.run(); } else { - const string path = m_patch_controller->model()->base_path() + name; + const string path = m_patch_controller->model()->path().base() + name; NodeModel* nm = new NodeModel(plugin, path); nm->polyphonic(polyphonic); if (m_new_module_x == 0 && m_new_module_y == 0) { - m_patch_controller->view()->canvas()->get_new_module_location( + m_patch_controller->get_view()->canvas()->get_new_module_location( m_new_module_x, m_new_module_y); } nm->x(m_new_module_x); diff --git a/src/progs/ingenuity/LoadPluginWindow.h b/src/progs/ingenuity/LoadPluginWindow.h index 9d875358..1654b777 100644 --- a/src/progs/ingenuity/LoadPluginWindow.h +++ b/src/progs/ingenuity/LoadPluginWindow.h @@ -87,7 +87,7 @@ class LoadPluginWindow : public Gtk::Window public: LoadPluginWindow(BaseObjectType* cobject, const Glib::RefPtr& xml); - void patch_controller(PatchController* pc); + void set_patch(CountedPtr pc); void set_plugin_list(const std::map >& m); void set_next_module_location(double x, double y) @@ -113,7 +113,7 @@ private: void plugin_selection_changed(); string generate_module_name(int offset = 0); - PatchController* m_patch_controller; + CountedPtr m_patch_controller; bool m_has_shown; // plugin list only populated on show to speed patch window creation Glib::RefPtr m_plugins_liststore; diff --git a/src/progs/ingenuity/LoadSubpatchWindow.cpp b/src/progs/ingenuity/LoadSubpatchWindow.cpp index 832c917f..486c92b4 100644 --- a/src/progs/ingenuity/LoadSubpatchWindow.cpp +++ b/src/progs/ingenuity/LoadSubpatchWindow.cpp @@ -33,7 +33,6 @@ namespace Ingenuity { LoadSubpatchWindow::LoadSubpatchWindow(BaseObjectType* cobject, const Glib::RefPtr& xml) : Gtk::FileChooserDialog(cobject), - m_patch_controller(NULL), m_new_module_x(0), m_new_module_y(0) { @@ -74,7 +73,7 @@ LoadSubpatchWindow::LoadSubpatchWindow(BaseObjectType* cobject, const Glib::RefP * This function MUST be called before using the window in any way! */ void -LoadSubpatchWindow::patch_controller(PatchController* pc) +LoadSubpatchWindow::set_patch(CountedPtr pc) { m_patch_controller = pc; @@ -130,7 +129,7 @@ LoadSubpatchWindow::enable_poly_spinner() void LoadSubpatchWindow::ok_clicked() { - assert(m_patch_controller != NULL); + assert(m_patch_controller); assert(m_patch_controller->model()); const string filename = get_filename(); @@ -148,13 +147,13 @@ LoadSubpatchWindow::ok_clicked() poly = m_patch_controller->patch_model()->poly(); if (m_new_module_x == 0 && m_new_module_y == 0) { - m_patch_controller->view()->canvas()->get_new_module_location( + m_patch_controller->get_view()->canvas()->get_new_module_location( m_new_module_x, m_new_module_y); } - PatchModel* pm = new PatchModel(m_patch_controller->model()->base_path() + name, poly); + CountedPtr pm(new PatchModel(m_patch_controller->model()->path().base() + name, poly)); pm->filename(filename); - pm->set_parent(m_patch_controller->model().get()); + pm->set_parent(m_patch_controller->model()); pm->x(m_new_module_x); pm->y(m_new_module_y); //if (name == "") diff --git a/src/progs/ingenuity/LoadSubpatchWindow.h b/src/progs/ingenuity/LoadSubpatchWindow.h index c4696628..e33880e8 100644 --- a/src/progs/ingenuity/LoadSubpatchWindow.h +++ b/src/progs/ingenuity/LoadSubpatchWindow.h @@ -21,6 +21,8 @@ #include "PluginModel.h" #include #include +#include "util/CountedPtr.h" +#include "PatchController.h" namespace Ingenuity { @@ -39,7 +41,7 @@ class LoadSubpatchWindow : public Gtk::FileChooserDialog public: LoadSubpatchWindow(BaseObjectType* cobject, const Glib::RefPtr& xml); - void patch_controller(PatchController* pc); + void set_patch(CountedPtr pc); void set_next_module_location(double x, double y) { m_new_module_x = x; m_new_module_y = y; } @@ -56,7 +58,7 @@ private: void ok_clicked(); void cancel_clicked(); - PatchController* m_patch_controller; + CountedPtr m_patch_controller; double m_new_module_x; double m_new_module_y; diff --git a/src/progs/ingenuity/Loader.cpp b/src/progs/ingenuity/Loader.cpp index fed49603..0ca6f966 100644 --- a/src/progs/ingenuity/Loader.cpp +++ b/src/progs/ingenuity/Loader.cpp @@ -211,14 +211,14 @@ Loader::m_thread_function(void *) void -Loader::load_patch(PatchModel* model, bool wait, bool merge) +Loader::load_patch(CountedPtr model, bool wait, bool merge) { set_event(new LoadPatchEvent(m_patch_librarian, model, wait, merge)); } void -Loader::save_patch(PatchModel* model, const string& filename, bool recursive) +Loader::save_patch(CountedPtr model, const string& filename, bool recursive) { cout << "[Loader] Saving patch " << filename << endl; set_event(new SavePatchEvent(m_patch_librarian, model, filename, recursive)); diff --git a/src/progs/ingenuity/Loader.h b/src/progs/ingenuity/Loader.h index 7ff6f189..58c301af 100644 --- a/src/progs/ingenuity/Loader.h +++ b/src/progs/ingenuity/Loader.h @@ -53,15 +53,15 @@ protected: class LoadPatchEvent : public LoaderEvent { public: - LoadPatchEvent(PatchLibrarian* pl, PatchModel* model, bool wait, bool merge) + LoadPatchEvent(PatchLibrarian* pl, CountedPtr model, bool wait, bool merge) : m_patch_librarian(pl), m_patch_model(model), m_wait(wait), m_merge(merge) {} virtual ~LoadPatchEvent() {} void execute(); private: - PatchLibrarian* m_patch_librarian; - PatchModel* m_patch_model; - bool m_wait; - bool m_merge; + PatchLibrarian* m_patch_librarian; + CountedPtr m_patch_model; + bool m_wait; + bool m_merge; }; @@ -72,15 +72,15 @@ private: class SavePatchEvent : public LoaderEvent { public: - SavePatchEvent(PatchLibrarian* pl, PatchModel* pm, const string& filename, bool recursive) + SavePatchEvent(PatchLibrarian* pl, CountedPtr pm, const string& filename, bool recursive) : m_patch_librarian(pl), m_patch_model(pm), m_filename(filename), m_recursive(recursive) {} virtual ~SavePatchEvent() {} void execute(); private: - PatchLibrarian* m_patch_librarian; - PatchModel* m_patch_model; - string m_filename; - bool m_recursive; + PatchLibrarian* m_patch_librarian; + CountedPtr m_patch_model; + string m_filename; + bool m_recursive; }; /* @@ -129,8 +129,8 @@ public: void launch(); void exit() { m_thread_exit_flag = true; } - void load_patch(PatchModel* model, bool wait, bool merge); - void save_patch(PatchModel* model, const string& filename, bool recursive); + void load_patch(CountedPtr model, bool wait, bool merge); + void save_patch(CountedPtr model, const string& filename, bool recursive); //void load_session(const string& filename); //void save_session(const string& filename); diff --git a/src/progs/ingenuity/Makefile.am b/src/progs/ingenuity/Makefile.am index 7cc405c1..5c4f5eed 100644 --- a/src/progs/ingenuity/Makefile.am +++ b/src/progs/ingenuity/Makefile.am @@ -43,6 +43,8 @@ ingenuity_SOURCES = \ PortController.cpp \ DSSIController.h \ DSSIController.cpp \ + ControllerFactory.h \ + ControllerFactory.cpp \ LoadPluginWindow.h \ LoadPluginWindow.cpp \ LoadPatchWindow.h \ @@ -61,6 +63,8 @@ ingenuity_SOURCES = \ PatchView.cpp \ PatchWindow.h \ PatchWindow.cpp \ + WindowFactory.h \ + WindowFactory.cpp \ OmFlowCanvas.h \ OmFlowCanvas.cpp \ ../../common/types.h \ diff --git a/src/progs/ingenuity/NewSubpatchWindow.cpp b/src/progs/ingenuity/NewSubpatchWindow.cpp index 0b94e3d2..7f434445 100644 --- a/src/progs/ingenuity/NewSubpatchWindow.cpp +++ b/src/progs/ingenuity/NewSubpatchWindow.cpp @@ -18,10 +18,10 @@ #include "ModelEngineInterface.h" #include "NewSubpatchWindow.h" #include "PatchController.h" -#include "PatchView.h" -#include "OmFlowCanvas.h" #include "NodeModel.h" #include "PatchModel.h" +#include "PatchView.h" +#include "OmFlowCanvas.h" namespace Ingenuity { @@ -50,7 +50,7 @@ NewSubpatchWindow::NewSubpatchWindow(BaseObjectType* cobject, const Glib::RefPtr * This function MUST be called before using the window in any way! */ void -NewSubpatchWindow::patch_controller(PatchController* pc) +NewSubpatchWindow::set_patch(CountedPtr pc) { m_patch_controller = pc; } @@ -83,11 +83,11 @@ void NewSubpatchWindow::ok_clicked() { PatchModel* pm = new PatchModel( - m_patch_controller->model()->base_path() + m_name_entry->get_text(), + m_patch_controller->model()->path().base() + m_name_entry->get_text(), m_poly_spinbutton->get_value_as_int()); if (m_new_module_x == 0 && m_new_module_y == 0) { - m_patch_controller->view()->canvas()->get_new_module_location( + m_patch_controller->get_view()->canvas()->get_new_module_location( m_new_module_x, m_new_module_y); } diff --git a/src/progs/ingenuity/NewSubpatchWindow.h b/src/progs/ingenuity/NewSubpatchWindow.h index 46fa9e1c..32560dde 100644 --- a/src/progs/ingenuity/NewSubpatchWindow.h +++ b/src/progs/ingenuity/NewSubpatchWindow.h @@ -14,13 +14,14 @@ * 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA */ - #ifndef NEWSUBPATCHWINDOW_H #define NEWSUBPATCHWINDOW_H #include "PluginModel.h" #include #include +#include "util/CountedPtr.h" +#include "PatchController.h" namespace Ingenuity { @@ -39,7 +40,7 @@ class NewSubpatchWindow : public Gtk::Window public: NewSubpatchWindow(BaseObjectType* cobject, const Glib::RefPtr& xml); - void patch_controller(PatchController* pc); + void set_patch(CountedPtr pc); void set_next_module_location(double x, double y) { m_new_module_x = x; m_new_module_y = y; } @@ -49,7 +50,7 @@ private: void ok_clicked(); void cancel_clicked(); - PatchController* m_patch_controller; + CountedPtr m_patch_controller; double m_new_module_x; double m_new_module_y; diff --git a/src/progs/ingenuity/NodeController.cpp b/src/progs/ingenuity/NodeController.cpp index 1c340516..677bc8b5 100644 --- a/src/progs/ingenuity/NodeController.cpp +++ b/src/progs/ingenuity/NodeController.cpp @@ -25,6 +25,7 @@ #include "GtkObjectController.h" #include "NodeControlWindow.h" #include "OmModule.h" +#include "ControllerFactory.h" #include "PatchController.h" #include "OmFlowCanvas.h" #include "RenameWindow.h" @@ -47,17 +48,13 @@ NodeController::NodeController(CountedPtr model) m_properties_window(NULL), m_bridge_port(NULL) { - assert(!model->controller()); - model->set_controller(this); - // Create port controllers for (PortModelList::const_iterator i = node_model()->ports().begin(); i != node_model()->ports().end(); ++i) { assert(!(*i)->controller()); assert((*i)->parent()); assert((*i)->parent().get() == node_model().get()); - // FIXME: leak - PortController* const pc = new PortController(*i); + CountedPtr pc = PtrCast(ControllerFactory::get_controller(*i)); assert((*i)->controller() == pc); // PortController() does this } @@ -92,6 +89,7 @@ NodeController::NodeController(CountedPtr model) } model->new_port_sig.connect(sigc::mem_fun(this, &NodeController::add_port)); + model->destroyed_sig.connect(sigc::mem_fun(this, &NodeController::destroy)); } @@ -101,6 +99,13 @@ NodeController::~NodeController() } +void +NodeController::destroy() +{ + delete this; +} + + void NodeController::create_module(OmFlowCanvas* canvas) { @@ -144,7 +149,7 @@ NodeController::set_path(const Path& new_path) i != node_model()->ports().end(); ++i) { GtkObjectController* const pc = (GtkObjectController*)((*i)->controller()); assert(pc != NULL); - pc->set_path(m_model->path().base_path() + pc->model()->name()); + pc->set_path(m_model->path().base() + pc->model()->name()); } // Handle bridge port, if this node represents one @@ -160,7 +165,7 @@ NodeController::set_path(const Path& new_path) */ } - +#if 0 void NodeController::destroy() { @@ -178,7 +183,7 @@ NodeController::destroy() //if (m_module != NULL) // delete m_module; } - +#endif void NodeController::metadata_update(const string& key, const string& value) @@ -197,8 +202,8 @@ NodeController::metadata_update(const string& key, const string& value) } } - if (m_bridge_port != NULL) - m_bridge_port->metadata_update(key, value); + //if (m_bridge_port != NULL) + // m_bridge_port->metadata_update(key, value); GtkObjectController::metadata_update(key, value); } @@ -209,12 +214,11 @@ NodeController::add_port(CountedPtr pm) { assert(pm->parent().get() == node_model().get()); assert(pm->parent() == node_model()); - assert(node_model()->get_port(pm->name()) == pm); + assert(node_model()->get_port(pm->path().name()) == pm); //cout << "[NodeController] Adding port " << pm->path() << endl; - // FIXME: leak - PortController* pc = new PortController(pm); + CountedPtr pc = PtrCast(ControllerFactory::get_controller(pm)); assert(pm->controller() == pc); if (m_module != NULL) { @@ -259,10 +263,10 @@ NodeController::show_rename_window() Glib::RefPtr xml = GladeFactory::new_glade_reference("rename_win"); xml->get_widget_derived("rename_win", win); - PatchController* parent = ((PatchController*)node_model()->parent()->controller()); - assert(parent != NULL); + CountedPtr parent = PtrCast(node_model()->parent()->controller()); + assert(parent); - if (parent->window() != NULL) + if (parent->window()) win->set_transient_for(*parent->window()); win->set_object(this); @@ -298,7 +302,7 @@ NodeController::on_menu_clone() clone_name = clone_name + clone_postfix; - const string path = node_model()->parent_patch()->base_path() + clone_name; + const string path = node_model()->parent_patch()->base() + clone_name; NodeModel* nm = new NodeModel(node_model()->plugin(), path); nm->polyphonic(node_model()->polyphonic()); nm->x(node_model()->x() + 20); @@ -324,36 +328,33 @@ NodeController::on_menu_disconnect_all() void NodeController::show_properties_window() { - PatchController* parent = ((PatchController*)node_model()->parent()->controller()); - assert(parent != NULL); + CountedPtr parent = PtrCast(node_model()->parent()->controller()); + assert(parent); - if (m_properties_window == NULL) { + if (m_properties_window) { Glib::RefPtr xml = GladeFactory::new_glade_reference("node_properties_win"); xml->get_widget_derived("node_properties_win", m_properties_window); } - assert(m_properties_window != NULL); - assert(parent != NULL); + assert(m_properties_window); + assert(parent); m_properties_window->set_node(node_model()); - if (parent->window() != NULL) + if (parent->window()) m_properties_window->set_transient_for(*parent->window()); m_properties_window->show(); } /** Create all (visual) ports and add them to module (and resize it). + * FIXME: this doesn't belong here */ void NodeController::create_all_ports() { - assert(m_module != NULL); + assert(m_module); - PortController* pc = NULL; for (PortModelList::const_iterator i = node_model()->ports().begin(); i != node_model()->ports().end(); ++i) { - pc = dynamic_cast((*i)->controller()); - // FIXME: leak - if (pc == NULL) - pc = new PortController(*i); + CountedPtr pc = PtrCast(ControllerFactory::get_controller(*i)); pc->create_port(m_module); } diff --git a/src/progs/ingenuity/NodeController.h b/src/progs/ingenuity/NodeController.h index 84ac09cd..c0ee6ea9 100644 --- a/src/progs/ingenuity/NodeController.h +++ b/src/progs/ingenuity/NodeController.h @@ -20,14 +20,13 @@ #include #include #include "util/Path.h" +#include "util/CountedPtr.h" #include "GtkObjectController.h" #include "NodeModel.h" using std::string; using namespace Ingen::Client; -template class CountedPtr; - namespace Ingen { namespace Client { class MetadataModel; class PortModel; @@ -49,10 +48,7 @@ class OmFlowCanvas; class NodeController : public GtkObjectController { public: - NodeController(CountedPtr model); virtual ~NodeController(); - - virtual void destroy(); virtual void metadata_update(const string& key, const string& value); @@ -68,10 +64,10 @@ public: OmModule* module() { return m_module; } - void bridge_port(PortController* port) { m_bridge_port = port; } - PortController* as_port() { return m_bridge_port; } + //void bridge_port(PortController* port) { m_bridge_port = port; } + //PortController* as_port() { return m_bridge_port; } - CountedPtr node_model() { return m_model; } + CountedPtr node_model() { return PtrCast(m_model); } NodeControlWindow* control_window() { return m_control_window; } void control_window(NodeControlWindow* cw) { m_control_window = cw; } @@ -89,6 +85,12 @@ public: virtual void disable_controls_menuitem(); protected: + friend class ControllerFactory; + + NodeController(CountedPtr model); + + virtual void destroy(); + virtual void add_port(CountedPtr pm); void create_all_ports(); diff --git a/src/progs/ingenuity/NodePropertiesWindow.cpp b/src/progs/ingenuity/NodePropertiesWindow.cpp index 2ee79527..a6bdfce1 100644 --- a/src/progs/ingenuity/NodePropertiesWindow.cpp +++ b/src/progs/ingenuity/NodePropertiesWindow.cpp @@ -26,7 +26,6 @@ using std::string; NodePropertiesWindow::NodePropertiesWindow(BaseObjectType* cobject, const Glib::RefPtr& glade_xml) : Gtk::Window(cobject) -, m_node_model(NULL) { glade_xml->get_widget("node_properties_path_label", m_node_path_label); glade_xml->get_widget("node_properties_polyphonic_checkbutton", m_node_polyphonic_toggle); diff --git a/src/progs/ingenuity/OmFlowCanvas.cpp b/src/progs/ingenuity/OmFlowCanvas.cpp index a6e3617e..d530bca0 100644 --- a/src/progs/ingenuity/OmFlowCanvas.cpp +++ b/src/progs/ingenuity/OmFlowCanvas.cpp @@ -101,22 +101,21 @@ OmFlowCanvas::connect(const Port* src_port, const Port* dst_port) if (src->model()->type() == PortModel::MIDI && dst->model()->type() == PortModel::CONTROL) { - // FIXME: leaks? - PluginModel* pm = new PluginModel(PluginModel::Internal, "", "midi_control_in", ""); - NodeModel* nm = new NodeModel(pm, m_patch_controller->model()->base_path() - + src->name() + "-" + dst->name()); + CountedPtr pm(new PluginModel(PluginModel::Internal, "", "midi_control_in", "")); + CountedPtr nm(new NodeModel(pm, m_patch_controller->model()->path().base() + + src->name() + "-" + dst->name())); nm->x(dst->module()->property_x() - dst->module()->width() - 20); nm->y(dst->module()->property_y()); - App::instance().engine()->create_node_from_model(nm); + App::instance().engine()->create_node_from_model(nm.get()); App::instance().engine()->connect(src->model()->path(), nm->path() + "/MIDI_In"); App::instance().engine()->connect(nm->path() + "/Out_(CR)", dst->model()->path()); App::instance().engine()->midi_learn(nm->path()); // Set control node range to port's user range - App::instance().engine()->set_port_value_queued(nm->path().base_path() + "Min", + App::instance().engine()->set_port_value_queued(nm->path().base() + "Min", atof(dst->model()->get_metadata("user-min").c_str())); - App::instance().engine()->set_port_value_queued(nm->path().base_path() + "Max", + App::instance().engine()->set_port_value_queued(nm->path().base() + "Max", atof(dst->model()->get_metadata("user-max").c_str())); } else { App::instance().engine()->connect(src->model()->path(), @@ -195,7 +194,7 @@ OmFlowCanvas::generate_port_name(const string& base) { void OmFlowCanvas::menu_add_port(const string& name, const string& type, bool is_output) { - const Path& path = m_patch_controller->path().base_path() + generate_port_name(name); + const Path& path = m_patch_controller->path().base() + generate_port_name(name); App::instance().engine()->create_port(path, type, is_output); char temp_buf[16]; @@ -224,7 +223,7 @@ void OmFlowCanvas::menu_add_audio_input() { string name = "audio_in"; - App::instance().engine()->create_port(m_patch_controller->path().base_path() + name, "AUDIO", false); + App::instance().engine()->create_port(m_patch_controller->path().base() + name, "AUDIO", false); } @@ -232,7 +231,7 @@ void OmFlowCanvas::menu_add_audio_output() { string name = "audio_out"; - App::instance().engine()->create_port(m_patch_controller->path().base_path() + name, "AUDIO", true); + App::instance().engine()->create_port(m_patch_controller->path().base() + name, "AUDIO", true); } @@ -240,7 +239,7 @@ void OmFlowCanvas::menu_add_control_input() { string name = "control_in"; - App::instance().engine()->create_port(m_patch_controller->path().base_path() + name, "CONTROL", false); + App::instance().engine()->create_port(m_patch_controller->path().base() + name, "CONTROL", false); } @@ -248,7 +247,7 @@ void OmFlowCanvas::menu_add_control_output() { string name = "control_out"; - App::instance().engine()->create_port(m_patch_controller->path().base_path() + name, "CONTROL", true); + App::instance().engine()->create_port(m_patch_controller->path().base() + name, "CONTROL", true); } @@ -256,7 +255,7 @@ void OmFlowCanvas::menu_add_midi_input() { string name = "midi_in"; - App::instance().engine()->create_port(m_patch_controller->path().base_path() + name, "MIDI", false); + App::instance().engine()->create_port(m_patch_controller->path().base() + name, "MIDI", false); } @@ -264,7 +263,7 @@ void OmFlowCanvas::menu_add_midi_output() { string name = "midi_out"; - App::instance().engine()->create_port(m_patch_controller->path().base_path() + name, "MIDI", true); + App::instance().engine()->create_port(m_patch_controller->path().base() + name, "MIDI", true); } */ diff --git a/src/progs/ingenuity/OmModule.cpp b/src/progs/ingenuity/OmModule.cpp index e34665a7..dd6f3da5 100644 --- a/src/progs/ingenuity/OmModule.cpp +++ b/src/progs/ingenuity/OmModule.cpp @@ -31,11 +31,11 @@ namespace Ingenuity { OmModule::OmModule(OmFlowCanvas* canvas, NodeController* node) -: LibFlowCanvas::Module(canvas, node->node_model()->name(), +: LibFlowCanvas::Module(canvas, node->node_model()->path().name(), node->node_model()->x(), node->node_model()->y()), m_node(node) { - assert(m_node != NULL); + assert(m_node); /*if (node_model()->polyphonic() && node_model()->parent() != NULL && node_model()->parent_patch()->poly() > 1) { diff --git a/src/progs/ingenuity/OmModule.h b/src/progs/ingenuity/OmModule.h index 2a31a84f..a3d56187 100644 --- a/src/progs/ingenuity/OmModule.h +++ b/src/progs/ingenuity/OmModule.h @@ -21,6 +21,7 @@ #include #include #include +#include "util/CountedPtr.h" #include "NodeController.h" using std::string; diff --git a/src/progs/ingenuity/OmPatchPort.cpp b/src/progs/ingenuity/OmPatchPort.cpp index b900a40c..0d3acf76 100644 --- a/src/progs/ingenuity/OmPatchPort.cpp +++ b/src/progs/ingenuity/OmPatchPort.cpp @@ -29,7 +29,7 @@ using namespace Ingen::Client; namespace Ingenuity { OmPatchPort::OmPatchPort(OmPortModule* module, CountedPtr pm) -: Port(module, pm->name(), !pm->is_input(), App::instance().configuration()->get_port_color(pm.get())), +: Port(module, pm->path().name(), !pm->is_input(), App::instance().configuration()->get_port_color(pm.get())), m_port_model(pm) { assert(module); diff --git a/src/progs/ingenuity/OmPort.cpp b/src/progs/ingenuity/OmPort.cpp index 69406b97..e2a32652 100644 --- a/src/progs/ingenuity/OmPort.cpp +++ b/src/progs/ingenuity/OmPort.cpp @@ -29,7 +29,7 @@ using namespace Ingen::Client; namespace Ingenuity { OmPort::OmPort(Module* module, CountedPtr pm) -: Port(module, pm->name(), pm->is_input(), App::instance().configuration()->get_port_color(pm.get())), +: Port(module, pm->path().name(), pm->is_input(), App::instance().configuration()->get_port_color(pm.get())), m_port_model(pm) { assert(module); diff --git a/src/progs/ingenuity/PatchController.cpp b/src/progs/ingenuity/PatchController.cpp index 6ef90d32..8dee4cd2 100644 --- a/src/progs/ingenuity/PatchController.cpp +++ b/src/progs/ingenuity/PatchController.cpp @@ -44,6 +44,7 @@ #include "DSSIController.h" #include "PatchModel.h" #include "Store.h" +#include "ControllerFactory.h" using std::cerr; using std::cout; using std::endl; using namespace Ingen::Client; @@ -55,26 +56,12 @@ PatchController::PatchController(CountedPtr model) : NodeController(model), m_properties_window(NULL), m_window(NULL), - m_patch_view(NULL), m_patch_model(model), m_module_x(0), m_module_y(0) { - assert(model->path().length() > 0); - assert(model->controller() == this); // NodeController() does this - assert(m_patch_model == model); - -/* FIXME if (model->path() != "/") { - PatchController* parent = Store::instance().patch(model->path().parent()); - if (parent != NULL) - parent->add_subpatch(this); - else - cerr << "[PatchController] " << path() << " ERROR: Parent not found." << endl; - }*/ - //model->new_port_sig.connect(sigc::mem_fun(this, &PatchController::add_port)); model->new_node_sig.connect(sigc::mem_fun(this, &PatchController::add_node)); - model->removed_node_sig.connect(sigc::mem_fun(this, &PatchController::remove_node)); model->new_connection_sig.connect(sigc::mem_fun(this, &PatchController::connection)); model->removed_connection_sig.connect(sigc::mem_fun(this, &PatchController::disconnection)); } @@ -82,27 +69,23 @@ PatchController::PatchController(CountedPtr model) PatchController::~PatchController() { - if (m_patch_view != NULL) { + if (m_patch_view) { claim_patch_view(); - m_patch_view->hide(); - delete m_patch_view; - m_patch_view = NULL; } - if (m_control_window != NULL) { + if (m_control_window) { m_control_window->hide(); delete m_control_window; m_control_window = NULL; } - if (m_window != NULL) { - m_window->hide(); + if (m_window) { delete m_window; m_window = NULL; } } - +#if 0 void PatchController::clear() { @@ -113,8 +96,8 @@ PatchController::clear() size_t remaining = nodes.size(); while (remaining > 0) { - NodeController* const nc = (NodeController*)(*nodes.begin()).second->controller(); - assert(nc != NULL); + CountedPtr nc = (*nodes.begin()).second->controller(); + assert(nc); nc->destroy(); assert(nodes.size() == remaining - 1); --remaining; @@ -123,13 +106,14 @@ PatchController::clear() patch_model()->clear(); - if (m_patch_view != NULL) { - assert(m_patch_view->canvas() != NULL); + if (m_patch_view) { + assert(m_patch_view->canvas()); m_patch_view->canvas()->destroy(); } } +#endif - +#if 0 void PatchController::destroy() { @@ -139,9 +123,8 @@ PatchController::destroy() size_t remaining = nodes.size(); while (remaining > 0) { - NodeController* const nc = (NodeController*) - (*nodes.begin()).second->controller(); - assert(nc != NULL); + CountedPtr nc = (*nodes.begin()).second->controller(); + assert(nc); nc->destroy(); assert(nodes.size() == remaining - 1); --remaining; @@ -158,14 +141,22 @@ PatchController::destroy() //Store::instance().remove_object(this); // Delete self from parent (this will delete model) - /*if (patch_model()->parent() != NULL) { + /*if (patch_model()->parent()) { PatchController* const parent = (PatchController*)patch_model()->parent()->controller(); - assert(parent != NULL); + assert(parent); parent->remove_node(name()); } else { //delete m_model; }*/ } +#endif + + +void +PatchController::destroy() +{ + delete this; +} void @@ -189,25 +180,25 @@ PatchController::set_path(const Path& new_path) i != patch_model()->nodes().end(); ++i) { const NodeModel* const nm = (*i).second.get(); assert(nm ); - NodeController* const nc = ((NodeController*)nm->controller()); + CountedPtr nc = PtrCast(nm->controller()); assert(nc ); - nc->set_path(new_path.base_path() + nc->node_model()->name()); + nc->set_path(new_path.base() + nc->node_model()->path().name()); } #ifdef DEBUG // Be sure ports were renamed by their bridge nodes for (PortModelList::const_iterator i = node_model()->ports().begin(); i != node_model()->ports().end(); ++i) { - GtkObjectController* const pc = (GtkObjectController*)((*i)->controller()); - assert(pc ); + CountedPtr pc = PtrCast((*i)->controller()); + assert(pc); assert(pc->path().parent()== new_path); } #endif App::instance().patch_tree()->patch_renamed(old_path, new_path); - if (m_window) - m_window->patch_renamed(new_path); + /*if (m_window) + m_window->patch_renamed(new_path);*/ if (m_control_window) m_control_window->set_title(new_path + " Controls"); @@ -215,11 +206,10 @@ PatchController::set_path(const Path& new_path) if (m_module) m_module->name(new_path.name()); - PatchController* parent = dynamic_cast( - patch_model()->parent()->controller()); + CountedPtr parent = PtrCast(patch_model()->parent()->controller()); - if (parent && parent->window()) - parent->window()->node_renamed(old_path, new_path); + //if (parent && parent->window()) + // parent->window()->node_renamed(old_path, new_path); //remove_from_store(); GtkObjectController::set_path(new_path); @@ -249,11 +239,12 @@ PatchController::create_module(OmFlowCanvas* canvas) //cerr << "Creating patch module " << m_model->path() << endl; - assert(canvas != NULL); + assert(canvas); assert(m_module == NULL); assert(!m_patch_view || canvas != m_patch_view->canvas()); - m_module = new SubpatchModule(canvas, this); + // FIXME: weirdo using model->controller() to get shared_ptr_from_this.. + m_module = new SubpatchModule(canvas, PtrCast(m_model->controller())); create_all_ports(); } @@ -261,23 +252,26 @@ PatchController::create_module(OmFlowCanvas* canvas) } -void -PatchController::create_view() +CountedPtr +PatchController::get_view() { - assert(m_patch_view == NULL); + if (m_patch_view) + return m_patch_view; Glib::RefPtr xml = GladeFactory::new_glade_reference(); - xml->get_widget_derived("patch_view_vbox", m_patch_view); - assert(m_patch_view != NULL); + PatchView* pv = NULL; + xml->get_widget_derived("patch_view_vbox", pv); + assert(pv); + m_patch_view = CountedPtr(pv); m_patch_view->patch_controller(this); - assert(m_patch_view->canvas() != NULL); + assert(m_patch_view->canvas()); // Create modules for nodes for (NodeModelMap::const_iterator i = patch_model()->nodes().begin(); i != patch_model()->nodes().end(); ++i) { - NodeModel* const nm = (*i).second.get(); + const CountedPtr nm = (*i).second; string val = nm->get_metadata("module-x"); if (val != "") @@ -294,9 +288,7 @@ PatchController::create_view() nm->y(y); } - NodeController* nc = ((NodeController*)nm->controller()); - if (!nc) - nc = create_controller_for_node(nm); + CountedPtr nc = PtrCast(ControllerFactory::get_controller(nm)); assert(nc); assert(nm->controller() == nc); @@ -308,7 +300,7 @@ PatchController::create_view() // Create pseudo modules for ports (ports on this canvas, not on our module) for (PortModelList::const_iterator i = patch_model()->ports().begin(); i != patch_model()->ports().end(); ++i) { - PortController* const pc = dynamic_cast((*i)->controller()); + CountedPtr pc = PtrCast((*i)->controller()); assert(pc); pc->create_module(m_patch_view->canvas()); } @@ -323,6 +315,8 @@ PatchController::create_view() // Set run checkbox if (patch_model()->enabled()) m_patch_view->enable(); + + return m_patch_view; } @@ -345,7 +339,7 @@ PatchController::show_properties_window() void PatchController::connection(CountedPtr cm) { - if (m_patch_view != NULL) { + if (m_patch_view) { // Deal with port "anonymous nodes" for this patch's own ports... const Path& src_parent_path = cm->src_port_path().parent(); @@ -365,97 +359,50 @@ PatchController::connection(CountedPtr cm) } -NodeController* -PatchController::create_controller_for_node(CountedPtr node) -{ - assert(!node->controller()); - NodeController* nc = NULL; - - CountedPtr patch(node); - if (patch) { - assert(patch == node); - assert(patch->parent() == m_patch_model); - nc = new PatchController(patch); - } else { - assert(node->plugin()); - if (node->plugin()->type() == PluginModel::DSSI) - nc = new DSSIController(node); - else - nc = new NodeController(node); - } - - assert(node->controller() == nc); - return nc; -} - - /** Add a child node to this patch. * * This is for plugin nodes and patches, and is responsible for creating the - * GtkObjectController for @a object (and through that the View if necessary) + * GtkObjectController for @a node (and through that the View if necessary) */ void -PatchController::add_node(CountedPtr object) +PatchController::add_node(CountedPtr node) { - assert(object); - assert(object->parent() == m_patch_model); - assert(patch_model()->get_node(object->name())); - - /*if (patch_model()->get_node(nm->name()) != NULL) { - cerr << "Ignoring existing\n"; - // Node already exists, ignore - //delete nm; - } else {*/ - + assert(node); + assert(node->parent().get() == m_patch_model.get()); + assert(node->parent() == m_patch_model); + assert(patch_model()->get_node(node->path().name())); - CountedPtr node(object); - if (node) { - assert(node->parent() == m_patch_model); + assert(node->parent() == m_patch_model); - NodeController* nc = create_controller_for_node(node); - assert(nc); - assert(node->controller() == nc); + CountedPtr nc = PtrCast(ControllerFactory::get_controller(node)); + assert(nc); + assert(node->controller() == nc); - if (m_patch_view != NULL) { - double x, y; - m_patch_view->canvas()->get_new_module_location(x, y); - node->x(x); - node->y(y); - - // Set zoom to 1.0 so module isn't messed up (Death to GnomeCanvas) - float old_zoom = m_patch_view->canvas()->get_zoom(); - if (old_zoom != 1.0) - m_patch_view->canvas()->set_zoom(1.0); - - nc->create_module(m_patch_view->canvas()); - assert(nc->module()); - nc->module()->resize(); - - // Reset zoom - if (old_zoom != 1.0) { - m_patch_view->canvas()->set_zoom(old_zoom); - nc->module()->zoom(old_zoom); - } - } + if (m_patch_view) { + double x, y; + m_patch_view->canvas()->get_new_module_location(x, y); + node->x(x); + node->y(y); - } -} + // Set zoom to 1.0 so module isn't messed up (Death to GnomeCanvas) + float old_zoom = m_patch_view->canvas()->get_zoom(); + if (old_zoom != 1.0) + m_patch_view->canvas()->set_zoom(1.0); + nc->create_module(m_patch_view->canvas()); + assert(nc->module()); + nc->module()->resize(); -/** Removes a node from this patch. - */ -void -PatchController::remove_node(const string& name) -{ - assert(name.find("/") == string::npos); - assert(!m_patch_model->get_node(name)); + // Reset zoom + if (old_zoom != 1.0) { + m_patch_view->canvas()->set_zoom(old_zoom); + nc->module()->zoom(old_zoom); + } + } - // Update breadcrumbs if necessary - if (m_window) - m_window->node_removed(name); } - +#if 0 /** Add a port to this patch. * * Will add a port to the subpatch module and the control window, if they @@ -466,34 +413,33 @@ PatchController::add_port(CountedPtr pm) { assert(pm); assert(pm->parent() == m_patch_model); - assert(patch_model()->get_port(pm->name())); + assert(patch_model()->get_port(pm->path().name())); //cerr << "[PatchController] Adding port " << pm->path() << endl; - /*if (patch_model()->get_port(pm->name())) { + /*if (patch_model()->get_port(pm->path().name())) { cerr << "[PatchController] Ignoring duplicate port " << pm->path() << endl; return; }*/ //node_model()->add_port(pm); - // FIXME: leak - PortController* pc = new PortController(pm); + CountedPtr pc = ControllerFactory::get_controller(pm); // Handle bridge ports/nodes (this is uglier than it should be) /*NodeController* nc = (NodeController*)Store::instance().node(pm->path())->controller(); - if (nc != NULL) + if (nc) nc->bridge_port(pc); */ // Create port on this patch's module (if there is one) - if (m_module != NULL) { + if (m_module) { pc->create_port(m_module); m_module->resize(); } // Create port's (pseudo) module on this patch's canvas (if there is one) - if (m_patch_view != NULL) { + if (m_patch_view) { // Set zoom to 1.0 so module isn't messed up (Death to GnomeCanvas) float old_zoom = m_patch_view->canvas()->get_zoom(); @@ -505,9 +451,9 @@ PatchController::add_port(CountedPtr pm) pc->module()->zoom(old_zoom); } - if (m_control_window != NULL) { - assert(m_control_window->control_panel() != NULL); - m_control_window->control_panel()->add_port(pc); + if (m_control_window) { + assert(m_control_window->control_panel()); + m_control_window->control_panel()->add_port(pm); m_control_window->resize(); } @@ -528,9 +474,9 @@ PatchController::remove_port(const Path& path, bool resize_module) //cerr << "[PatchController] Removing port " << path << endl; /* FIXME - if (m_control_panel != NULL) { + if (m_control_panel) { m_control_panel->remove_port(path); - if (m_control_window != NULL) { + if (m_control_window) { assert(m_control_window->control_panel() == m_control_panel); m_control_window->resize(); } @@ -546,7 +492,7 @@ PatchController::remove_port(const Path& path, bool resize_module) if (!has_control_inputs()) disable_controls_menuitem(); } - +#endif void PatchController::disconnection(const Path& src_port_path, const Path& dst_port_path) @@ -571,32 +517,13 @@ PatchController::disconnection(const Path& src_port_path, const Path& dst_port_p /* // Enable control slider in destination node control window PortController* p = (PortController)Store::instance().port(dst_port_path)->controller(); - assert(p != NULL); + assert(p); - if (p->control_panel() != NULL) + if (p->control_panel()) p->control_panel()->enable_port(p->path()); */ } -void -PatchController::show_patch_window() -{ - if (m_window == NULL) { - Glib::RefPtr xml = GladeFactory::new_glade_reference(); - - xml->get_widget_derived("patch_win", m_window); - assert(m_window != NULL); - - if (m_patch_view == NULL) - create_view(); - - m_window->patch_controller(this); - } - - assert(m_window != NULL); - m_window->present(); -} - /** Become the parent of the patch view. * @@ -605,7 +532,7 @@ PatchController::show_patch_window() void PatchController::claim_patch_view() { - assert(m_patch_view != NULL); + assert(m_patch_view); m_patch_view->hide(); m_patch_view->reparent(m_patch_view_bin); @@ -625,24 +552,4 @@ PatchController::show_control_window() } -void -PatchController::enable_controls_menuitem() -{ - if (m_window != NULL) - m_window->menu_view_control_window()->property_sensitive() = true; - - NodeController::enable_controls_menuitem(); -} - - -void -PatchController::disable_controls_menuitem() -{ - if (m_window != NULL) - m_window->menu_view_control_window()->property_sensitive() = false; - - NodeController::disable_controls_menuitem(); -} - - } // namespace Ingenuity diff --git a/src/progs/ingenuity/PatchController.h b/src/progs/ingenuity/PatchController.h index a5370826..2ee01738 100644 --- a/src/progs/ingenuity/PatchController.h +++ b/src/progs/ingenuity/PatchController.h @@ -19,9 +19,9 @@ #include #include +#include "util/CountedPtr.h" #include "NodeController.h" #include "PatchModel.h" -template class CountedPtr; namespace Ingen { namespace Client { class PatchModel; @@ -60,7 +60,6 @@ class NodeController; class PatchController : public NodeController { public: - PatchController(CountedPtr model); virtual ~PatchController(); /* @@ -68,16 +67,14 @@ public: virtual void remove_from_store(); */ - virtual void destroy(); - virtual void metadata_update(const string& key, const string& value); - virtual void add_port(CountedPtr pm); - virtual void remove_port(const Path& path, bool resize_module); + //virtual void add_port(CountedPtr pm); + //virtual void remove_port(const Path& path, bool resize_module); void connection(CountedPtr cm); void disconnection(const Path& src_port_path, const Path& dst_port_path); - void clear(); + //void clear(); void show_control_window(); void show_properties_window(); @@ -86,35 +83,31 @@ public: void claim_patch_view(); void create_module(OmFlowCanvas* canvas); - void create_view(); - PatchView* view() const { return m_patch_view; } + CountedPtr get_view(); PatchWindow* window() const { return m_window; } void window(PatchWindow* pw) { m_window = pw; } - inline string name() const { return m_model->name(); } - inline const Path& path() const { return m_model->path(); } - void set_path(const Path& new_path); //void enable(); //void disable(); - CountedPtr patch_model() const { return m_patch_model; } + const CountedPtr patch_model() const { return m_patch_model; } - void enable_controls_menuitem(); - void disable_controls_menuitem(); - private: - void add_node(CountedPtr object); - void remove_node(const string& name); + friend class ControllerFactory; + PatchController(CountedPtr model); - NodeController* create_controller_for_node(CountedPtr node); + void destroy(); + void add_node(CountedPtr object); + PatchPropertiesWindow* m_properties_window; - PatchWindow* m_window; ///< Patch Window currently showing m_patch_view - PatchView* m_patch_view; ///< View (canvas) of this patch + // FIXME: remove these + PatchWindow* m_window; ///< Patch Window currently showing m_patch_view + CountedPtr m_patch_view; ///< View (canvas) of this patch CountedPtr m_patch_model; diff --git a/src/progs/ingenuity/PatchPropertiesWindow.cpp b/src/progs/ingenuity/PatchPropertiesWindow.cpp index ec440c15..2271cc21 100644 --- a/src/progs/ingenuity/PatchPropertiesWindow.cpp +++ b/src/progs/ingenuity/PatchPropertiesWindow.cpp @@ -24,7 +24,6 @@ using std::string; PatchPropertiesWindow::PatchPropertiesWindow(BaseObjectType* cobject, const Glib::RefPtr& glade_xml) : Gtk::Window(cobject) -, m_patch_model(NULL) { glade_xml->get_widget("properties_author_entry", m_author_entry); glade_xml->get_widget("properties_description_textview", m_textview); diff --git a/src/progs/ingenuity/PatchTreeWindow.cpp b/src/progs/ingenuity/PatchTreeWindow.cpp index 0f8c947a..090aba18 100644 --- a/src/progs/ingenuity/PatchTreeWindow.cpp +++ b/src/progs/ingenuity/PatchTreeWindow.cpp @@ -23,6 +23,7 @@ #include "Store.h" #include "SubpatchModule.h" #include "PatchModel.h" +#include "WindowFactory.h" #include "util/Path.h" namespace Ingenuity { @@ -74,14 +75,14 @@ PatchTreeWindow::init(Store& store) void PatchTreeWindow::new_object(CountedPtr object) { - CountedPtr patch = object; - if (patch && dynamic_cast(patch->controller())) - add_patch(dynamic_cast(patch->controller())); + CountedPtr patch = PtrCast(object); + if (patch) + add_patch(PtrCast(patch->controller())); } void -PatchTreeWindow::add_patch(PatchController* pc) +PatchTreeWindow::add_patch(CountedPtr pc) { const CountedPtr pm = pc->patch_model(); @@ -89,7 +90,7 @@ PatchTreeWindow::add_patch(PatchController* pc) Gtk::TreeModel::iterator iter = m_patch_treestore->append(); Gtk::TreeModel::Row row = *iter; if (pm->path() == "/") { - CountedPtr osc_sender = App::instance().engine(); + CountedPtr osc_sender = PtrCast(App::instance().engine()); string root_name = osc_sender ? osc_sender->engine_url() : "Internal"; // Hack off trailing '/' if it's there (ugly) //if (root_name.substr(root_name.length()-1,1) == "/") @@ -130,10 +131,8 @@ PatchTreeWindow::remove_patch(const Path& path) Gtk::TreeModel::iterator PatchTreeWindow::find_patch(Gtk::TreeModel::Children root, const Path& path) { - PatchController* pc = NULL; - for (Gtk::TreeModel::iterator c = root.begin(); c != root.end(); ++c) { - pc = (*c)[m_patch_tree_columns.patch_controller_col]; + CountedPtr pc = (*c)[m_patch_tree_columns.patch_controller_col]; if (pc->model()->path() == path) { return c; } else if ((*c)->children().size() > 0) { @@ -152,7 +151,7 @@ PatchTreeWindow::event_patch_selected() Gtk::TreeModel::iterator active = m_patch_tree_selection->get_selected(); if (active) { Gtk::TreeModel::Row row = *active; - PatchController* pc = row[m_patch_tree_columns.patch_controller_col]; + CountedPtr pc = row[m_patch_tree_columns.patch_controller_col]; } } */ @@ -166,8 +165,8 @@ PatchTreeWindow::show_patch_menu(GdkEventButton* ev) Gtk::TreeModel::iterator active = m_patch_tree_selection->get_selected(); if (active) { Gtk::TreeModel::Row row = *active; - PatchController* pc = row[m_patch_tree_columns.patch_controller_col]; - if (pc != NULL) + CountedPtr pc = row[m_patch_tree_columns.patch_controller_col]; + if (pc) pc->show_menu(ev); } } @@ -178,9 +177,9 @@ PatchTreeWindow::event_patch_activated(const Gtk::TreeModel::Path& path, Gtk::Tr { Gtk::TreeModel::iterator active = m_patch_treestore->get_iter(path); Gtk::TreeModel::Row row = *active; - PatchController* pc = row[m_patch_tree_columns.patch_controller_col]; + CountedPtr pc = row[m_patch_tree_columns.patch_controller_col]; - pc->show_patch_window(); + App::instance().window_factory()->present(pc); } @@ -191,10 +190,10 @@ PatchTreeWindow::event_patch_enabled_toggled(const Glib::ustring& path_str) Gtk::TreeModel::iterator active = m_patch_treestore->get_iter(path); Gtk::TreeModel::Row row = *active; - PatchController* pc = row[m_patch_tree_columns.patch_controller_col]; + CountedPtr pc = row[m_patch_tree_columns.patch_controller_col]; Glib::ustring patch_path = pc->model()->path(); - assert(pc != NULL); + assert(pc); if ( ! pc->patch_model()->enabled()) { if (m_enable_signal) diff --git a/src/progs/ingenuity/PatchTreeWindow.h b/src/progs/ingenuity/PatchTreeWindow.h index a43703a7..d23c4e54 100644 --- a/src/progs/ingenuity/PatchTreeWindow.h +++ b/src/progs/ingenuity/PatchTreeWindow.h @@ -50,7 +50,7 @@ public: void patch_disabled(const Path& path); void patch_renamed(const Path& old_path, const Path& new_path); - void add_patch(PatchController* pc); + void add_patch(CountedPtr pc); void remove_patch(const Path& path); void show_patch_menu(GdkEventButton* ev); @@ -68,9 +68,9 @@ protected: PatchTreeModelColumns() { add(name_col); add(enabled_col); add(patch_controller_col); } - Gtk::TreeModelColumn name_col; - Gtk::TreeModelColumn enabled_col; - Gtk::TreeModelColumn patch_controller_col; + Gtk::TreeModelColumn name_col; + Gtk::TreeModelColumn enabled_col; + Gtk::TreeModelColumn > patch_controller_col; }; bool m_enable_signal; diff --git a/src/progs/ingenuity/PatchView.cpp b/src/progs/ingenuity/PatchView.cpp index bce44369..90c0083e 100644 --- a/src/progs/ingenuity/PatchView.cpp +++ b/src/progs/ingenuity/PatchView.cpp @@ -52,36 +52,34 @@ PatchView::PatchView(BaseObjectType* cobject, const Glib::RefPtrget_widget("patch_view_zoom_full_but", _zoom_full_but); xml->get_widget("patch_view_zoom_normal_but", _zoom_normal_but); xml->get_widget("patch_view_scrolledwindow", _canvas_scrolledwindow); - - _process_but->signal_toggled().connect(sigc::mem_fun(this, &PatchView::process_toggled)); } -/** Sets the patch controller for this window and initializes everything. - * - * This function MUST be called before using the window in any way! - */ void PatchView::patch_controller(PatchController* pc) { - //m_patch = new PatchController(pm, controller); _patch = pc; - _canvas = new OmFlowCanvas(pc, 1600*2, 1200*2); _canvas_scrolledwindow->add(*_canvas); - //_canvas->show(); - //_canvas_scrolledwindow->show(); _poly_spin->set_value(pc->patch_model()->poly()); - + _destroy_but->set_sensitive(pc->path() != "/"); //_description_window->patch_model(pc->model()); + pc->patch_model()->enabled_sig.connect(sigc::mem_fun(this, &PatchView::enable)); pc->patch_model()->disabled_sig.connect(sigc::mem_fun(this, &PatchView::disable)); + + _process_but->signal_toggled().connect(sigc::mem_fun(this, &PatchView::process_toggled)); + + _clear_but->signal_clicked().connect(sigc::mem_fun(this, &PatchView::clear_clicked)); + + _zoom_normal_but->signal_clicked().connect(sigc::bind(sigc::mem_fun( + static_cast(_canvas), &FlowCanvas::set_zoom), 1.0)); - _zoom_normal_but->signal_clicked().connect(sigc::bind(sigc::mem_fun(static_cast(_canvas), &FlowCanvas::set_zoom), 1.0)); - _zoom_full_but->signal_clicked().connect(sigc::mem_fun(static_cast(_canvas), &FlowCanvas::zoom_full)); + _zoom_full_but->signal_clicked().connect( + sigc::mem_fun(static_cast(_canvas), &FlowCanvas::zoom_full)); } @@ -108,6 +106,11 @@ PatchView::process_toggled() } } +void +PatchView::clear_clicked() +{ + App::instance().engine()->clear_patch(_patch->path()); +} void PatchView::enable() diff --git a/src/progs/ingenuity/PatchView.h b/src/progs/ingenuity/PatchView.h index 3c507b94..59e16aad 100644 --- a/src/progs/ingenuity/PatchView.h +++ b/src/progs/ingenuity/PatchView.h @@ -63,7 +63,7 @@ public: PatchController* patch_controller() const { return _patch; } Gtk::Viewport* breadcrumb_container() const { return _breadcrumb_container; } void show_control_window(); - void process_toggled(); + void enable(); void disable(); @@ -71,6 +71,9 @@ public: void zoom_full(); private: + void process_toggled(); + void clear_clicked(); + PatchController* _patch; OmFlowCanvas* _canvas; diff --git a/src/progs/ingenuity/PatchWindow.cpp b/src/progs/ingenuity/PatchWindow.cpp index 2dfcded0..c395c265 100644 --- a/src/progs/ingenuity/PatchWindow.cpp +++ b/src/progs/ingenuity/PatchWindow.cpp @@ -20,7 +20,6 @@ #include #include "App.h" #include "ModelEngineInterface.h" -#include "PatchView.h" #include "OmFlowCanvas.h" #include "PatchController.h" #include "LoadPluginWindow.h" @@ -37,13 +36,15 @@ #include "Store.h" #include "ConnectWindow.h" #include "Loader.h" +#include "ControllerFactory.h" +#include "WindowFactory.h" +#include "PatchView.h" namespace Ingenuity { PatchWindow::PatchWindow(BaseObjectType* cobject, const Glib::RefPtr& xml) : Gtk::Window(cobject), - m_patch(NULL), m_load_plugin_window(NULL), m_new_subpatch_window(NULL), m_enable_signal(true), @@ -75,6 +76,7 @@ PatchWindow::PatchWindow(BaseObjectType* cobject, const Glib::RefPtrget_widget("patch_view_patch_tree_window_menuitem", m_menu_view_patch_tree_window); xml->get_widget("patch_help_about_menuitem", m_menu_help_about); + // FIXME: these shouldn't be loaded here xml->get_widget_derived("load_plugin_win", m_load_plugin_window); xml->get_widget_derived("new_subpatch_win", m_new_subpatch_window); xml->get_widget_derived("load_patch_win", m_load_patch_window); @@ -96,8 +98,6 @@ PatchWindow::PatchWindow(BaseObjectType* cobject, const Glib::RefPtrsignal_activate().connect( sigc::mem_fun(this, &PatchWindow::event_save_as)); - m_menu_close->signal_activate().connect( - sigc::mem_fun(this, &PatchWindow::event_close)); m_menu_quit->signal_activate().connect( sigc::mem_fun(this, &PatchWindow::event_quit)); m_menu_configuration->signal_activate().connect( @@ -125,7 +125,7 @@ PatchWindow::PatchWindow(BaseObjectType* cobject, const Glib::RefPtr(App::instance().about_dialog(), &Gtk::Dialog::present)); m_breadcrumb_box = new BreadCrumbBox(); - m_breadcrumb_box->signal_patch_selected.connect(sigc::mem_fun(this, &PatchWindow::patch)); + m_breadcrumb_box->signal_patch_selected.connect(sigc::mem_fun(this, &PatchWindow::set_patch_from_path)); App::instance().add_patch_window(this); } @@ -133,38 +133,32 @@ PatchWindow::PatchWindow(BaseObjectType* cobject, const Glib::RefPtrclaim_patch_view(); + App::instance().remove_patch_window(this); hide(); delete m_new_subpatch_window; delete m_load_subpatch_window; + delete m_breadcrumb_box; } -/** Set the patch controller from a Path (for BreadCrumbs) +/** Set the patch controller from a Path (for use by eg. BreadCrumbBox) */ void -PatchWindow::patch(const Path& path) +PatchWindow::set_patch_from_path(const Path& path) { - CountedPtr model = App::instance().store()->object(path); + CountedPtr model = PtrCast(App::instance().store()->object(path)); if (!model) return; // can't really do anything useful.. - PatchController* pc = dynamic_cast(model->controller()); - - if (!pc) { - pc = new PatchController(model); - model->set_controller(pc); - } - + CountedPtr pc = PtrCast(ControllerFactory::get_controller(model)); assert(pc); - if (pc->window() != NULL && pc->window()->is_visible()) { - pc->show_patch_window(); - } else { - patch_controller(pc); - } + App::instance().window_factory()->present(pc, this); } @@ -173,20 +167,15 @@ PatchWindow::patch(const Path& path) * This function MUST be called before using the window in any way! */ void -PatchWindow::patch_controller(PatchController* pc) +PatchWindow::set_patch(CountedPtr pc) { if (!pc || pc == m_patch) return; m_enable_signal = false; - assert(pc); - assert(m_patch != pc); - assert(m_patch == NULL || - pc->model()->path() != m_patch->model()->path()); - - PatchController* old_pc = m_patch; - if (old_pc != NULL) { + if (m_patch) { + CountedPtr old_pc = m_patch; assert(old_pc->window() == NULL || old_pc->window() == this); old_pc->claim_patch_view(); old_pc->window(NULL); @@ -194,12 +183,8 @@ PatchWindow::patch_controller(PatchController* pc) m_patch = pc; - if (pc->view() == NULL) - pc->create_view(); - assert(pc->view()); - - PatchView* const patch_view = pc->view(); - assert(patch_view != NULL); + CountedPtr patch_view = pc->get_view(); + assert(patch_view); patch_view->reparent(*m_viewport); if (m_breadcrumb_box->get_parent()) @@ -217,10 +202,10 @@ PatchWindow::patch_controller(PatchController* pc) assert(m_load_patch_window != NULL); assert(m_load_subpatch_window != NULL); - m_load_patch_window->patch_controller(m_patch); - m_load_plugin_window->patch_controller(m_patch); - m_new_subpatch_window->patch_controller(m_patch); - m_load_subpatch_window->patch_controller(m_patch); + m_load_patch_window->set_patch(m_patch); + m_load_plugin_window->set_patch(m_patch); + m_new_subpatch_window->set_patch(m_patch); + m_load_subpatch_window->set_patch(m_patch); m_menu_view_control_window->property_sensitive() = pc->has_control_inputs(); @@ -239,7 +224,6 @@ PatchWindow::patch_controller(PatchController* pc) else m_menu_destroy_patch->set_sensitive(true); - assert(old_pc == NULL || old_pc->window() != this); assert(m_patch == pc); assert(m_patch->window() == this); @@ -247,7 +231,6 @@ PatchWindow::patch_controller(PatchController* pc) } - void PatchWindow::event_show_engine() { @@ -272,58 +255,6 @@ PatchWindow::event_show_properties() } -/** Notification a node has been removed from the PatchView this window - * currently contains. - * - * This is used to update the breadcrumbs in case the Node is a patch which has - * a button present in the breadcrumbs that needs to be removed. - */ -void -PatchWindow::node_removed(const string& name) -{ - throw; // FIXME -/* - for (list::iterator i = m_breadcrumbs.begin(); i != m_breadcrumbs.end(); ++i) { - if ((*i)->path() == m_patch->model()->base_path() + name) { - for (list::iterator j = i; j != m_breadcrumbs.end(); ) { - BreadCrumb* bc = *j; - j = m_breadcrumbs.erase(j); - m_breadcrumb_box->remove(*bc); - } - break; - } - }*/ -} - - -/** Same as @a node_removed, but for renaming. - */ -void -PatchWindow::node_renamed(const string& old_path, const string& new_path) -{ - throw; // FIXME - /* - for (list::iterator i = m_breadcrumbs.begin(); i != m_breadcrumbs.end(); ++i) { - if ((*i)->path() == old_path) - (*i)->set_path(new_path); - }*/ -} - - -/** Notification the patch this window is currently showing was renamed. - */ -void -PatchWindow::patch_renamed(const string& new_path) -{ - throw; // FIXME - /* - set_title(new_path); - for (list::iterator i = m_breadcrumbs.begin(); i != m_breadcrumbs.end(); ++i) { - if ((*i)->path() == m_patch->path()) - (*i)->set_path(new_path); - }*/ -} - /* void PatchWindow::event_open() @@ -344,7 +275,7 @@ PatchWindow::event_import() void PatchWindow::event_save() { - PatchModel* const model = m_patch->patch_model().get(); + CountedPtr model(m_patch->patch_model().get()); if (model->filename() == "") event_save_as(); @@ -404,7 +335,7 @@ PatchWindow::event_save_as() fin.close(); if (confirm) { - App::instance().loader()->save_patch(m_patch->patch_model().get(), filename, recursive); + App::instance().loader()->save_patch(m_patch->patch_model(), filename, recursive); m_patch->patch_model()->filename(filename); } } @@ -432,21 +363,13 @@ PatchWindow::on_hide() } -bool -PatchWindow::on_delete_event(GdkEventAny* ev) -{ - event_close(); - return true; // destroy window -} - - bool PatchWindow::on_key_press_event(GdkEventKey* event) { if (event->keyval == GDK_Delete) { - if (m_patch != NULL && m_patch->view() != NULL) { - assert(m_patch->view()->canvas() != NULL); - m_patch->view()->canvas()->destroy_selected(); + if (m_patch && m_patch->get_view()) { + assert(m_patch->get_view()->canvas()); + m_patch->get_view()->canvas()->destroy_selected(); } return true; } else { @@ -455,27 +378,6 @@ PatchWindow::on_key_press_event(GdkEventKey* event) } -void -PatchWindow::event_close() -{ - if (App::instance().num_open_patch_windows() > 1) { - hide(); - } else { - Gtk::MessageDialog d(*this, "This is the last remaining open patch " - "window. Closing this window will exit Ingenuity (the engine will " - "remain running).\n\nAre you sure you want to quit?", - true, Gtk::MESSAGE_WARNING, Gtk::BUTTONS_NONE, true); - d.add_button(Gtk::Stock::CANCEL, Gtk::RESPONSE_CANCEL); - d.add_button(Gtk::Stock::QUIT, Gtk::RESPONSE_CLOSE); - int ret = d.run(); - if (ret == Gtk::RESPONSE_CLOSE) - App::instance().quit(); - else - d.hide(); - } -} - - void PatchWindow::event_quit() { diff --git a/src/progs/ingenuity/PatchWindow.h b/src/progs/ingenuity/PatchWindow.h index 07a3e92e..d538db78 100644 --- a/src/progs/ingenuity/PatchWindow.h +++ b/src/progs/ingenuity/PatchWindow.h @@ -23,9 +23,11 @@ #include #include #include "util/Path.h" - +#include "util/CountedPtr.h" +#include "PatchController.h" using std::string; using std::list; + namespace Ingen { namespace Client { class PatchModel; class NodeModel; @@ -40,7 +42,6 @@ namespace Ingenuity { class PatchController; class OmFlowCanvas; -class PatchView; class LoadPluginWindow; class LoadPatchWindow; class NewSubpatchWindow; @@ -63,18 +64,13 @@ public: PatchWindow(BaseObjectType* cobject, const Glib::RefPtr& glade_xml); ~PatchWindow(); - void patch(const Path& path); - void patch_controller(PatchController* pc); + void set_patch_from_path(const Path& path); + void set_patch(CountedPtr pc); - PatchController* patch_controller() const { return m_patch; } - LoadPluginWindow* load_plugin_window() const { return m_load_plugin_window; } - LoadSubpatchWindow* load_subpatch_window() const { return m_load_subpatch_window; } - NewSubpatchWindow* new_subpatch_window() const { return m_new_subpatch_window; } - - // Breadcrumb management - void node_removed(const string& name); - void node_renamed(const string& old_path, const string& new_path); - void patch_renamed(const string& new_path); + CountedPtr patch_controller() const { return m_patch; } + LoadPluginWindow* load_plugin_window() const { return m_load_plugin_window; } + LoadSubpatchWindow* load_subpatch_window() const { return m_load_subpatch_window; } + NewSubpatchWindow* new_subpatch_window() const { return m_new_subpatch_window; } Gtk::MenuItem* menu_view_control_window() { return m_menu_view_control_window; } @@ -83,14 +79,12 @@ public: protected: void on_show(); void on_hide(); - bool on_delete_event(GdkEventAny* ev); bool on_key_press_event(GdkEventKey* event); private: void event_import(); void event_save(); void event_save_as(); - void event_close(); void event_quit(); void event_destroy(); void event_clear(); @@ -99,7 +93,8 @@ private: void event_show_controls(); void event_show_engine(); - PatchController* m_patch; + CountedPtr m_patch; + LoadPluginWindow* m_load_plugin_window; LoadPatchWindow* m_load_patch_window; NewSubpatchWindow* m_new_subpatch_window; diff --git a/src/progs/ingenuity/PortController.cpp b/src/progs/ingenuity/PortController.cpp index 6a23169f..167cb594 100644 --- a/src/progs/ingenuity/PortController.cpp +++ b/src/progs/ingenuity/PortController.cpp @@ -34,9 +34,6 @@ PortController::PortController(CountedPtr model) { assert(model); assert(model->parent()); - assert(model->controller() == NULL); - - model->set_controller(this); } @@ -44,8 +41,8 @@ void PortController::destroy() { assert(m_model->parent()); - NodeController* parent = (NodeController*)m_model->parent()->controller(); - assert(parent != NULL); + CountedPtr parent = PtrCast(m_model->parent()->controller()); + assert(parent); parent->remove_port(path(), false); } @@ -73,7 +70,7 @@ PortController::create_module(OmFlowCanvas* canvas) m_module = new OmPortModule(canvas, this, x, y); - if (CountedPtr(port_model()->parent())) { + if (PtrCast(port_model()->parent())) { if (m_patch_port) delete m_patch_port; diff --git a/src/progs/ingenuity/PortController.h b/src/progs/ingenuity/PortController.h index 1a58491b..e28cd21d 100644 --- a/src/progs/ingenuity/PortController.h +++ b/src/progs/ingenuity/PortController.h @@ -47,7 +47,6 @@ class OmFlowCanvas; class PortController : public GtkObjectController { public: - PortController(CountedPtr model); virtual ~PortController() {} virtual void destroy(); @@ -63,9 +62,12 @@ public: void set_path(const Path& new_path); - CountedPtr port_model() const { return m_model; } + CountedPtr port_model() const { return PtrCast(m_model); } private: + friend class ControllerFactory; + PortController(CountedPtr model); + OmPatchPort* m_patch_port; ///< Port on m_module OmPortModule* m_module; ///< Port pseudo-module (for patch ports only) OmPort* m_port; ///< Port on some other canvas module diff --git a/src/progs/ingenuity/RenameWindow.cpp b/src/progs/ingenuity/RenameWindow.cpp index dfe4b07b..7a458e31 100644 --- a/src/progs/ingenuity/RenameWindow.cpp +++ b/src/progs/ingenuity/RenameWindow.cpp @@ -70,7 +70,7 @@ RenameWindow::name_changed() m_message_label->set_text("Name may not contain '/'"); m_ok_button->property_sensitive() = false; //} else if (m_object->parent()->patch_model()->get_node(name) != NULL) { - } else if (App::instance().store()->object(m_object->model()->parent()->base_path() + name)) { + } else if (App::instance().store()->object(m_object->model()->parent()->path().base() + name)) { m_message_label->set_text("An object already exists with that name."); m_ok_button->property_sensitive() = false; } else if (name.length() == 0) { diff --git a/src/progs/ingenuity/SubpatchModule.cpp b/src/progs/ingenuity/SubpatchModule.cpp index d0492618..598cc9db 100644 --- a/src/progs/ingenuity/SubpatchModule.cpp +++ b/src/progs/ingenuity/SubpatchModule.cpp @@ -26,51 +26,46 @@ #include "OmFlowCanvas.h" #include "PatchController.h" #include "OmPort.h" +#include "WindowFactory.h" using std::cerr; using std::cout; using std::endl; namespace Ingenuity { -SubpatchModule::SubpatchModule(OmFlowCanvas* canvas, PatchController* patch) -: OmModule(canvas, patch), +SubpatchModule::SubpatchModule(OmFlowCanvas* canvas, CountedPtr patch) +: OmModule(canvas, patch.get()), m_patch(patch) { - assert(canvas != NULL); - assert(patch != NULL); + assert(canvas); + assert(patch); } void SubpatchModule::on_double_click(GdkEventButton* event) { - assert(m_patch != NULL); - - // If window is visible - if (m_patch->window() != NULL - && m_patch->window()->is_visible()) { - m_patch->show_patch_window(); // just raise it - // No window visible - } else { - if (event->state & GDK_SHIFT_MASK) - m_patch->show_patch_window(); // open a new window - else - browse_to_patch(); - } + assert(m_patch); + + CountedPtr parent = PtrCast(m_patch->model()->parent()->controller()); + + PatchWindow* const preferred + = (event->state & GDK_SHIFT_MASK) ? NULL : parent->window(); + + App::instance().window_factory()->present(m_patch, preferred); } -/** Browse to this patch in current (parent's) window. */ +/** Browse to this patch in current (parent's) window + * (unless an existing window is displaying it) + */ void SubpatchModule::browse_to_patch() { assert(m_patch->model()->parent()); - PatchController* pc = (PatchController*)m_patch->model()->parent()->controller(); - assert(pc != NULL); - assert(pc->window() != NULL); + CountedPtr pc = PtrCast(m_patch->model()->parent()->controller()); - assert(pc->window() != NULL); - pc->window()->patch_controller(m_patch); + App::instance().window_factory()->present(m_patch, pc->window()); } diff --git a/src/progs/ingenuity/SubpatchModule.h b/src/progs/ingenuity/SubpatchModule.h index c530311e..a8e6e971 100644 --- a/src/progs/ingenuity/SubpatchModule.h +++ b/src/progs/ingenuity/SubpatchModule.h @@ -21,6 +21,7 @@ #include #include #include "OmModule.h" +#include "util/CountedPtr.h" #include "PatchController.h" using std::string; using std::list; @@ -36,7 +37,6 @@ namespace Ingenuity { class OmFlowCanvas; class NodeControlWindow; -class PatchController; /** A module to represent a subpatch @@ -46,7 +46,7 @@ class PatchController; class SubpatchModule : public OmModule { public: - SubpatchModule(OmFlowCanvas* canvas, PatchController* controller); + SubpatchModule(OmFlowCanvas* canvas, CountedPtr controller); virtual ~SubpatchModule() {} void on_double_click(GdkEventButton* ev); @@ -55,10 +55,10 @@ public: void browse_to_patch(); void menu_remove(); - PatchController* patch() { return m_patch; } + CountedPtr patch() { return m_patch; } protected: - PatchController* m_patch; + CountedPtr m_patch; }; diff --git a/src/progs/ingenuity/WindowFactory.cpp b/src/progs/ingenuity/WindowFactory.cpp new file mode 100644 index 00000000..6a913663 --- /dev/null +++ b/src/progs/ingenuity/WindowFactory.cpp @@ -0,0 +1,108 @@ +/* This file is part of Ingen. Copyright (C) 2006 Dave Robillard. + * + * Ingen is free software; you can redistribute it and/or modify it under the + * terms of the GNU General Public License as published by the Free Software + * Foundation; either version 2 of the License, or (at your option) any later + * version. + * + * Ingen is distributed in the hope that it will be useful, but WITHOUT ANY + * WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS + * FOR A PARTICULAR PURPOSE. See the GNU General Public License for details. + * + * You should have received a copy of the GNU General Public License along + * with this program; if not, write to the Free Software Foundation, Inc., + * 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA + */ + +#include "WindowFactory.h" +#include "PatchWindow.h" +#include "GladeFactory.h" +#include "App.h" + +namespace Ingenuity { + + +/** Present a PatchWindow for a Patch. + * + * If @a preferred is not NULL, it will be set to display @a patch if the patch + * does not already have a visible window, otherwise that window will be presented and + * @a preferred left unmodified. + */ +void +WindowFactory::present(CountedPtr patch, PatchWindow* preferred) +{ + std::map::iterator w = _windows.find(patch->model()->path()); + + if (w != _windows.end()) { + (*w).second->present(); + } else if (preferred) { + w = _windows.find(preferred->patch_controller()->model()->path()); + assert((*w).second == preferred); + + preferred->set_patch(patch); + _windows.erase(w); + _windows[patch->model()->path()] = preferred; + preferred->present(); + + } else { + PatchWindow* win = create_new(patch); + win->present(); + } +} + + +PatchWindow* +WindowFactory::create_new(CountedPtr patch) +{ + // FIXME: can't just load "patch_win" and children because PatchWindow + // loads things it really shouldn't. + //Glib::RefPtr xml = GladeFactory::new_glade_reference("patch_win"); + Glib::RefPtr xml = GladeFactory::new_glade_reference(); + + PatchWindow* win = NULL; + xml->get_widget_derived("patch_win", win); + assert(win); + + win->set_patch(patch); + _windows[patch->model()->path()] = win; + + win->signal_delete_event().connect(sigc::bind<0>( + sigc::mem_fun(this, &WindowFactory::remove), win)); + + cerr << "Created window - count: " << _windows.size() << endl; + + return win; +} + + +bool +WindowFactory::remove(PatchWindow* win, GdkEventAny* ignored) +{ + if (_windows.size() <= 1) { + Gtk::MessageDialog d(*win, "This is the last remaining open patch " + "window. Closing this window will exit Ingenuity (the engine will " + "remain running).\n\nAre you sure you want to quit?", + true, Gtk::MESSAGE_WARNING, Gtk::BUTTONS_NONE, true); + d.add_button(Gtk::Stock::CANCEL, Gtk::RESPONSE_CANCEL); + d.add_button(Gtk::Stock::QUIT, Gtk::RESPONSE_CLOSE); + int ret = d.run(); + if (ret == Gtk::RESPONSE_CLOSE) + App::instance().quit(); + else + return false; + } + + std::map::iterator w + = _windows.find(win->patch_controller()->model()->path()); + + assert((*w).second == win); + _windows.erase(w); + + delete win; + + cerr << "Removed window " << win << "- count: " << _windows.size() << endl; + + return true; +} + +} // namespace Ingenuity diff --git a/src/progs/ingenuity/WindowFactory.h b/src/progs/ingenuity/WindowFactory.h new file mode 100644 index 00000000..add2d97b --- /dev/null +++ b/src/progs/ingenuity/WindowFactory.h @@ -0,0 +1,42 @@ +/* This file is part of Ingen. Copyright (C) 2006 Dave Robillard. + * + * Ingen is free software; you can redistribute it and/or modify it under the + * terms of the GNU General Public License as published by the Free Software + * Foundation; either version 2 of the License, or (at your option) any later + * version. + * + * Ingen is distributed in the hope that it will be useful, but WITHOUT ANY + * WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS + * FOR A PARTICULAR PURPOSE. See the GNU General Public License for details. + * + * You should have received a copy of the GNU General Public License along + * with this program; if not, write to the Free Software Foundation, Inc., + * 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA + */ + +#ifndef WINDOW_FACTORY_H +#define WINDOW_FACTORY_H + +#include +#include "util/CountedPtr.h" +#include "PatchController.h" + +namespace Ingenuity { + +class PatchWindow; + + +class WindowFactory { +public: + void present(CountedPtr patch, PatchWindow* preferred = NULL); + +private: + PatchWindow* create_new(CountedPtr patch); + bool remove(PatchWindow* win, GdkEventAny* ignored); + + std::map _windows; +}; + +} + +#endif // WINDOW_FACTORY_H diff --git a/src/progs/ingenuity/ingenuity.glade b/src/progs/ingenuity/ingenuity.glade index 84540c7c..2221a805 100644 --- a/src/progs/ingenuity/ingenuity.glade +++ b/src/progs/ingenuity/ingenuity.glade @@ -1770,7 +1770,7 @@ 0 - + True GTK_ORIENTATION_HORIZONTAL GTK_TOOLBAR_BOTH @@ -1822,7 +1822,7 @@ - + True GTK_ORIENTATION_HORIZONTAL GTK_TOOLBAR_ICONS @@ -1942,7 +1942,8 @@ True - Destroy all children nodes (twice to clear ports as well) + Destroy all children nodes +(Click twice to clear ports as well) True gtk-clear diff --git a/src/progs/patch_loader/patch_loader.cpp b/src/progs/patch_loader/patch_loader.cpp index 105a379c..f7464840 100644 --- a/src/progs/patch_loader/patch_loader.cpp +++ b/src/progs/patch_loader/patch_loader.cpp @@ -52,29 +52,28 @@ int main(int argc, char** argv) /* **** Mr. Spock.. Engage **** */ - OSCModelEngineInterface engine(engine_url); - PatchLibrarian librarian(&engine); + CountedPtr engine(new OSCModelEngineInterface(engine_url)); + PatchLibrarian librarian(engine); /* Connect to engine */ - engine.attach(-1, client_port); - engine.activate(); - //engine.register_client(NULL); // FIXME + engine->attach(-1, client_port); + engine->activate(); + //engine->register_client(NULL); // FIXME - //int id = engine.get_next_request_id(); - //engine.set_wait_response_id(id); - //engine.load_plugins(id); - //engine.wait_for_response(); + //int id = engine->get_next_request_id(); + //engine->set_wait_response_id(id); + //engine->load_plugins(id); + //engine->wait_for_response(); /* FIXME: Make this work like this: - * engine.load_plugins(); - * engine.wait_for_response(); + * engine->load_plugins(); + * engine->wait_for_response(); */ // Load patches for (uint i=0; i < args_info.inputs_num; ++i) { - PatchModel* pm = new PatchModel("", 0); + CountedPtr pm(new PatchModel("", 0)); pm->filename(args_info.inputs[i]); librarian.load_patch(pm, true); - delete pm; } return 0; diff --git a/src/progs/server/Makefile.am b/src/progs/server/Makefile.am index fc5b770b..e2a38742 100644 --- a/src/progs/server/Makefile.am +++ b/src/progs/server/Makefile.am @@ -1,4 +1,4 @@ -AM_CXXFLAGS = @JACK_CFLAGS@ @LOSC_CFLAGS@ @ALSA_CFLAGS@ @LASH_CFLAGS@ @SLV2_CFLAGS@ -I$(top_srcdir)/src/common -I$(top_srcdir)/src/libs/engine -I$(top_srcdir)/src/libs/engine/events -fno-exceptions +AM_CXXFLAGS = @JACK_CFLAGS@ @LOSC_CFLAGS@ @ALSA_CFLAGS@ @LASH_CFLAGS@ @SLV2_CFLAGS@ -I$(top_srcdir)/src/common -I$(top_srcdir)/src/libs/engine -I$(top_srcdir)/src/libs/engine/events MAINTAINERCLEANFILES = Makefile.in diff --git a/src/progs/server/main.cpp b/src/progs/server/main.cpp index cd79a7b4..c33014a5 100644 --- a/src/progs/server/main.cpp +++ b/src/progs/server/main.cpp @@ -137,7 +137,7 @@ main(int argc, char** argv) set_denormal_flags(); - engine = new Engine(); + engine = CountedPtr(new Engine()); OSCEngineReceiver* receiver = new OSCEngineReceiver( engine, pre_processor_queue_size, args_info.port_arg); -- cgit v1.2.1