switch to using boost::signals2 instead of sigc++, at least for libardour. not finish...
[ardour.git] / libs / ardour / plugin_insert.cc
index 75930b78062f3bc5a4965692533969ed4943daf5..c2bf7f3562728389ecb773b73ee9b7fa55bc1e59 100644 (file)
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2000 Paul Davis 
+    Copyright (C) 2000 Paul Davis
 
     This program is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
 
 */
 
+#ifdef WAF_BUILD
+#include "libardour-config.h"
+#endif
+
 #include <string>
 
-#include <sigc++/bind.h>
 
-#include <pbd/failed_constructor.h>
-#include <pbd/xml++.h>
+#include "pbd/failed_constructor.h"
+#include "pbd/xml++.h"
 
-#include <ardour/plugin_insert.h>
-#include <ardour/plugin.h>
-#include <ardour/port.h>
-#include <ardour/route.h>
-#include <ardour/ladspa_plugin.h>
-#include <ardour/buffer_set.h>
-#include <ardour/automation_list.h>
-#include <ardour/audio_buffer.h>
+#include "ardour/audio_buffer.h"
+#include "ardour/automation_list.h"
+#include "ardour/buffer_set.h"
+#include "ardour/event_type_map.h"
+#include "ardour/ladspa_plugin.h"
+#include "ardour/plugin.h"
+#include "ardour/plugin_insert.h"
+#include "ardour/port.h"
+#include "ardour/route.h"
 
 #ifdef HAVE_SLV2
-#include <ardour/lv2_plugin.h>
+#include "ardour/lv2_plugin.h"
 #endif
 
 #ifdef VST_SUPPORT
-#include <ardour/vst_plugin.h>
+#include "ardour/vst_plugin.h"
 #endif
 
 #ifdef HAVE_AUDIOUNITS
-#include <ardour/audio_unit.h>
+#include "ardour/audio_unit.h"
 #endif
 
-#include <ardour/audioengine.h>
-#include <ardour/session.h>
-#include <ardour/types.h>
+#include "ardour/audioengine.h"
+#include "ardour/session.h"
+#include "ardour/types.h"
 
 #include "i18n.h"
 
@@ -57,16 +61,15 @@ using namespace PBD;
 
 const string PluginInsert::port_automation_node_name = "PortAutomation";
 
