From 86607097d618f778ea984cba2a7dc9c403fe8966 Mon Sep 17 00:00:00 2001
From: Robin Gareus <robin@gareus.org>
Date: Sun, 5 Jun 2016 19:04:33 +0200
Subject: [PATCH] plugin support for Controllable::NotAutomatable

---
 libs/ardour/automation_control.cc |  3 ++
 libs/ardour/lv2_plugin.cc         |  2 +-
 libs/ardour/plugin_insert.cc      | 47 +++++++++++++++++++------------
 3 files changed, 33 insertions(+), 19 deletions(-)

diff --git a/libs/ardour/automation_control.cc b/libs/ardour/automation_control.cc
index d8dd582505..7dc39114a2 100644
--- a/libs/ardour/automation_control.cc
+++ b/libs/ardour/automation_control.cc
@@ -140,6 +140,9 @@ AutomationControl::set_list (boost::shared_ptr<Evoral::ControlList> list)
 void
 AutomationControl::set_automation_state (AutoState as)
 {
+	if (flags() & NotAutomatable) {
+		return;
+	}
 	if (_list && as != alist()->automation_state()) {
 
 		alist()->set_automation_state (as);
diff --git a/libs/ardour/lv2_plugin.cc b/libs/ardour/lv2_plugin.cc
index fe69546e3a..ce090a79a4 100644
--- a/libs/ardour/lv2_plugin.cc
+++ b/libs/ardour/lv2_plugin.cc
@@ -2067,7 +2067,7 @@ LV2Plugin::automatable() const
 	set<Evoral::Parameter> ret;
 
 	for (uint32_t i = 0; i < parameter_count(); ++i) {
-		if (parameter_is_input(i) && parameter_is_control(i)) {
+		if (parameter_is_input(i) && parameter_is_control(i) && !(_port_flags[i] & PORT_NOAUTO)) {
 			ret.insert(ret.end(), Evoral::Parameter(PluginAutomation, 0, i));
 		}
 	}
diff --git a/libs/ardour/plugin_insert.cc b/libs/ardour/plugin_insert.cc
index 688645eb94..291ccbfc54 100644
--- a/libs/ardour/plugin_insert.cc
+++ b/libs/ardour/plugin_insert.cc
@@ -409,31 +409,42 @@ PluginInsert::create_automatable_parameters ()
 {
 	assert (!_plugins.empty());
 
+	boost::shared_ptr<Plugin> plugin = _plugins.front();
 	set<Evoral::Parameter> a = _plugins.front()->automatable ();
 
-	for (set<Evoral::Parameter>::iterator i = a.begin(); i != a.end(); ++i) {
-		if (i->type() == PluginAutomation) {
+	for (uint32_t i = 0; i < plugin->parameter_count(); ++i) {
+		if (!plugin->parameter_is_control (i) || !plugin->parameter_is_input (i)) {
+			continue;
+		}
+		Evoral::Parameter param (PluginAutomation, 0, i);
 
-			Evoral::Parameter param(*i);
+		ParameterDescriptor desc;
+		plugin->get_parameter_descriptor(i, desc);
 
-			ParameterDescriptor desc;
-			_plugins.front()->get_parameter_descriptor(i->id(), desc);
+		const bool automatable = a.find(param) != a.end();
 
+		if (automatable) {
 			can_automate (param);
-			boost::shared_ptr<AutomationList> list(new AutomationList(param, desc));
-			boost::shared_ptr<AutomationControl> c (new PluginControl(this, param, desc, list));
-			add_control (c);
-			_plugins.front()->set_automation_control (i->id(), c);
-		} else if (i->type() == PluginPropertyAutomation) {
-			Evoral::Parameter param(*i);
-			const ParameterDescriptor& desc = _plugins.front()->get_property_descriptor(param.id());
-			if (desc.datatype != Variant::NOTHING) {
-				boost::shared_ptr<AutomationList> list;
-				if (Variant::type_is_numeric(desc.datatype)) {
-					list = boost::shared_ptr<AutomationList>(new AutomationList(param, desc));
-				}
-				add_control (boost::shared_ptr<AutomationControl> (new PluginPropertyControl(this, param, desc, list)));
+		}
+		boost::shared_ptr<AutomationList> list(new AutomationList(param, desc));
+		boost::shared_ptr<AutomationControl> c (new PluginControl(this, param, desc, list));
+		if (!automatable) {
+			c->set_flags (Controllable::Flag ((int)c->flags() | Controllable::NotAutomatable));
+		}
+		add_control (c);
+		plugin->set_automation_control (i, c);
+	}
+
+	const Plugin::PropertyDescriptors& pdl (plugin->get_supported_properties ());
+	for (Plugin::PropertyDescriptors::const_iterator p = pdl.begin(); p != pdl.end(); ++p) {
+		Evoral::Parameter param (PluginPropertyAutomation, 0, p->first);
+		const ParameterDescriptor& desc = plugin->get_property_descriptor(param.id());
+		if (desc.datatype != Variant::NOTHING) {
+			boost::shared_ptr<AutomationList> list;
+			if (Variant::type_is_numeric(desc.datatype)) {
+				list = boost::shared_ptr<AutomationList>(new AutomationList(param, desc));
 			}
+			add_control (boost::shared_ptr<AutomationControl> (new PluginPropertyControl(this, param, desc, list)));
 		}
 	}
 }
-- 
2.30.2