Fix a potential memory-corruption
[ardour.git] / libs / ardour / plugin_insert.cc
index 4fc97fb0c816ab0bcafaa9d7de0b82cead81b708..9cb654bcb4470b522aea8e909425e8c92796adc0 100644 (file)
@@ -824,14 +824,14 @@ PluginInsert::inplace_silence_unconnected (BufferSet& bufs, const PinMappings& o
 void
 PluginInsert::connect_and_run (BufferSet& bufs, samplepos_t start, samplepos_t end, double speed, pframes_t nframes, samplecnt_t offset, bool with_auto)
 {
-       // TODO: atomically copy maps & _no_inplace
-       PinMappings in_map (_in_map);
-       PinMappings out_map (_out_map);
-       ChanMapping thru_map (_thru_map);
-       if (_mapping_changed) { // ToDo use a counters, increment until match.
+       if (_mapping_changed) { // ToDo use a counter, increment until match
                _no_inplace = check_inplace ();
                _mapping_changed = false;
        }
+       // TODO: atomically copy maps & _no_inplace
+       PinMappings in_map (_in_map); // TODO Split case below overrides, use const& in_map
+       PinMappings const& out_map (_out_map);
+       ChanMapping const& thru_map (_thru_map);
 
        if (_latency_changed) {
                /* delaylines are configured with the max possible latency (as reported by the plugin)
@@ -860,11 +860,11 @@ PluginInsert::connect_and_run (BufferSet& bufs, samplepos_t start, samplepos_t e
                                continue;
                        }
                        bool valid;
-                       uint32_t first_idx = in_map[0].get (*t, 0, &valid);
+                       uint32_t first_idx = in_map.p(0).get (*t, 0, &valid);
                        assert (valid && first_idx == 0); // check_inplace ensures this
                        /* copy the first stream's buffer contents to the others */
                        for (uint32_t i = 1; i < natural_input_streams ().get (*t); ++i) {
-                               uint32_t idx = in_map[0].get (*t, i, &valid);
+                               uint32_t idx = in_map.p(0).get (*t, i, &valid);
                                if (valid) {
                                        assert (idx == 0);
                                        bufs.get (*t, i).read_from (bufs.get (*t, first_idx), nframes, offset, offset);
@@ -882,29 +882,20 @@ PluginInsert::connect_and_run (BufferSet& bufs, samplepos_t start, samplepos_t e
 
                uint32_t n = 0;
 
-               for (Controls::iterator li = controls().begin(); li != controls().end(); ++li, ++n) {
-
-                       boost::shared_ptr<AutomationControl> c
-                               = boost::dynamic_pointer_cast<AutomationControl>(li->second);
+               for (Controls::const_iterator li = controls().begin(); li != controls().end(); ++li, ++n) {
 
-                       if (c->list() && c->automation_playback()) {
+                       /* boost::dynamic_pointer_cast<> has significant overhead, since we know that
+                        * all controls are AutomationControl and their lists - if any - are AutomationList,
+                        * we can use static_cast<>. This yields a speedup of 2.8/4.6 over to the
+                        * previous code (measuerd with VeeSeeVSTRack 10k parameters, optimized build) */
+                       AutomationControl& c = static_cast<AutomationControl&> (*(li->second));
+                       boost::shared_ptr<const Evoral::ControlList> clist (c.list());
+                       if (clist && (static_cast<AutomationList const&> (*clist)).automation_playback ()) {
                                bool valid;
-
-                               const float val = c->list()->rt_safe_eval (start, valid);
-
+                               const float val = c.list()->rt_safe_eval (start, valid);
                                if (valid) {
-                                       /* This is the ONLY place where we are
-                                        *  allowed to call
-                                        *  AutomationControl::set_value_unchecked(). We
-                                        *  know that the control is in
-                                        *  automation playback mode, so no
-                                        *  check on writable() is required
-                                        *  (which must be done in AutomationControl::set_value()
-                                        *
-                                        */
-                                       c->set_value_unchecked(val);
+                                       c.set_value_unchecked(val);
                                }
-
                        }
                }
        }
@@ -922,7 +913,7 @@ PluginInsert::connect_and_run (BufferSet& bufs, samplepos_t start, samplepos_t e
                //std::cerr << "               streams " << internal_input_streams().n_audio() << std::endl;
                //std::cerr << "filling buffer with " << collect_signal_nframes << " samples at " << _signal_analysis_collected_nframes << std::endl;
 
-               _signal_analysis_inputs.set_count(input_streams());
+               _signal_analysis_inputs.set_count (ChanCount (DataType::AUDIO, input_streams().n_audio()));
 
                for (uint32_t i = 0; i < input_streams().n_audio(); ++i) {
                        _signal_analysis_inputs.get_audio(i).read_from (
@@ -959,7 +950,7 @@ PluginInsert::connect_and_run (BufferSet& bufs, samplepos_t start, samplepos_t e
                        for (DataType::iterator t = DataType::begin(); t != DataType::end(); ++t) {
                                for (uint32_t out = 0; out < natural_output_streams().get (*t); ++out) {
                                        bool valid;
-                                       uint32_t out_idx = out_map[pc].get (*t, out, &valid);
+                                       uint32_t out_idx = out_map.p(pc).get (*t, out, &valid);
                                        if (valid) {
                                                used_outputs.set (*t, out_idx, 1); // mark as used
                                        }
@@ -991,14 +982,14 @@ PluginInsert::connect_and_run (BufferSet& bufs, samplepos_t start, samplepos_t e
                for (Plugins::iterator i = _plugins.begin(); i != _plugins.end(); ++i, ++pc) {
 
                        ARDOUR::ChanMapping i_in_map (natural_input_streams());
-                       ARDOUR::ChanMapping i_out_map (out_map[pc]);
+                       ARDOUR::ChanMapping i_out_map (out_map.p(pc));
                        ARDOUR::ChanCount mapped;
 
                        /* map inputs sequentially */
                        for (DataType::iterator t = DataType::begin(); t != DataType::end(); ++t) {
                                for (uint32_t in = 0; in < natural_input_streams().get (*t); ++in) {
                                        bool valid;
-                                       uint32_t in_idx = in_map[pc].get (*t, in, &valid);
+                                       uint32_t in_idx = in_map.p(pc).get (*t, in, &valid);
                                        uint32_t m = mapped.get (*t);
                                        if (valid) {
                                                inplace_bufs.get (*t, m).read_from (bufs.get (*t, in_idx), nframes, offset, offset);
@@ -1044,7 +1035,7 @@ PluginInsert::connect_and_run (BufferSet& bufs, samplepos_t start, samplepos_t e
                /* in-place processing */
                uint32_t pc = 0;
                for (Plugins::iterator i = _plugins.begin(); i != _plugins.end(); ++i, ++pc) {
-                       if ((*i)->connect_and_run(bufs, start, end, speed, in_map[pc], out_map[pc], nframes, offset)) {
+                       if ((*i)->connect_and_run(bufs, start, end, speed, in_map.p(pc), out_map.p(pc), nframes, offset)) {
                                deactivate ();
                        }
                }
@@ -1057,7 +1048,7 @@ PluginInsert::connect_and_run (BufferSet& bufs, samplepos_t start, samplepos_t e
                //std::cerr << "       output, bufs " << bufs.count().n_audio() << " count,  " << bufs.available().n_audio() << " available" << std::endl;
                //std::cerr << "               streams " << internal_output_streams().n_audio() << std::endl;
 
-               _signal_analysis_outputs.set_count(output_streams());
+               _signal_analysis_outputs.set_count (ChanCount (DataType::AUDIO, output_streams().n_audio()));
 
                for (uint32_t i = 0; i < output_streams().n_audio(); ++i) {
                        _signal_analysis_outputs.get_audio(i).read_from(
@@ -1090,14 +1081,13 @@ PluginInsert::bypass (BufferSet& bufs, pframes_t nframes)
        /* bypass the plugin(s) not the whole processor.
         * -> use mappings just like connect_and_run
         */
-
-       // TODO: atomically copy maps & _no_inplace
-       const ChanMapping in_map (no_sc_input_map ());
-       const ChanMapping out_map (output_map ());
        if (_mapping_changed) {
                _no_inplace = check_inplace ();
                _mapping_changed = false;
        }
+       // TODO: atomically copy maps & _no_inplace
+       ChanMapping const& in_map (no_sc_input_map ());
+       ChanMapping const& out_map (output_map ());
 
        bufs.set_count(ChanCount::max(bufs.count(), _configured_internal));
        bufs.set_count(ChanCount::max(bufs.count(), _configured_out));
@@ -1207,8 +1197,8 @@ PluginInsert::silence (samplecnt_t nframes, samplepos_t start_sample)
 
        _delaybuffers.flush ();
 
-       ChanMapping in_map (natural_input_streams ());
-       ChanMapping out_map (natural_output_streams ());
+       const ChanMapping in_map (natural_input_streams ());
+       const ChanMapping out_map (natural_output_streams ());
        ChanCount maxbuf = ChanCount::max (natural_input_streams (), natural_output_streams());
 #ifdef MIXBUS
        if (is_channelstrip ()) {
@@ -1655,7 +1645,7 @@ PluginInsert::check_inplace ()
                 *
                 * but allows     in-port 1 -> sink-pin 2  ||  source-pin 2 -> out port 1
                 */
-               ChanMapping in_map (input_map ());
+               ChanMapping const& in_map (input_map ());
                const ChanMapping::Mappings out_m (output_map ().mappings ());
                for (ChanMapping::Mappings::const_iterator t = out_m.begin (); t != out_m.end () && inplace_ok; ++t) {
                        for (ChanMapping::TypeMapping::const_iterator c = (*t).second.begin (); c != (*t).second.end () ; ++c) {
@@ -2073,15 +2063,20 @@ PluginInsert::configure_io (ChanCount in, ChanCount out)
        _delaybuffers.configure (_configured_out, _plugins.front ()->max_latency ());
        _latency_changed = true;
 
-       // 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
-       // analysis window
-       session().ensure_buffer_set (_signal_analysis_inputs, in);
-       _signal_analysis_inputs.set_count (in);
+       /* 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
+        * analysis window. We also only analyze audio, so we can ignore
+        * MIDI buffers.
+        */
+       ChanCount cc_analysis_in (DataType::AUDIO, in.n_audio());
+       ChanCount cc_analysis_out (DataType::AUDIO, out.n_audio());
+
+       session().ensure_buffer_set (_signal_analysis_inputs, cc_analysis_in);
+       _signal_analysis_inputs.set_count (cc_analysis_in);
 
-       session().ensure_buffer_set (_signal_analysis_outputs, out);
-       _signal_analysis_outputs.set_count (out);
+       session().ensure_buffer_set (_signal_analysis_outputs, cc_analysis_out);
+       _signal_analysis_outputs.set_count (cc_analysis_out);
 
        // std::cerr << "set counts to i" << in.n_audio() << "/o" << out.n_audio() << std::endl;
 
@@ -2763,10 +2758,16 @@ PluginInsert::set_state(const XMLNode& node, int version)
                // sidechain is a Processor (IO)
                if ((*i)->name () ==  Processor::state_node_name) {
                        if (!_sidechain) {
-                               add_sidechain (0);
+                               if (regenerate_xml_or_string_ids ()) {
+                                       add_sidechain_from_xml (**i, version);
+                               } else {
+                                       add_sidechain (0);
+                               }
                        }
                        if (!regenerate_xml_or_string_ids ()) {
                                _sidechain->set_state (**i, version);
+                       } else {
+                               update_sidechain_name ();
                        }
                }
        }
@@ -3096,6 +3097,42 @@ PluginInsert::add_plugin (boost::shared_ptr<Plugin> plugin)
        _plugins.push_back (plugin);
 }
 
+void
+PluginInsert::add_sidechain_from_xml (const XMLNode& node, int version)
+{
+       if (version < 3000) {
+               return;
+       }
+
+       XMLNodeList nlist = node.children();
+
+       if (nlist.size() == 0) {
+               return;
+       }
+
+       uint32_t audio = 0;
+       uint32_t midi = 0;
+
+       XMLNodeConstIterator it = nlist.front()->children().begin();
+       for ( ; it != nlist.front()->children().end(); ++ it) {
+               if ((*it)->name() == "Port") {
+                       DataType type(DataType::NIL);
+                       (*it)->get_property ("type", type);
+                       if (type == DataType::AUDIO) {
+                               ++audio;
+                       } else if (type == DataType::MIDI) {
+                               ++midi;
+                       }
+               }
+       }
+
+       ChanCount in_cc = ChanCount();
+       in_cc.set (DataType::AUDIO, audio);
+       in_cc.set (DataType::MIDI, midi);
+
+       add_sidechain (audio, midi);
+}
+
 bool
 PluginInsert::load_preset (ARDOUR::Plugin::PresetRecord pr)
 {