diff options
-rw-r--r-- | ingen/Configuration.hpp | 2 | ||||
-rw-r--r-- | src/gui/GraphCanvas.cpp | 19 | ||||
-rw-r--r-- | src/gui/GraphPortModule.cpp | 5 | ||||
-rw-r--r-- | src/gui/GraphPortModule.hpp | 3 | ||||
-rw-r--r-- | src/gui/NodeModule.cpp | 3 | ||||
-rw-r--r-- | src/gui/Port.cpp | 52 | ||||
-rw-r--r-- | src/gui/Port.hpp | 4 |
7 files changed, 61 insertions, 27 deletions
diff --git a/ingen/Configuration.hpp b/ingen/Configuration.hpp index 2f272498..50543986 100644 --- a/ingen/Configuration.hpp +++ b/ingen/Configuration.hpp @@ -112,7 +112,7 @@ public: const std::string& filename); const Atom& option(const std::string& long_name) const; - bool set(const std::string& long_name, const Atom& value); + bool set(const std::string& long_name, const Atom& value); const std::list<std::string>& files() const { return _files; } diff --git a/src/gui/GraphCanvas.cpp b/src/gui/GraphCanvas.cpp index b06b47b1..1fe665e7 100644 --- a/src/gui/GraphCanvas.cpp +++ b/src/gui/GraphCanvas.cpp @@ -256,13 +256,30 @@ void GraphCanvas::show_human_names(bool b) { _human_names = b; + _app.world()->conf().set("human-names", _app.forge().make(b)); + for_each_node(show_module_human_names, &b); } +static void +ensure_port_labels(GanvNode* node, void* data) +{ + if (GANV_IS_MODULE(node)) { + Ganv::Module* module = Glib::wrap(GANV_MODULE(node)); + for (Ganv::Port* p : *module) { + Ingen::GUI::Port* port = dynamic_cast<Ingen::GUI::Port*>(p); + if (port) { + port->ensure_label(); + } + } + } +} + void GraphCanvas::show_port_names(bool b) { ganv_canvas_set_direction(gobj(), b ? GANV_DIRECTION_RIGHT : GANV_DIRECTION_DOWN); + for_each_node(ensure_port_labels, &b); } void @@ -314,7 +331,7 @@ GraphCanvas::remove_block(SPtr<const BlockModel> bm) void GraphCanvas::add_port(SPtr<const PortModel> pm) { - GraphPortModule* view = GraphPortModule::create(*this, pm, _human_names); + GraphPortModule* view = GraphPortModule::create(*this, pm); _views.insert(std::make_pair(pm, view)); view->show(); } diff --git a/src/gui/GraphPortModule.cpp b/src/gui/GraphPortModule.cpp index dc781a35..99704eaa 100644 --- a/src/gui/GraphPortModule.cpp +++ b/src/gui/GraphPortModule.cpp @@ -66,11 +66,10 @@ GraphPortModule::GraphPortModule(GraphCanvas& canvas, GraphPortModule* GraphPortModule::create(GraphCanvas& canvas, - SPtr<const PortModel> model, - bool human) + SPtr<const PortModel> model) { GraphPortModule* ret = new GraphPortModule(canvas, model); - Port* port = Port::create(canvas.app(), *ret, model, human, true); + Port* port = Port::create(canvas.app(), *ret, model, true); ret->set_port(port); if (model->is_numeric()) { diff --git a/src/gui/GraphPortModule.hpp b/src/gui/GraphPortModule.hpp index 5cffa7bc..8dcee1c9 100644 --- a/src/gui/GraphPortModule.hpp +++ b/src/gui/GraphPortModule.hpp @@ -48,8 +48,7 @@ class GraphPortModule : public Ganv::Module public: static GraphPortModule* create( GraphCanvas& canvas, - SPtr<const Client::PortModel> model, - bool human); + SPtr<const Client::PortModel> model); App& app() const; diff --git a/src/gui/NodeModule.cpp b/src/gui/NodeModule.cpp index 40303ad7..c03618cf 100644 --- a/src/gui/NodeModule.cpp +++ b/src/gui/NodeModule.cpp @@ -268,8 +268,7 @@ NodeModule::rename() void NodeModule::new_port_view(SPtr<const PortModel> port) { - Port::create(app(), *this, port, - app().world()->conf().option("human-names").get<int32_t>()); + Port::create(app(), *this, port); port->signal_value_changed().connect( sigc::bind<0>(sigc::mem_fun(this, &NodeModule::port_value_changed), diff --git a/src/gui/Port.cpp b/src/gui/Port.cpp index f15d067c..07e654db 100644 --- a/src/gui/Port.cpp +++ b/src/gui/Port.cpp @@ -44,25 +44,9 @@ Port* Port::create(App& app, Ganv::Module& module, SPtr<const PortModel> pm, - bool human_name, bool flip) { - Glib::ustring label; - if (app.world()->conf().option("port-labels").get<int32_t>()) { - if (human_name) { - const Atom& name = pm->get_property(app.uris().lv2_name); - if (name.type() == app.forge().String) { - label = name.ptr<char>(); - } else { - const SPtr<const BlockModel> parent(dynamic_ptr_cast<const BlockModel>(pm->parent())); - if (parent && parent->plugin_model()) - label = parent->plugin_model()->port_human_name(pm->index()); - } - } else { - label = pm->path().symbol(); - } - } - return new Port(app, module, pm, label, flip); + return new Port(app, module, pm, port_label(app, pm), flip); } /** @param flip Make an input port appear as an output port, and vice versa. @@ -118,6 +102,40 @@ Port::~Port() _app.activity_port_destroyed(this); } +std::string +Port::port_label(App& app, SPtr<const PortModel> pm) +{ + if (!pm) { + return ""; + } + + std::string label; + if (app.world()->conf().option("port-labels").get<int32_t>()) { + if (app.world()->conf().option("human-names").get<int32_t>()) { + const Atom& name = pm->get_property(app.uris().lv2_name); + if (name.type() == app.forge().String) { + label = name.ptr<char>(); + } else { + const SPtr<const BlockModel> parent( + dynamic_ptr_cast<const BlockModel>(pm->parent())); + if (parent && parent->plugin_model()) + label = parent->plugin_model()->port_human_name(pm->index()); + } + } else { + label = pm->path().symbol(); + } + } + return label; +} + +void +Port::ensure_label() +{ + if (!get_label()) { + set_label(port_label(_app, _port_model.lock()).c_str()); + } +} + void Port::update_metadata() { diff --git a/src/gui/Port.hpp b/src/gui/Port.hpp index e547fcc4..44703fb4 100644 --- a/src/gui/Port.hpp +++ b/src/gui/Port.hpp @@ -50,7 +50,6 @@ public: App& app, Ganv::Module& module, SPtr<const Client::PortModel> pm, - bool human_name, bool flip = false); ~Port(); @@ -59,6 +58,7 @@ public: bool show_menu(GdkEventButton* ev); void update_metadata(); + void ensure_label(); void value_changed(const Atom& value); void activity(const Atom& value); @@ -73,6 +73,8 @@ private: const std::string& name, bool flip = false); + static std::string port_label(App& app, SPtr<const Client::PortModel> pm); + Gtk::Menu* build_enum_menu(); Gtk::Menu* build_uri_menu(); GraphBox* get_graph_box() const; |