Add a null check to ports when loading an export channel from xml + fix indentation...
[ardour.git] / libs / ardour / plugin_insert.cc
index 73655b24b1b4a1f505baa65f31edefe3a6b3bcd3..25c885f5a676357c4b0aa3d512dbf6f3655c6728 100644 (file)
@@ -26,6 +26,7 @@
 
 #include "pbd/failed_constructor.h"
 #include "pbd/xml++.h"
+#include "pbd/convert.h"
 
 #include "ardour/audio_buffer.h"
 #include "ardour/automation_list.h"
@@ -92,7 +93,7 @@ PluginInsert::set_count (uint32_t num)
                uint32_t diff = num - _plugins.size();
 
                for (uint32_t n = 0; n < diff; ++n) {
-                       _plugins.push_back (plugin_factory (_plugins[0]));
+                       add_plugin_with_activation (_plugins[0]);
 
                        if (require_state) {
                                /* XXX do something */
@@ -190,15 +191,15 @@ PluginInsert::set_automatable ()
                        Evoral::Parameter param(*i);
 
                        _plugins.front()->get_parameter_descriptor(i->id(), desc);
-                       
+                        
                        /* the Parameter belonging to the actual plugin doesn't have its range set
                           but we want the Controllable related to this Parameter to have those limits.
                        */
 
-                       param.set_range (desc.lower, desc.upper, _plugins.front()->default_value(i->id()));
+                       param.set_range (desc.lower, desc.upper, _plugins.front()->default_value(i->id()), desc.toggled);
                        can_automate (param);
                        boost::shared_ptr<AutomationList> list(new AutomationList(param));
-                       add_control (boost::shared_ptr<AutomationControl>(new PluginControl(this, param, list)));
+                        add_control (boost::shared_ptr<AutomationControl> (new PluginControl(this, param, list)));
                }
        }
 }
@@ -222,7 +223,7 @@ PluginInsert::parameter_changed (Evoral::Parameter which, float val)
 }
 
 int
