remove some unnecessary and hard-to-compile-on-OSX debug stuff
[ardour.git] / libs / ardour / plugin_insert.cc
index 8322770d31dffc2417e3d93a3bc076182df9d3c7..85e96411c28fb35b8442d365a433e25c00ecb9e7 100644 (file)
@@ -92,7 +92,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 */
@@ -114,7 +114,7 @@ PluginInsert::~PluginInsert ()
 }
 
 void
-PluginInsert::auto_state_changed (Evoral::Parameter which)
+PluginInsert::control_list_automation_state_changed (Evoral::Parameter which, AutoState s)
 {
        if (which.type() != PluginAutomation)
                return;
@@ -122,7 +122,7 @@ PluginInsert::auto_state_changed (Evoral::Parameter which)
        boost::shared_ptr<AutomationControl> c
                        = boost::dynamic_pointer_cast<AutomationControl>(control (which));
 
-       if (c && ((AutomationList*)c->list().get())->automation_state() != Off) {
+       if (c && s != Off) {
                _plugins[0]->set_parameter (which.id(), c->list()->eval (_session.transport_frame()));
        }
 }
@@ -195,7 +195,7 @@ PluginInsert::set_automatable ()
                           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)));
@@ -221,22 +221,26 @@ PluginInsert::parameter_changed (Evoral::Parameter which, float val)
        }
 }
 
-void
+int
 PluginInsert::set_block_size (nframes_t nframes)
 {
+        int ret = 0;
        for (Plugins::iterator i = _plugins.begin(); i != _plugins.end(); ++i) {
-               (*i)->set_block_size (nframes);
+               if ((*i)->set_block_size (nframes) != 0) {
+                        ret = -1;
+                }
        }
+        return ret;
 }
 
 void
 PluginInsert::activate ()
 {
-       Processor::activate ();
-
        for (Plugins::iterator i = _plugins.begin(); i != _plugins.end(); ++i) {
                (*i)->activate ();
        }
+       
+       Processor::activate ();
 }
 
 void
@@ -249,6 +253,14 @@ PluginInsert::deactivate ()
        }
 }
 
+void
+PluginInsert::flush ()
+{
+        for (vector<boost::shared_ptr<Plugin> >::iterator i = _plugins.begin(); i != _plugins.end(); ++i) {
+                (*i)->flush ();
+        }
+}
+
 void
 PluginInsert::connect_and_run (BufferSet& bufs, nframes_t nframes, nframes_t offset, bool with_auto, nframes_t now)
 {
@@ -355,9 +367,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*/, nframes_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);
@@ -390,8 +403,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 */
 
@@ -436,7 +450,7 @@ PluginInsert::automation_run (BufferSet& bufs, nframes_t nframes)
                return;
        }
 
-       if (!find_next_event (now, end, next_event)) {
+       if (!find_next_event (now, end, next_event) || requires_fixed_sized_buffers()) {
 
                /* no events have a time within the relevant range */
 
@@ -671,25 +685,6 @@ 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);
-
        return node;
 }
 
@@ -768,10 +763,10 @@ 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));
                }
        }
 
@@ -795,6 +790,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) */
@@ -804,10 +803,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
@@ -823,66 +818,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)
 {
@@ -1069,3 +1004,12 @@ 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 ();
+       }
+}