From ec6a85f921b2b6c950c6c94f0392c059cffba5dd Mon Sep 17 00:00:00 2001 From: Robin Gareus Date: Sat, 9 Sep 2017 04:39:10 +0200 Subject: [PATCH] GenericUI: subscribe to property changes before querying values Fixes displaying the initial/current filename/path for plugins that have a LV2:Patch path property. --- gtk2_ardour/generic_pluginui.cc | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/gtk2_ardour/generic_pluginui.cc b/gtk2_ardour/generic_pluginui.cc index 62a69bef1e..6d26da1b1e 100644 --- a/gtk2_ardour/generic_pluginui.cc +++ b/gtk2_ardour/generic_pluginui.cc @@ -208,14 +208,6 @@ GenericPluginUI::GenericPluginUI (boost::shared_ptr pi, bool scrol scroller.set_policy (Gtk::POLICY_AUTOMATIC, Gtk::POLICY_NEVER); } - - /* Listen for property changes that are not notified normally because - * AutomationControl has only support for numeric values currently. - * The only case is Variant::PATH for now */ - plugin->PropertyChanged.connect(*this, invalidator(*this), - boost::bind(&GenericPluginUI::path_property_changed, this, _1, _2), - gui_context()); - main_contents.show (); } @@ -392,6 +384,14 @@ GenericPluginUI::build () control_uis.push_back(cui); } if (!descs.empty()) { + /* Listen for property changes that are not notified normally because + * AutomationControl has only support for numeric values currently. + * The only case is Variant::PATH for now */ + plugin->PropertyChanged.connect(*this, invalidator(*this), + boost::bind(&GenericPluginUI::path_property_changed, this, _1, _2), + gui_context()); + + /* and query current property value */ plugin->announce_property_values(); }