-PluginInsert::set_block_size (nframes_t nframes)
+PluginInsert::set_block_size (pframes_t nframes)
 {
         int ret = 0;
        for (Plugins::iterator i = _plugins.begin(); i != _plugins.end(); ++i) {
@@ -262,11 +263,11 @@ PluginInsert::flush ()
 }
 
 void
-PluginInsert::connect_and_run (BufferSet& bufs, nframes_t nframes, nframes_t offset, bool with_auto, nframes_t now)
+PluginInsert::connect_and_run (BufferSet& bufs, pframes_t nframes, framecnt_t offset, bool with_auto, framepos_t now)
 {
        // Calculate if, and how many frames we need to collect for analysis
-       nframes_t collect_signal_nframes = (_signal_analysis_collect_nframes_max -
-                                            _signal_analysis_collected_nframes);
+       framecnt_t collect_signal_nframes = (_signal_analysis_collect_nframes_max -
+                                            _signal_analysis_collected_nframes);
        if (nframes < collect_signal_nframes) { // we might not get all frames now
                collect_signal_nframes = nframes;
        }
@@ -354,7 +355,7 @@ PluginInsert::connect_and_run (BufferSet& bufs, nframes_t nframes, nframes_t off
 }
 
 void
-PluginInsert::silence (nframes_t nframes)
+PluginInsert::silence (framecnt_t nframes)
 {
        ChanMapping in_map(input_streams());
        ChanMapping out_map(output_streams());
@@ -367,9 +368,10 @@ PluginInsert::silence (nframes_t nframes)
 }
 
 void
-PluginInsert::run (BufferSet& bufs, sframes_t /*start_frame*/, sframes_t /*end_frame*/, nframes_t nframes, bool)
+PluginInsert::run (BufferSet& bufs, framepos_t /*start_frame*/, framepos_t /*end_frame*/, pframes_t nframes, bool)
 {
-       if (_active || _pending_active) {
+       if (_pending_active) {
+               /* run as normal if we are active or moving from inactive to active */
 
                if (_session.transport_rolling()) {
                        automation_run (bufs, nframes);
@@ -402,8 +404,9 @@ PluginInsert::run (BufferSet& bufs, sframes_t /*start_frame*/, sframes_t /*end_f
 void
 PluginInsert::set_parameter (Evoral::Parameter param, float val)
 {
-       if (param.type() != PluginAutomation)
+       if (param.type() != PluginAutomation) {
                return;
+        }
 
        /* the others will be set from the event triggered by this */
 
@@ -434,12 +437,12 @@ PluginInsert::get_parameter (Evoral::Parameter param)
 }
 
 void
-PluginInsert::automation_run (BufferSet& bufs, nframes_t nframes)
+PluginInsert::automation_run (BufferSet& bufs, pframes_t nframes)
 {
        Evoral::ControlEvent next_event (0, 0.0f);
-       nframes_t now = _session.transport_frame ();
-       nframes_t end = now + nframes;
-       nframes_t offset = 0;
+       framepos_t now = _session.transport_frame ();
+       framepos_t end = now + nframes;
+       framecnt_t offset = 0;
 
        Glib::Mutex::Lock lm (control_lock(), Glib::TRY_LOCK);
 
@@ -458,7 +461,7 @@ PluginInsert::automation_run (BufferSet& bufs, nframes_t nframes)
 
        while (nframes) {
 
-               nframes_t cnt = min (((nframes_t) ceil (next_event.when) - now), nframes);
+               framecnt_t cnt = min (((framecnt_t) ceil (next_event.when) - now), (framecnt_t) nframes);
 
                connect_and_run (bufs, cnt, offset, true, now);
 
@@ -683,28 +686,41 @@ PluginInsert::state (bool full)
        node.add_property("count", string_compose("%1", _plugins.size()));
        node.add_child_nocopy (_plugins[0]->get_state());
 
-       /* add port automation state */
-       XMLNode *autonode = new XMLNode(port_automation_node_name);
-       set<Evoral::Parameter> automatable = _plugins[0]->automatable();
-
-       for (set<Evoral::Parameter>::iterator x = automatable.begin(); x != automatable.end(); ++x) {
-               
-               /*XMLNode* child = new XMLNode("port");
-               snprintf(buf, sizeof(buf), "%" PRIu32, *x);
-               child->add_property("number", string(buf));
-               
-               child->add_child_nocopy (automation_list (*x).state (full));
-               autonode->add_child_nocopy (*child);
-               */
-               AutomationList* list = dynamic_cast<AutomationList*>(control(*x)->list().get());
-               autonode->add_child_nocopy (list->state (full));
-       }
-
-       node.add_child_nocopy (*autonode);
+       for (Controls::iterator c = controls().begin(); c != controls().end(); ++c) {
+                boost::shared_ptr<AutomationControl> ac = boost::dynamic_pointer_cast<AutomationControl> ((*c).second);
+                if (ac) {
+                        node.add_child_nocopy (ac->get_state());
+                }
+        }
 
        return node;
 }
 
+void
+PluginInsert::set_control_ids (const XMLNode& node, int version)
+{
+        const XMLNodeList& nlist = node.children();
+        XMLNodeConstIterator iter;
+        set<Evoral::Parameter>::const_iterator p;
+
+        for (iter = nlist.begin(); iter != nlist.end(); ++iter) {
+                if ((*iter)->name() == Controllable::xml_node_name) {
+                        const XMLProperty* prop;
+
+                        if ((prop = (*iter)->property (X_("parameter"))) != 0) {
+                                uint32_t p = atoi (prop->value());
+                                boost::shared_ptr<Evoral::Control> c = control (Evoral::Parameter (PluginAutomation, 0, p));
+                                if (!c) {
+                                        continue;
+                                }
+                                boost::shared_ptr<AutomationControl> ac = boost::dynamic_pointer_cast<AutomationControl> (c);
+                                if (ac) {
+                                        ac->set_state (**iter, version);
+                                }
+                        }
+                }
+        }
+}
 int
 PluginInsert::set_state(const XMLNode& node, int version)
 {
@@ -780,15 +796,16 @@ PluginInsert::set_state(const XMLNode& node, int version)
 
        if (_plugins.size() != count) {
 
-               _plugins.push_back (plugin);
+               add_plugin_with_activation (plugin);
 
                for (uint32_t n = 1; n < count; ++n) {
-                       _plugins.push_back (plugin_factory (plugin));
+                       add_plugin_with_activation (plugin_factory (plugin));
                }
        }
 
        if (need_automatables) {
                set_automatable ();
+                set_control_ids (node, version);
        }
 
        /* Handle the node list for this Processor (or Insert if an A2 session) */
@@ -807,6 +824,10 @@ PluginInsert::set_state(const XMLNode& node, int version)
 
        if (version < 3000) {
 
+                /* Only 2.X sessions need a call to set_parameter_state() - in 3.X and above
+                   this is all handled by Automatable
+                */
+
                for (niter = nlist.begin(); niter != nlist.end(); ++niter) {
                        if ((*niter)->name() == "Redirect") {
                                /* XXX do we need to tackle placement? i think not (pd; oct 16 2009) */
@@ -816,10 +837,6 @@ PluginInsert::set_state(const XMLNode& node, int version)
                }
                
                set_parameter_state_2X (node, version);
-               
-       } else {
-
-               set_parameter_state (node, version);
        }
 
        // The name of the PluginInsert comes from the plugin, nothing else
@@ -835,66 +852,6 @@ PluginInsert::set_state(const XMLNode& node, int version)
        return 0;
 }
 
-void
-PluginInsert::set_parameter_state (const XMLNode& node, int version)
-{
-       XMLNodeList nlist = node.children();
-       XMLNodeIterator niter;
-
-       for (niter = nlist.begin(); niter != nlist.end(); ++niter) {
-               
-               if ((*niter)->name() != port_automation_node_name) {
-                       continue;
-               }
-               
-               XMLNodeList cnodes;
-               XMLProperty *cprop;
-               XMLNodeConstIterator iter;
-               XMLNode *child;
-               const char *port;
-               uint32_t port_id;
-
-               cnodes = (*niter)->children ("AutomationList");
-
-               for (iter = cnodes.begin(); iter != cnodes.end(); ++iter) {
-
-                       child = *iter;
-
-                       /* XXX this code knows way too much about the internal details of an AutomationList state node */
-
-                       if ((cprop = child->property("automation-id")) != 0) {
-                               port = cprop->value().c_str();
-                       } else {
-                               warning << _("PluginInsert: Auto: no plugin parameter number seen") << endmsg;
-                               continue;
-                       }
-
-                       if (sscanf (port, "parameter-%" PRIu32, &port_id) != 1) {
-                               warning << _("PluginInsert: Auto: no parameter number found") << endmsg;
-                               continue;
-                       }
-
-                       if (port_id >= _plugins[0]->parameter_count()) {
-                               warning << _("PluginInsert: Auto: plugin parameter out of range") << endmsg;
-                               continue;
-                       }
-
-                       boost::shared_ptr<AutomationControl> c = boost::dynamic_pointer_cast<AutomationControl>(
-                                       control(Evoral::Parameter(PluginAutomation, 0, port_id), true));
-
-                       if (c) {
-                               c->alist()->set_state (*child, version);
-                       } else {
-                               error << string_compose (_("PluginInsert: automatable control %1 not found - ignored"), port_id) << endmsg;
-                       }
-               }
-
-               /* done */
-               
-               break;
-       }
-}
-
 void
 PluginInsert::set_parameter_state_2X (const XMLNode& node, int version)
 {
@@ -958,13 +915,14 @@ PluginInsert::set_parameter_state_2X (const XMLNode& node, int version)
 string
 PluginInsert::describe_parameter (Evoral::Parameter param)
 {
-       if (param.type() != PluginAutomation)
+       if (param.type() != PluginAutomation) {
                return Automatable::describe_parameter(param);
+        }
 
        return _plugins[0]->describe_parameter (param);
 }
 
-ARDOUR::nframes_t
+ARDOUR::framecnt_t
 PluginInsert::signal_latency() const
 {
        if (_user_latency) {
@@ -1032,6 +990,18 @@ PluginInsert::PluginControl::set_value (double val)
        AutomationControl::set_value(val);
 }
 
+XMLNode&
+PluginInsert::PluginControl::get_state ()
+{
+        stringstream ss;
+
+        XMLNode& node (AutomationControl::get_state());
+        ss << parameter().id();
+        node.add_property (X_("parameter"), ss.str());
+
+        return node;
+}
+
 double
 PluginInsert::PluginControl::get_value (void) const
 {
@@ -1070,7 +1040,7 @@ PluginInsert::get_impulse_analysis_plugin()
 }
 
 void
-PluginInsert::collect_signal_for_analysis(nframes_t nframes)
+PluginInsert::collect_signal_for_analysis (framecnt_t nframes)
 {
        // called from outside the audio thread, so this should be safe
        // only do audio as analysis is (currently) only for audio plugins
@@ -1081,3 +1051,20 @@ PluginInsert::collect_signal_for_analysis(nframes_t nframes)
        _signal_analysis_collect_nframes_max = nframes;
 }
 
+/** Add a plugin to our list and activate it if we have already been activated */
+void
+PluginInsert::add_plugin_with_activation (boost::shared_ptr<Plugin> plugin)
+{
+       _plugins.push_back (plugin);
+       if (active()) {
+               plugin->activate ();
+       }
+}
+
+void
+PluginInsert::realtime_handle_transport_stopped ()
+{
+       for (Plugins::iterator i = _plugins.begin(); i != _plugins.end(); ++i) {
+               (*i)->realtime_handle_transport_stopped ();
+       }
+}