summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDavid Robillard <d@drobilla.net>2015-10-24 22:14:06 +0000
committerDavid Robillard <d@drobilla.net>2015-10-24 22:14:06 +0000
commit557f9d42641f7f602f9fec4d2f3aae50b893de6b (patch)
tree06b7ef9e9d5f153bd878fc084ba0ab838bf8bd71
parente4d549a7d7322103904ac8ba8d5a4711c6901870 (diff)
downloadingen-557f9d42641f7f602f9fec4d2f3aae50b893de6b.tar.gz
ingen-557f9d42641f7f602f9fec4d2f3aae50b893de6b.tar.bz2
ingen-557f9d42641f7f602f9fec4d2f3aae50b893de6b.zip
Fix LV2 UIs that send during instantiation
git-svn-id: http://svn.drobilla.net/lad/trunk/ingen@5780 a436a847-0d15-0410-975c-d299462d15a1
-rw-r--r--ingen/client/PluginUI.hpp21
-rw-r--r--src/client/PluginUI.cpp93
-rw-r--r--src/gui/App.cpp4
-rw-r--r--src/gui/NodeModule.cpp20
4 files changed, 91 insertions, 47 deletions
diff --git a/ingen/client/PluginUI.hpp b/ingen/client/PluginUI.hpp
index f29f2994..1ccadc58 100644
--- a/ingen/client/PluginUI.hpp
+++ b/ingen/client/PluginUI.hpp
@@ -42,10 +42,24 @@ class INGEN_API PluginUI {
public:
~PluginUI();
+ /** Create a UI for the given block and plugin.
+ *
+ * This does not actually instantiate the UI itself, so signals can be
+ * connected first. The caller should connect to signal_property_changed,
+ * then call instantiate().
+ */
static SPtr<PluginUI> create(Ingen::World* world,
SPtr<const BlockModel> block,
const LilvPlugin* plugin);
+ /** Instantiate the UI.
+ *
+ * If true is returned, instantiation was successfull and the widget can be
+ * obtained with get_widget(). Otherwise, instantiation failed, so there is
+ * no widget and the UI can not be used.
+ */
+ bool instantiate();
+
SuilWidget get_widget();
void port_event(uint32_t port_index,
@@ -71,12 +85,17 @@ public:
private:
PluginUI(Ingen::World* world,
SPtr<const BlockModel> block,
- const LilvNode* ui_node);
+ LilvUIs* uis,
+ const LilvUI* ui,
+ const LilvNode* ui_type);
Ingen::World* _world;
SPtr<const BlockModel> _block;
SuilInstance* _instance;
+ LilvUIs* _uis;
+ const LilvUI* _ui;
LilvNode* _ui_node;
+ LilvNode* _ui_type;
std::set<uint32_t> _subscribed_ports;
static SuilHost* ui_host;
diff --git a/src/client/PluginUI.cpp b/src/client/PluginUI.cpp
index a16dcb2c..9ec97d82 100644
--- a/src/client/PluginUI.cpp
+++ b/src/client/PluginUI.cpp
@@ -146,11 +146,16 @@ lv2_ui_unsubscribe(SuilController controller,
PluginUI::PluginUI(Ingen::World* world,
SPtr<const BlockModel> block,
- const LilvNode* ui_node)
+ LilvUIs* uis,
+ const LilvUI* ui,
+ const LilvNode* ui_type)
: _world(world)
, _block(block)
, _instance(NULL)
- , _ui_node(lilv_node_duplicate(ui_node))
+ , _uis(uis)
+ , _ui(ui)
+ , _ui_node(lilv_node_duplicate(lilv_ui_get_uri(ui)))
+ , _ui_type(lilv_node_duplicate(ui_type))
{
}
@@ -161,6 +166,8 @@ PluginUI::~PluginUI()
}
suil_instance_free(_instance);
lilv_node_free(_ui_node);
+ lilv_node_free(_ui_type);
+ lilv_uis_free(_uis);
}
SPtr<PluginUI>
@@ -202,35 +209,46 @@ PluginUI::create(Ingen::World* world,
}
// Create the PluginUI, but don't instantiate yet
- SPtr<PluginUI> ret(new PluginUI(world, block, lilv_ui_get_uri(ui)));
+ SPtr<PluginUI> ret(new PluginUI(world, block, uis, ui, ui_type));
ret->_features = world->lv2_features().lv2_features(
world, const_cast<BlockModel*>(block.get()));
+ return ret;
+}
+
+bool
+PluginUI::instantiate()
+{
+ const URIs& uris = _world->uris();
+ const std::string plugin_uri = _block->plugin()->uri();
+
/* Subscribe (enable broadcast) for any requested port notifications. This
must be done before instantiation so responses to any events sent by the
UI's init() will be sent back to this client. */
- LilvWorld* lworld = world->lilv_world();
+ LilvWorld* lworld = _world->lilv_world();
LilvNode* ui_portNotification = lilv_new_uri(lworld, LV2_UI__portNotification);
LilvNode* ui_plugin = lilv_new_uri(lworld, LV2_UI__plugin);
LilvNodes* notes = lilv_world_find_nodes(
- lworld, lilv_ui_get_uri(ui), ui_portNotification, NULL);
+ lworld, lilv_ui_get_uri(_ui), ui_portNotification, NULL);
LILV_FOREACH(nodes, n, notes) {
const LilvNode* note = lilv_nodes_get(notes, n);
const LilvNode* sym = lilv_world_get(lworld, note, uris.lv2_symbol, NULL);
const LilvNode* plug = lilv_world_get(lworld, note, ui_plugin, NULL);
- if (plug && !lilv_node_is_uri(plug)) {
- world->log().error(fmt("%1% UI has non-URI ui:plugin\n")
- % block->plugin()->uri().c_str());
- continue;
- }
- if (plug && strcmp(lilv_node_as_uri(plug), block->plugin()->uri().c_str())) {
- continue; // Notification is for another plugin
- }
- if (sym) {
- uint32_t index = lv2_ui_port_index(ret.get(), lilv_node_as_string(sym));
+ if (!plug) {
+ _world->log().error(fmt("%1% UI %2% notification missing plugin\n")
+ % plugin_uri % lilv_node_as_string(_ui_node));
+ } else if (!sym) {
+ _world->log().error(fmt("%1% UI %2% notification missing symbol\n")
+ % plugin_uri % lilv_node_as_string(_ui_node));
+ } else if (!lilv_node_is_uri(plug)) {
+ _world->log().error(fmt("%1% UI %2% notification has non-URI plugin\n")
+ % plugin_uri % lilv_node_as_string(_ui_node));
+ } else if (strcmp(lilv_node_as_uri(plug), plugin_uri.c_str())) {
+ // Notification is valid and for this plugin
+ uint32_t index = lv2_ui_port_index(this, lilv_node_as_string(sym));
if (index != LV2UI_INVALID_PORT_INDEX) {
- lv2_ui_subscribe(ret.get(), index, 0, NULL);
- ret->_subscribed_ports.insert(index);
+ lv2_ui_subscribe(this, index, 0, NULL);
+ _subscribed_ports.insert(index);
}
}
}
@@ -238,39 +256,36 @@ PluginUI::create(Ingen::World* world,
lilv_node_free(ui_plugin);
lilv_node_free(ui_portNotification);
- const char* bundle_uri = lilv_node_as_uri(lilv_ui_get_bundle_uri(ui));
- const char* binary_uri = lilv_node_as_uri(lilv_ui_get_binary_uri(ui));
+ const char* bundle_uri = lilv_node_as_uri(lilv_ui_get_bundle_uri(_ui));
+ const char* binary_uri = lilv_node_as_uri(lilv_ui_get_binary_uri(_ui));
char* bundle_path = lilv_file_uri_parse(bundle_uri, NULL);
char* binary_path = lilv_file_uri_parse(binary_uri, NULL);
// Instantiate the actual plugin UI via Suil
- SuilInstance* instance = suil_instance_new(
+ _instance = suil_instance_new(
PluginUI::ui_host,
- ret.get(),
- lilv_node_as_uri(gtk_ui),
- lilv_node_as_uri(lilv_plugin_get_uri(plugin)),
- lilv_node_as_uri(lilv_ui_get_uri(ui)),
- lilv_node_as_uri(ui_type),
+ this,
+ LV2_UI__GtkUI,
+ plugin_uri.c_str(),
+ lilv_node_as_uri(lilv_ui_get_uri(_ui)),
+ lilv_node_as_uri(_ui_type),
bundle_path,
binary_path,
- ret->_features->array());
+ _features->array());
lilv_free(binary_path);
lilv_free(bundle_path);
- lilv_node_free(gtk_ui);
- if (!instance) {
- world->log().error("Failed to instantiate LV2 UI\n");
+ if (!_instance) {
+ _world->log().error("Failed to instantiate LV2 UI\n");
// Cancel any subscriptions
- for (uint32_t i : ret->_subscribed_ports) {
- lv2_ui_unsubscribe(ret.get(), i, 0, NULL);
+ for (uint32_t i : _subscribed_ports) {
+ lv2_ui_unsubscribe(this, i, 0, NULL);
}
- return SPtr<PluginUI>();
+ return false;
}
- ret->_instance = instance;
-
- return ret;
+ return true;
}
SuilWidget
@@ -285,8 +300,12 @@ PluginUI::port_event(uint32_t port_index,
uint32_t format,
const void* buffer)
{
- suil_instance_port_event(
- _instance, port_index, buffer_size, format, buffer);
+ if (_instance) {
+ suil_instance_port_event(
+ _instance, port_index, buffer_size, format, buffer);
+ } else {
+ _world->log().warn("LV2 UI port event with no instance\n");
+ }
}
bool
diff --git a/src/gui/App.cpp b/src/gui/App.cpp
index 0c7aea8c..2624d1e9 100644
--- a/src/gui/App.cpp
+++ b/src/gui/App.cpp
@@ -234,7 +234,9 @@ App::set_property(const Raul::URI& subject,
feedback and bandwidth wastage, see Delta.cpp). So, assume everything
went as planned here and fire the signal ourselves as if the server
feedback came back immediately. */
- _client->signal_property_change().emit(subject, key, value);
+ if (key != uris().ingen_activity) {
+ _client->signal_property_change().emit(subject, key, value);
+ }
}
void
diff --git a/src/gui/NodeModule.cpp b/src/gui/NodeModule.cpp
index ae9a997d..4e149bbf 100644
--- a/src/gui/NodeModule.cpp
+++ b/src/gui/NodeModule.cpp
@@ -235,14 +235,18 @@ NodeModule::embed_gui(bool embed)
_plugin_ui->signal_property_changed().connect(
sigc::mem_fun(app(), &App::set_property));
- GtkWidget* c_widget = (GtkWidget*)_plugin_ui->get_widget();
- _gui_widget = Glib::wrap(c_widget);
-
- Gtk::Container* container = new Gtk::EventBox();
- container->set_name("IngenEmbeddedUI");
- container->set_border_width(4.0);
- container->add(*_gui_widget);
- Ganv::Module::embed(container);
+ if (!_plugin_ui->instantiate()) {
+ app().log().error("Failed to instantiate LV2 UI\n");
+ } else {
+ GtkWidget* c_widget = (GtkWidget*)_plugin_ui->get_widget();
+ _gui_widget = Glib::wrap(c_widget);
+
+ Gtk::Container* container = new Gtk::EventBox();
+ container->set_name("IngenEmbeddedUI");
+ container->set_border_width(4.0);
+ container->add(*_gui_widget);
+ Ganv::Module::embed(container);
+ }
} else {
app().log().error("Failed to create LV2 UI\n");
}