From 5546fa38fc0ba2d33288914d4fb40fe4d3d837bf Mon Sep 17 00:00:00 2001 From: Carl Hetherington Date: Mon, 6 Dec 2010 02:41:46 +0000 Subject: Manage attempts to save plugin presets with the same name. Helps with #2662. git-svn-id: svn://localhost/ardour2/branches/3.0@8191 d708f5d6-7413-0410-9779-e7cbd77b26cf --- libs/ardour/ardour/ladspa_plugin.h | 3 +- libs/ardour/ardour/lv2_plugin.h | 5 +- libs/ardour/ardour/plugin.h | 5 ++ libs/ardour/ladspa_plugin.cc | 6 ++ libs/ardour/lv2_plugin.cc | 6 ++ libs/ardour/plugin.cc | 128 +++++++++++++++++++++++++++++++------ 6 files changed, 129 insertions(+), 24 deletions(-) (limited to 'libs/ardour') diff --git a/libs/ardour/ardour/ladspa_plugin.h b/libs/ardour/ardour/ladspa_plugin.h index 968342a909..4358de792e 100644 --- a/libs/ardour/ardour/ladspa_plugin.h +++ b/libs/ardour/ardour/ladspa_plugin.h @@ -99,7 +99,8 @@ class LadspaPlugin : public ARDOUR::Plugin XMLNode& get_state(); int set_state (const XMLNode&, int version); - bool save_preset(std::string name); + bool save_preset (std::string name); + void remove_preset (std::string name); bool has_editor() const { return false; } diff --git a/libs/ardour/ardour/lv2_plugin.h b/libs/ardour/ardour/lv2_plugin.h index 821922903b..bdaf1a153b 100644 --- a/libs/ardour/ardour/lv2_plugin.h +++ b/libs/ardour/ardour/lv2_plugin.h @@ -114,8 +114,9 @@ class LV2Plugin : public ARDOUR::Plugin XMLNode& get_state(); int set_state(const XMLNode& node, int version); - bool save_preset(std::string uri); - bool load_preset(const std::string& uri); + bool save_preset (std::string uri); + void remove_preset (std::string uri); + bool load_preset (const std::string& uri); virtual std::vector get_presets(); bool has_editor() const; diff --git a/libs/ardour/ardour/plugin.h b/libs/ardour/ardour/plugin.h index bab2b71f31..4a2153c6e0 100644 --- a/libs/ardour/ardour/plugin.h +++ b/libs/ardour/ardour/plugin.h @@ -129,6 +129,7 @@ class Plugin : public PBD::StatefulDestructible, public Latent virtual bool parameter_is_output(uint32_t) const = 0; virtual bool save_preset (std::string) = 0; + virtual void remove_preset (std::string) = 0; virtual bool load_preset (const std::string& uri); struct PresetRecord { @@ -187,6 +188,10 @@ class Plugin : public PBD::StatefulDestructible, public Latent virtual void set_parameter (uint32_t which, float val) = 0; bool save_preset (std::string, std::string /* vst, ladspa etc. */); + void remove_preset (std::string, std::string); + bool write_preset_file (std::string, std::string); + std::string preset_source (std::string, std::string) const; + std::string preset_envvar () const; ARDOUR::AudioEngine& _engine; ARDOUR::Session& _session; diff --git a/libs/ardour/ladspa_plugin.cc b/libs/ardour/ladspa_plugin.cc index fb5a88e5f8..00454cea5e 100644 --- a/libs/ardour/ladspa_plugin.cc +++ b/libs/ardour/ladspa_plugin.cc @@ -372,6 +372,12 @@ LadspaPlugin::save_preset (string name) return Plugin::save_preset (name, "ladspa"); } +void +LadspaPlugin::remove_preset (string name) +{ + return Plugin::remove_preset (name, "ladspa"); +} + int LadspaPlugin::set_state (const XMLNode& node, int version) { diff --git a/libs/ardour/lv2_plugin.cc b/libs/ardour/lv2_plugin.cc index 9edd929aad..082c2f1ba3 100644 --- a/libs/ardour/lv2_plugin.cc +++ b/libs/ardour/lv2_plugin.cc @@ -386,6 +386,12 @@ LV2Plugin::save_preset (string /*name*/) return false; } +void +LV2Plugin::remove_preset (string /*name*/) +{ + return; +} + bool LV2Plugin::has_editor() const { diff --git a/libs/ardour/plugin.cc b/libs/ardour/plugin.cc index 366f683e56..4fe51141f6 100644 --- a/libs/ardour/plugin.cc +++ b/libs/ardour/plugin.cc @@ -162,6 +162,109 @@ Plugin::load_preset(const string& preset_uri) return true; } +/* XXX: should be in liblrdf */ +static void +lrdf_remove_preset (const char *source, const char *setting_uri) +{ + lrdf_statement p; + lrdf_statement *q; + lrdf_statement *i; + char setting_uri_copy[64]; + char buf[64]; + + strncpy(setting_uri_copy, setting_uri, sizeof(setting_uri_copy)); + + p.subject = setting_uri_copy; + strncpy(buf, LADSPA_BASE "hasPortValue", sizeof(buf)); + p.predicate = buf; + p.object = NULL; + q = lrdf_matches(&p); + + p.predicate = NULL; + p.object = NULL; + for (i = q; i; i = i->next) { + p.subject = i->object; + lrdf_remove_matches(&p); + } + + lrdf_free_statements(q); + + p.subject = NULL; + strncpy(buf, LADSPA_BASE "hasSetting", sizeof(buf)); + p.predicate = buf; + p.object = setting_uri_copy; + lrdf_remove_matches(&p); + + p.subject = setting_uri_copy; + p.predicate = NULL; + p.object = NULL; + lrdf_remove_matches (&p); +} + +void +Plugin::remove_preset (string name, string domain) +{ + string const envvar = preset_envvar (); + if (envvar.empty()) { + warning << _("Could not locate HOME. Preset not removed.") << endmsg; + return; + } + + Plugin::PresetRecord const * p = preset_by_label (name); + if (!p) { + return; + } + + string const source = preset_source (envvar, domain); + lrdf_remove_preset (source.c_str(), p->uri.c_str ()); + + presets.erase (p->uri); + + write_preset_file (envvar, domain); +} + +string +Plugin::preset_envvar () const +{ + char* envvar; + if ((envvar = getenv ("HOME")) == 0) { + return ""; + } + + return envvar; +} + +string +Plugin::preset_source (string envvar, string domain) const +{ + return string_compose ("file:%1/.%2/rdf/ardour-presets.n3", envvar, domain); +} + +bool +Plugin::write_preset_file (string envvar, string domain) +{ + string path = string_compose("%1/.%2", envvar, domain); + if (g_mkdir_with_parents (path.c_str(), 0775)) { + warning << string_compose(_("Could not create %1. Preset not saved. (%2)"), path, strerror(errno)) << endmsg; + return false; + } + + path += "/rdf"; + if (g_mkdir_with_parents (path.c_str(), 0775)) { + warning << string_compose(_("Could not create %1. Preset not saved. (%2)"), path, strerror(errno)) << endmsg; + return false; + } + + string const source = preset_source (envvar, domain); + + if (lrdf_export_by_source (source.c_str(), source.substr(5).c_str())) { + warning << string_compose(_("Error saving presets file %1."), source) << endmsg; + return false; + } + + return true; +} + bool Plugin::save_preset (string name, string domain) { @@ -191,13 +294,13 @@ Plugin::save_preset (string name, string domain) } } - char* envvar; - if ((envvar = getenv ("HOME")) == 0) { + string const envvar = preset_envvar (); + if (envvar.empty()) { warning << _("Could not locate HOME. Preset not saved.") << endmsg; return false; } - string source(string_compose("file:%1/.%2/rdf/ardour-presets.n3", envvar, domain)); + string const source = preset_source (envvar, domain); char* uri = lrdf_add_preset (source.c_str(), name.c_str(), id, &defaults); @@ -206,24 +309,7 @@ Plugin::save_preset (string name, string domain) presets.insert (make_pair (uri, PresetRecord (uri, name))); free (uri); - string path = string_compose("%1/.%2", envvar, domain); - if (g_mkdir_with_parents (path.c_str(), 0775)) { - warning << string_compose(_("Could not create %1. Preset not saved. (%2)"), path, strerror(errno)) << endmsg; - return false; - } - - path += "/rdf"; - if (g_mkdir_with_parents (path.c_str(), 0775)) { - warning << string_compose(_("Could not create %1. Preset not saved. (%2)"), path, strerror(errno)) << endmsg; - return false; - } - - if (lrdf_export_by_source(source.c_str(), source.substr(5).c_str())) { - warning << string_compose(_("Error saving presets file %1."), source) << endmsg; - return false; - } - - return true; + return write_preset_file (envvar, domain); } PluginPtr -- cgit v1.2.3