-PluginInsert::PluginInsert (Session& s, boost::shared_ptr<Plugin> plug, Placement placement)
-       : Processor (s, plug->name(), placement),
-          _signal_analysis_collected_nframes(0),
-          _signal_analysis_collect_nframes_max(0)
+PluginInsert::PluginInsert (Session& s, boost::shared_ptr<Plugin> plug)
+       : Processor (s, plug->name())
+       , _signal_analysis_collected_nframes(0)
+       , _signal_analysis_collect_nframes_max(0)
 {
        /* the first is the master */
 
        _plugins.push_back (plug);
-
-       init ();
+       set_automatable ();
 
        {
                Glib::Mutex::Lock em (_session.engine().process_lock());
@@ -77,18 +80,15 @@ PluginInsert::PluginInsert (Session& s, boost::shared_ptr<Plugin> plug, Placemen
 }
 
 PluginInsert::PluginInsert (Session& s, const XMLNode& node)
-       : Processor (s, "unnamed plugin insert", PreFader),
+       : Processor (s, "unnamed plugin insert"),
           _signal_analysis_collected_nframes(0),
           _signal_analysis_collect_nframes_max(0)
 {
-       if (set_state (node)) {
+       if (set_state (node, Stateful::loading_state_version)) {
                throw failed_constructor();
        }
 
-       // XXX: This would dump all automation, which has already been loaded by
-       //      Processor. But this could also have been related to the Parameter change..
-       //      will look into this later.
-       //set_automatable ();
+       _pending_active = _active;
 
        {
                Glib::Mutex::Lock em (_session.engine().process_lock());
@@ -102,10 +102,10 @@ PluginInsert::set_count (uint32_t num)
        bool require_state = !_plugins.empty();
 
        /* this is a bad idea.... we shouldn't do this while active.
-          only a route holding their redirect_lock should be calling this 
+          only a route holding their redirect_lock should be calling this
        */
 
-       if (num == 0) { 
+       if (num == 0) {
                return false;
        } else if (num > _plugins.size()) {
                uint32_t diff = num - _plugins.size();
@@ -128,15 +128,9 @@ PluginInsert::set_count (uint32_t num)
        return true;
 }
 
-void
-PluginInsert::init ()
-{
-       set_automatable ();
-}
-
 PluginInsert::~PluginInsert ()
 {
-       GoingAway (); /* EMIT SIGNAL */
+       drop_references ();
 }
 
 void
@@ -159,14 +153,10 @@ PluginInsert::output_streams() const
        ChanCount out = _plugins.front()->get_info()->n_outputs;
 
        if (out == ChanCount::INFINITE) {
-
                return _plugins.front()->output_streams ();
-
        } else {
-
                out.set_audio (out.n_audio() * _plugins.size());
                out.set_midi (out.n_midi() * _plugins.size());
-
                return out;
        }
 }
@@ -175,13 +165,12 @@ ChanCount
 PluginInsert::input_streams() const
 {
        ChanCount in = _plugins[0]->get_info()->n_inputs;
-       
+
        if (in == ChanCount::INFINITE) {
                return _plugins[0]->input_streams ();
        } else {
                in.set_audio (in.n_audio() * _plugins.size());
                in.set_midi (in.n_midi() * _plugins.size());
-
                return in;
        }
 }
@@ -233,7 +222,7 @@ PluginInsert::parameter_changed (Evoral::Parameter which, float val)
        if (which.type() != PluginAutomation)
                return;
 
-       vector<boost::shared_ptr<Plugin> >::iterator i = _plugins.begin();
+       Plugins::iterator i = _plugins.begin();
 
        /* don't set the first plugin, just all the slaves */
 
@@ -248,7 +237,7 @@ PluginInsert::parameter_changed (Evoral::Parameter which, float val)
 void
 PluginInsert::set_block_size (nframes_t nframes)
 {
-       for (vector<boost::shared_ptr<Plugin> >::iterator i = _plugins.begin(); i != _plugins.end(); ++i) {
+       for (Plugins::iterator i = _plugins.begin(); i != _plugins.end(); ++i) {
                (*i)->set_block_size (nframes);
        }
 }
@@ -256,7 +245,9 @@ PluginInsert::set_block_size (nframes_t nframes)
 void
 PluginInsert::activate ()
 {
-       for (vector<boost::shared_ptr<Plugin> >::iterator i = _plugins.begin(); i != _plugins.end(); ++i) {
+       Processor::activate ();
+
+       for (Plugins::iterator i = _plugins.begin(); i != _plugins.end(); ++i) {
                (*i)->activate ();
        }
 }
@@ -264,7 +255,9 @@ PluginInsert::activate ()
 void
 PluginInsert::deactivate ()
 {
-       for (vector<boost::shared_ptr<Plugin> >::iterator i = _plugins.begin(); i != _plugins.end(); ++i) {
+       Processor::deactivate ();
+
+       for (Plugins::iterator i = _plugins.begin(); i != _plugins.end(); ++i) {
                (*i)->deactivate ();
        }
 }
@@ -279,8 +272,8 @@ PluginInsert::connect_and_run (BufferSet& bufs, nframes_t nframes, nframes_t off
                collect_signal_nframes = nframes;
        }
 
-       uint32_t in_index = 0;
-       uint32_t out_index = 0;
+       ChanMapping in_map(input_streams());
+       ChanMapping out_map(output_streams());
 
        /* Note that we've already required that plugins
           be able to handle in-place processing.
@@ -289,22 +282,22 @@ PluginInsert::connect_and_run (BufferSet& bufs, nframes_t nframes, nframes_t off
        if (with_auto) {
 
                uint32_t n = 0;
-               
+
                for (Controls::iterator li = data().controls().begin(); li != data().controls().end(); ++li, ++n) {
-                       
+
                        boost::shared_ptr<AutomationControl> c
                                = boost::dynamic_pointer_cast<AutomationControl>(li->second);
 
                        if (c->parameter().type() == PluginAutomation && c->automation_playback()) {
                                bool valid;
 
-                               const float val = c->list()->rt_safe_eval (now, valid);                         
+                               const float val = c->list()->rt_safe_eval (now, valid);
 
                                if (valid) {
                                        c->set_value(val);
                                }
 
-                       } 
+                       }
                }
        }
 
@@ -313,26 +306,37 @@ PluginInsert::connect_and_run (BufferSet& bufs, nframes_t nframes, nframes_t off
                //std::cerr << "collect input, bufs " << bufs.count().n_audio() << " count,  " << bufs.available().n_audio() << " available" << std::endl;
                //std::cerr << "               streams " << input_streams().n_audio() << std::endl;
                //std::cerr << "filling buffer with " << collect_signal_nframes << " frames at " << _signal_analysis_collected_nframes << std::endl;
+
+               _signal_analysis_inputs.set_count(input_streams());
+
                for (uint32_t i = 0; i < input_streams().n_audio(); ++i) {
-                       _signal_analysis_input_bufferset.get_audio(i).read_from(
+                       _signal_analysis_inputs.get_audio(i).read_from(
                                bufs.get_audio(i),
                                collect_signal_nframes,
                                _signal_analysis_collected_nframes); // offset is for target buffer
                }
+
        }
 
-       for (vector<boost::shared_ptr<Plugin> >::iterator i = _plugins.begin(); i != _plugins.end(); ++i) {
-               (*i)->connect_and_run (bufs, in_index, out_index, nframes, offset);
+       for (Plugins::iterator i = _plugins.begin(); i != _plugins.end(); ++i) {
+               (*i)->connect_and_run(bufs, in_map, out_map, nframes, offset);
+               for (DataType::iterator t = DataType::begin(); t != DataType::end(); ++t) {
+                       in_map.offset_to(*t, natural_input_streams().get(*t));
+                       out_map.offset_to(*t, natural_output_streams().get(*t));
+               }
        }
 
        if (collect_signal_nframes > 0) {
                // collect output
                //std::cerr << "       output, bufs " << bufs.count().n_audio() << " count,  " << bufs.available().n_audio() << " available" << std::endl;
                //std::cerr << "               streams " << output_streams().n_audio() << std::endl;
+
+               _signal_analysis_outputs.set_count(output_streams());
+
                for (uint32_t i = 0; i < output_streams().n_audio(); ++i) {
-                       _signal_analysis_output_bufferset.get_audio(i).read_from(
-                               bufs.get_audio(i), 
-                               collect_signal_nframes, 
+                       _signal_analysis_outputs.get_audio(i).read_from(
+                               bufs.get_audio(i),
+                               collect_signal_nframes,
                                _signal_analysis_collected_nframes); // offset is for target buffer
                }
 
@@ -343,36 +347,37 @@ PluginInsert::connect_and_run (BufferSet& bufs, nframes_t nframes, nframes_t off
                        _signal_analysis_collect_nframes_max = 0;
                        _signal_analysis_collected_nframes   = 0;
 
-                       AnalysisDataGathered(&_signal_analysis_input_bufferset, 
-                                            &_signal_analysis_output_bufferset);
+                       AnalysisDataGathered(&_signal_analysis_inputs,
+                                            &_signal_analysis_outputs);
                }
        }
        /* leave remaining channel buffers alone */
 }
 
 void
-PluginInsert::silence (nframes_t nframes, nframes_t offset)
+PluginInsert::silence (nframes_t nframes)
 {
-       uint32_t in_index = 0;
-       uint32_t out_index = 0;
+       ChanMapping in_map(input_streams());
+       ChanMapping out_map(output_streams());
 
        if (active()) {
-               for (vector<boost::shared_ptr<Plugin> >::iterator i = _plugins.begin(); i != _plugins.end(); ++i) {
-                       (*i)->connect_and_run (_session.get_silent_buffers ((*i)->get_info()->n_inputs), in_index, out_index, nframes, offset);
+               for (Plugins::iterator i = _plugins.begin(); i != _plugins.end(); ++i) {
+                       (*i)->connect_and_run (_session.get_silent_buffers ((*i)->get_info()->n_inputs), in_map, out_map, nframes, 0);
                }
        }
 }
-       
+
 void
-PluginInsert::run_in_place (BufferSet& bufs, nframes_t start_frame, nframes_t end_frame, nframes_t nframes, nframes_t offset)
+PluginInsert::run (BufferSet& bufs, sframes_t /*start_frame*/, sframes_t /*end_frame*/, nframes_t nframes, bool)
 {
-       if (active()) {
+       if (_active || _pending_active) {
 
                if (_session.transport_rolling()) {
-                       automation_run (bufs, nframes, offset);
+                       automation_run (bufs, nframes);
                } else {
-                       connect_and_run (bufs, nframes, offset, false);
+                       connect_and_run (bufs, nframes, 0, false);
                }
+
        } else {
 
                /* FIXME: type, audio only */
@@ -383,14 +388,16 @@ PluginInsert::run_in_place (BufferSet& bufs, nframes_t start_frame, nframes_t en
                if (out > in) {
 
                        /* not active, but something has make up for any channel count increase */
-                       
+
                        for (uint32_t n = out - in; n < out; ++n) {
-                               memcpy (bufs.get_audio(n).data(nframes, offset), bufs.get_audio(in - 1).data(nframes, offset), sizeof (Sample) * nframes);
+                               memcpy (bufs.get_audio(n).data(), bufs.get_audio(in - 1).data(), sizeof (Sample) * nframes);
                        }
                }
 
                bufs.count().set_audio(out);
        }
+
+       _active = _pending_active;
 }
 
 void
@@ -402,10 +409,10 @@ PluginInsert::set_parameter (Evoral::Parameter param, float val)
        /* the others will be set from the event triggered by this */
 
        _plugins[0]->set_parameter (param.id(), val);
-       
+
        boost::shared_ptr<AutomationControl> ac
                        = boost::dynamic_pointer_cast<AutomationControl>(data().control(param));
-       
+
        if (ac) {
                ac->set_value(val);
        } else {
@@ -427,11 +434,12 @@ PluginInsert::get_parameter (Evoral::Parameter param)
 }
 
 void
-PluginInsert::automation_run (BufferSet& bufs, nframes_t nframes, nframes_t offset)
+PluginInsert::automation_run (BufferSet& bufs, nframes_t nframes)
 {
        Evoral::ControlEvent next_event (0, 0.0f);
        nframes_t now = _session.transport_frame ();
        nframes_t end = now + nframes;
+       nframes_t offset = 0;
 
        Glib::Mutex::Lock lm (data().control_lock(), Glib::TRY_LOCK);
 
@@ -439,36 +447,36 @@ PluginInsert::automation_run (BufferSet& bufs, nframes_t nframes, nframes_t offs
                connect_and_run (bufs, nframes, offset, false);
                return;
        }
-       
+
        if (!data().find_next_event (now, end, next_event)) {
-               
-               /* no events have a time within the relevant range */
-               
-               connect_and_run (bufs, nframes, offset, true, now);
-               return;
-       }
-       
-       while (nframes) {
+
+               /* no events have a time within the relevant range */
+
+               connect_and_run (bufs, nframes, offset, true, now);
+               return;
+       }
+
+       while (nframes) {
 
                nframes_t cnt = min (((nframes_t) ceil (next_event.when) - now), nframes);
-  
-               connect_and_run (bufs, cnt, offset, true, now);
-               
-               nframes -= cnt;
-               offset += cnt;
+
+               connect_and_run (bufs, cnt, offset, true, now);
+
+               nframes -= cnt;
+               offset += cnt;
                now += cnt;
 
                if (!data().find_next_event (now, end, next_event)) {
                        break;
                }
-       }
-  
-       /* cleanup anything that is left to do */
-  
-       if (nframes) {
-               connect_and_run (bufs, nframes, offset, true, now);
-       }
-}      
+       }
+
+       /* cleanup anything that is left to do */
+
+       if (nframes) {
+               connect_and_run (bufs, nframes, offset, true, now);
+       }
+}
 
 float
 PluginInsert::default_parameter_value (const Evoral::Parameter& param)
@@ -542,14 +550,15 @@ PluginInsert::configure_io (ChanCount in, ChanCount out)
 
        // we don't know the analysis window size, so we must work with the
        // current buffer size here. each request for data fills in these
-       // buffers and the analyser makes sure it gets enough data for the 
+       // buffers and the analyser makes sure it gets enough data for the
        // analysis window
-       _signal_analysis_input_bufferset.ensure_buffers (in,  session().engine().frames_per_cycle());
-       _signal_analysis_input_bufferset.set_count(in);
+       session().ensure_buffer_set (_signal_analysis_inputs, in);
+       //_signal_analysis_inputs.set_count (in);
 
-       _signal_analysis_output_bufferset.ensure_buffers(out, session().engine().frames_per_cycle());
-       _signal_analysis_output_bufferset.set_count(out);
+       session().ensure_buffer_set (_signal_analysis_outputs, out);
+       //_signal_analysis_outputs.set_count (out);
 
+       // std::cerr << "set counts to i" << in.n_audio() << "/o" << out.n_audio() << std::endl;
 
        return Processor::configure_io (in, out);
 }
@@ -557,68 +566,67 @@ PluginInsert::configure_io (ChanCount in, ChanCount out)
 bool
 PluginInsert::can_support_io_configuration (const ChanCount& in, ChanCount& out) const
 {
+       // Plugin has flexible I/O, so delegate to it
        if (_plugins.front()->reconfigurable_io()) {
-               /* plugin has flexible I/O, so delegate to it */
                return _plugins.front()->can_support_io_configuration (in, out);
        }
 
+       ChanCount inputs  = _plugins[0]->get_info()->n_inputs;
        ChanCount outputs = _plugins[0]->get_info()->n_outputs;
-       ChanCount inputs = _plugins[0]->get_info()->n_inputs;
 
-       if ((inputs.n_total() == 0)
-                       || (inputs.n_total() == 1 && outputs == inputs)
-                       || (inputs.n_total() == 1 && outputs == inputs
-                               && ((inputs.n_audio() == 0 && in.n_audio() == 0)
-                                       || (inputs.n_midi() == 0 && in.n_midi() == 0)))
-                       || (inputs == in)) {
+       // Plugin inputs match requested inputs exactly
+       if (inputs == in) {
                out = outputs;
                return true;
        }
 
-       bool can_replicate = true;
-
-       /* if number of inputs is a factor of the requested input
-          configuration for every type, we can replicate.
-       */
+       // See if replication is possible
+       // We can replicate if there exists a single factor f such that, for every type,
+       // the number of plugin inputs * f = the requested number of inputs
+       uint32_t f             = 0;
+       bool     can_replicate = true;
        for (DataType::iterator t = DataType::begin(); t != DataType::end(); ++t) {
-               if (inputs.get(*t) >= in.get(*t) || (inputs.get(*t) % in.get(*t) != 0)) {
+               // No inputs of this type
+               if (inputs.get(*t) == 0 && in.get(*t) == 0) {
+                       continue;
+
+               // Plugin has more inputs than requested, can not replicate
+               } else if (inputs.get(*t) >= in.get(*t)) {
+                       can_replicate = false;
+                       break;
+
+               // Plugin inputs is not a factor of requested inputs, can not replicate
+               } else if (inputs.get(*t) == 0 || in.get(*t) % inputs.get(*t) != 0) {
                        can_replicate = false;
                        break;
+
+               // Potential factor not set yet
+               } else if (f == 0) {
+                       f = in.get(*t) / inputs.get(*t);;
                }
-       }
 
-       if (!can_replicate || (in.n_total() % inputs.n_total() != 0)) {
-               return false;
+               // Factor for this type does not match another type, can not replicate
+               if (f != (in.get(*t) / inputs.get(*t))) {
+                       can_replicate = false;
+                       break;
+               }
        }
 
-       if (inputs.n_total() == 0) {
-               /* instrument plugin, always legal, but throws away any existing streams */
-               out = outputs;
-       } else if (inputs.n_total() == 1 && outputs == inputs
-                       && ((inputs.n_audio() == 0 && in.n_audio() == 0)
-                           || (inputs.n_midi() == 0 && in.n_midi() == 0))) {
-               /* mono, single-typed plugin, replicate as needed to match in */
-               out = in;
-       } else if (inputs == in) {
-               /* exact match */
-               out = outputs;
-       } else {
-               /* replicate - note that we've already verified that
-                  the replication count is constant across all data types.
-               */
+       if (can_replicate) {
                for (DataType::iterator t = DataType::begin(); t != DataType::end(); ++t) {
-                       out.set (*t, outputs.get(*t) * (in.get(*t) / inputs.get(*t)));
+                       out.set (*t, outputs.get(*t) * f);
                }
+               return true;
+       } else {
+               return false;
        }
-               
-       return true;
 }
 
 /* Number of plugin instances required to support a given channel configuration.
  * (private helper)
  */
 int32_t
-PluginInsert::count_for_configuration (ChanCount in, ChanCount out) const
+PluginInsert::count_for_configuration (ChanCount in, ChanCount /*out*/) const
 {
        if (_plugins.front()->reconfigurable_io()) {
                /* plugin has flexible I/O, so the answer is always 1 */
@@ -627,7 +635,7 @@ PluginInsert::count_for_configuration (ChanCount in, ChanCount out) const
        }
 
        // FIXME: take 'out' into consideration
-       
+
        ChanCount outputs = _plugins[0]->get_info()->n_outputs;
        ChanCount inputs = _plugins[0]->get_info()->n_inputs;
 
@@ -676,9 +684,9 @@ PluginInsert::state (bool full)
        node.add_child_nocopy (_plugins[0]->get_state());
 
        /* add port automation state */
-       //XMLNode *autonode = new XMLNode(port_automation_node_name);
+       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");
@@ -688,16 +696,16 @@ PluginInsert::state (bool full)
                child->add_child_nocopy (automation_list (*x).state (full));
                autonode->add_child_nocopy (*child);
                */
-               //autonode->add_child_nocopy (((AutomationList*)data().control(*x)->list().get())->state (full));
+               autonode->add_child_nocopy (((AutomationList*)data().control(*x)->list().get())->state (full));
        }
 
-       //node.add_child_nocopy (*autonode);
-       
+       node.add_child_nocopy (*autonode);
+
        return node;
 }
 
 int
-PluginInsert::set_state(const XMLNode& node)
+PluginInsert::set_state(const XMLNode& node, int version)
 {
        XMLNodeList nlist = node.children();
        XMLNodeIterator niter;
@@ -706,7 +714,7 @@ PluginInsert::set_state(const XMLNode& node)
        ARDOUR::PluginType type;
 
        if ((prop = node.property ("type")) == 0) {
-               error << _("XML node describing insert is missing the `type' field") << endmsg;
+               error << _("XML node describing plugin is missing the `type' field") << endmsg;
                return -1;
        }
 
@@ -716,65 +724,166 @@ PluginInsert::set_state(const XMLNode& node)
                type = ARDOUR::LV2;
        } else if (prop->value() == X_("vst")) {
                type = ARDOUR::VST;
+       } else if (prop->value() == X_("audiounit")) {
+               type = ARDOUR::AudioUnit;
        } else {
                error << string_compose (_("unknown plugin type %1 in plugin insert state"),
                                  prop->value())
                      << endmsg;
                return -1;
        }
-       
+
        prop = node.property ("unique-id");
+
        if (prop == 0) {
-               error << _("Plugin has no unique ID field") << endmsg;
-               return -1;
+#ifdef VST_SUPPORT
+               /* older sessions contain VST plugins with only an "id" field.
+                */
+               
+               if (type == ARDOUR::VST) {
+                       prop = node.property ("id");
+               }
+#endif         
+               /* recheck  */
+
+               if (prop == 0) {
+                       error << _("Plugin has no unique ID field") << endmsg;
+                       return -1;
+               }
        }
 
        boost::shared_ptr<Plugin> plugin;
-       
-       plugin = find_plugin (_session, prop->value(), type);   
+
+       plugin = find_plugin (_session, prop->value(), type);
 
        if (plugin == 0) {
                error << string_compose(_("Found a reference to a plugin (\"%1\") that is unknown.\n"
-                                  "Perhaps it was removed or moved since it was last used."), prop->value()) 
+                                  "Perhaps it was removed or moved since it was last used."), prop->value())
                      << endmsg;
                return -1;
        }
 
        uint32_t count = 1;
+       bool need_automatables = true;
+
+       if (_plugins.empty()) {
+               /* if we are adding the first plugin, we will need to set
+                  up automatable controls.
+               */
+               need_automatables = true;
+       }
 
        if ((prop = node.property ("count")) != 0) {
                sscanf (prop->value().c_str(), "%u", &count);
        }
 
        if (_plugins.size() != count) {
-               
+
                _plugins.push_back (plugin);
-               
-               for (uint32_t n=1; n < count; ++n) {
+
+               for (uint32_t n = 1; n < count; ++n) {
                        _plugins.push_back (plugin_factory (plugin));
                }
        }
-       
+
+       if (need_automatables) {
+               set_automatable ();
+       }
+
        for (niter = nlist.begin(); niter != nlist.end(); ++niter) {
                if ((*niter)->name() == plugin->state_node_name()) {
-                       for (vector<boost::shared_ptr<Plugin> >::iterator i = _plugins.begin(); i != _plugins.end(); ++i) {
-                               (*i)->set_state (**niter);
+                       for (Plugins::iterator i = _plugins.begin(); i != _plugins.end(); ++i) {
+                               (*i)->set_state (**niter, version);
                        }
                        break;
                }
-       } 
+       }
+
+       if (version < 3000) {
+               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) */
+                               Processor::set_state (**niter, version);
+                               break;
+                       }
+               }
+               set_parameter_state_2X (node, version);
+       } else {
+               Processor::set_state (node, version);
+               set_parameter_state (node, version);
+       }
+
+       // The name of the PluginInsert comes from the plugin, nothing else
+       _name = plugin->get_info()->name;
+
+       return 0;
+}
 
-       const XMLNode* insert_node = &node;
+void
+PluginInsert::set_parameter_state (const XMLNode& node, int version)
+{
+       XMLNodeList nlist = node.children();
+       XMLNodeIterator niter;
 
-       // legacy sessions: search for child IOProcessor node
        for (niter = nlist.begin(); niter != nlist.end(); ++niter) {
-               if ((*niter)->name() == "IOProcessor") {
-                       insert_node = *niter;
-                       break;
+               
+               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>(
+                                       data().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;
        }
-       
-       Processor::set_state (*insert_node);
+}
+
+void
+PluginInsert::set_parameter_state_2X (const XMLNode& node, int version)
+{
+       XMLNodeList nlist = node.children();
+       XMLNodeIterator niter;
 
        /* look for port automation node */
        
@@ -791,7 +900,7 @@ PluginInsert::set_state(const XMLNode& node)
                const char *port;
                uint32_t port_id;
                
-               cnodes = (*niter)->children ("Port");
+               cnodes = (*niter)->children ("port");
                
                for(iter = cnodes.begin(); iter != cnodes.end(); ++iter){
                        
@@ -800,7 +909,7 @@ PluginInsert::set_state(const XMLNode& node)
                        if ((cprop = child->property("number")) != 0) {
                                port = cprop->value().c_str();
                        } else {
-                               warning << _("PluginInsert: Auto: no plugin port number") << endmsg;
+                               warning << _("PluginInsert: Auto: no ladspa port number") << endmsg;
                                continue;
                        }
                        
@@ -814,42 +923,22 @@ PluginInsert::set_state(const XMLNode& node)
                        boost::shared_ptr<AutomationControl> c = boost::dynamic_pointer_cast<AutomationControl>(
                                        data().control(Evoral::Parameter(PluginAutomation, 0, port_id), true));
 
-                       if (!child->children().empty()) {
-                               c->alist()->set_state (*child->children().front());
-                       } else {
-                               if ((cprop = child->property("auto")) != 0) {
-                                       
-                                       /* old school */
-
-                                       int x;
-                                       sscanf (cprop->value().c_str(), "0x%x", &x);
-                                       c->alist()->set_automation_state (AutoState (x));
-
-                               } else {
-                                       
-                                       /* missing */
-                                       
-                                       c->alist()->set_automation_state (Off);
+                       if (c) {
+                               if (!child->children().empty()) {
+                                       c->alist()->set_state (*child->children().front(), version);
                                }
+                       } else {
+                               error << string_compose (_("PluginInsert: automatable control %1 not found - ignored"), port_id) << endmsg;
                        }
-
                }
-
+               
                /* done */
-
+               
                break;
        } 
-
-       if (niter == nlist.end()) {
-               warning << string_compose(_("XML node describing a port automation is missing the `%1' information"), port_automation_node_name) << endmsg;
-       }
-       
-       // The name of the PluginInsert comes from the plugin, nothing else
-       _name = plugin->get_info()->name;
-       
-       return 0;
 }
 
+
 string
 PluginInsert::describe_parameter (Evoral::Parameter param)
 {
@@ -859,7 +948,7 @@ PluginInsert::describe_parameter (Evoral::Parameter param)
        return _plugins[0]->describe_parameter (param);
 }
 
-ARDOUR::nframes_t 
+ARDOUR::nframes_t
 PluginInsert::signal_latency() const
 {
        if (_user_latency) {
@@ -879,7 +968,7 @@ PluginInsert::type ()
 #ifdef HAVE_AUDIOUNITS
        boost::shared_ptr<AUPlugin> ap;
 #endif
-       
+
        PluginPtr other = plugin ();
 
        if ((lp = boost::dynamic_pointer_cast<LadspaPlugin> (other)) != 0) {
@@ -907,12 +996,12 @@ PluginInsert::PluginControl::PluginControl (PluginInsert* p, const Evoral::Param
        _logarithmic = desc.logarithmic;
        _toggled = desc.toggled;
 }
-        
+
 void
 PluginInsert::PluginControl::set_value (float val)
 {
        /* FIXME: probably should be taking out some lock here.. */
-       
+
        if (_toggled) {
                if (val > 0.5) {
                        val = 1.0;
@@ -920,7 +1009,7 @@ PluginInsert::PluginControl::set_value (float val)
                        val = 0.0;
                }
        } else {
-                       
+
                /*const float range = _list->get_max_y() - _list->get_min_y();
                const float lower = _list->get_min_y();
 
@@ -937,7 +1026,7 @@ PluginInsert::PluginControl::set_value (float val)
 
        }
 
-       for (vector<boost::shared_ptr<Plugin> >::iterator i = _plugin->_plugins.begin();
+       for (Plugins::iterator i = _plugin->_plugins.begin();
                        i != _plugin->_plugins.end(); ++i) {
                (*i)->set_parameter (_list->parameter().id(), val);
        }
@@ -954,21 +1043,21 @@ float
 PluginInsert::PluginControl::get_value (void) const
 {
        /* FIXME: probably should be taking out some lock here.. */
-       
+
        float val = _plugin->get_parameter (_list->parameter());
 
        return val;
 
        /*if (_toggled) {
-               
+
                return val;
-               
+
        } else {
-               
+
                if (_logarithmic) {
                        val = log(val);
                }
-               
+
                return ((val - lower) / range);
        }*/
 }
@@ -987,3 +1076,15 @@ PluginInsert::get_impulse_analysis_plugin()
        return ret;
 }
 
+void
+PluginInsert::collect_signal_for_analysis(nframes_t nframes)
+{
+       // called from outside the audio thread, so this should be safe
+       // only do audio as analysis is (currently) only for audio plugins
+       _signal_analysis_inputs.ensure_buffers(  DataType::AUDIO, input_streams().n_audio(),  nframes);
+       _signal_analysis_outputs.ensure_buffers( DataType::AUDIO, output_streams().n_audio(), nframes);
+
+       _signal_analysis_collected_nframes   = 0;
+       _signal_analysis_collect_nframes_max = nframes;
+}
+