Optimize plugin-processing for non-automated params
[ardour.git] / libs / ardour / automation_list.cc
index 40c924d293a2aad23410d59f8688a038f893f773..cb09786c649f448b20ce948c66d0427ef74bbc01 100644 (file)
@@ -24,7 +24,7 @@
 #include <sstream>
 #include <algorithm>
 #include "ardour/automation_list.h"
-#include "ardour/beats_frames_converter.h"
+#include "ardour/beats_samples_converter.h"
 #include "ardour/event_type_map.h"
 #include "ardour/parameter_descriptor.h"
 #include "ardour/parameter_types.h"
@@ -59,8 +59,8 @@ AutomationList::AutomationList (const Evoral::Parameter& id, const Evoral::Param
        , _before (0)
 {
        _state = Off;
-       _style = Absolute;
        g_atomic_int_set (&_touching, 0);
+       _interpolation = default_interpolation ();
 
        create_curve_if_necessary();
 
@@ -73,8 +73,8 @@ AutomationList::AutomationList (const Evoral::Parameter& id)
        , _before (0)
 {
        _state = Off;
-       _style = Absolute;
        g_atomic_int_set (&_touching, 0);
+       _interpolation = default_interpolation ();
 
        create_curve_if_necessary();
 
@@ -87,7 +87,6 @@ AutomationList::AutomationList (const AutomationList& other)
        , StatefulDestructible()
        , _before (0)
 {
-       _style = other._style;
        _state = other._state;
        g_atomic_int_set (&_touching, other.touching());
 
@@ -101,7 +100,6 @@ AutomationList::AutomationList (const AutomationList& other, double start, doubl
        : ControlList(other, start, end)
        , _before (0)
 {
-       _style = other._style;
        _state = other._state;
        g_atomic_int_set (&_touching, other.touching());
 
@@ -119,8 +117,8 @@ AutomationList::AutomationList (const XMLNode& node, Evoral::Parameter id)
        , _before (0)
 {
        g_atomic_int_set (&_touching, 0);
+       _interpolation = default_interpolation ();
        _state = Off;
-       _style = Absolute;
 
        set_state (node, Stateful::loading_state_version);
 
@@ -163,21 +161,22 @@ AutomationList::create_curve_if_necessary()
        default:
                break;
        }
+
+       WritePassStarted.connect_same_thread (_writepass_connection, boost::bind (&AutomationList::snapshot_history, this, false));
 }
 
 AutomationList&
 AutomationList::operator= (const AutomationList& other)
 {
        if (this != &other) {
-
-
+               ControlList::freeze ();
+               /* ControlList::operator= calls copy_events() which calls
+                * mark_dirty() and maybe_signal_changed()
+                */
                ControlList::operator= (other);
                _state = other._state;
-               _style = other._style;
                _touching = other._touching;
-
-               mark_dirty ();
-               maybe_signal_changed ();
+               ControlList::thaw ();
        }
 
        return *this;
@@ -193,39 +192,54 @@ AutomationList::maybe_signal_changed ()
        }
 }
 
+AutoState
+AutomationList::automation_state() const
+{
+       Glib::Threads::RWLock::ReaderLock lm (Evoral::ControlList::_lock);
+       return _state;
+}
+
 void
 AutomationList::set_automation_state (AutoState s)
 {
-       if (s != _state) {
+       {
+               Glib::Threads::RWLock::ReaderLock lm (Evoral::ControlList::_lock);
+
+               if (s == _state) {
+                       return;
+               }
                _state = s;
-               delete _before;
                if (s == Write && _desc.toggled) {
-                       _before = &get_state ();
-               } else {
-                       _before = 0;
+                       snapshot_history (true);
                }
-               automation_state_changed (s); /* EMIT SIGNAL */
        }
+
+       automation_state_changed (s); /* EMIT SIGNAL */
 }
 
-void
-AutomationList::set_automation_style (AutoStyle s)
+Evoral::ControlList::InterpolationStyle
+AutomationList::default_interpolation () const
 {
-       if (s != _style) {
-               _style = s;
-               automation_style_changed (); /* EMIT SIGNAL */
+       switch (_parameter.type()) {
+               case GainAutomation:
+               case BusSendLevel:
+               case EnvelopeAutomation:
+                       return ControlList::Exponential;
+                       break;
+               case TrimAutomation:
+                       return ControlList::Logarithmic;
+                       break;
+               default:
+                       break;
        }
+       /* based on Evoral::ParameterDescriptor log,toggle,.. */
+       return ControlList::default_interpolation ();
 }
 
 void
 AutomationList::start_write_pass (double when)
 {
-       delete _before;
-       if (in_new_write_pass ()) {
-               _before = &get_state ();
-       } else {
-               _before = 0;
-       }
+       snapshot_history (true);
        ControlList::start_write_pass (when);
 }
 
@@ -238,15 +252,15 @@ AutomationList::write_pass_finished (double when, double thinning_factor)
 void
 AutomationList::start_touch (double when)
 {
-        if (_state == Touch) {
+       if (_state == Touch) {
                start_write_pass (when);
-        }
+       }
 
        g_atomic_int_set (&_touching, 1);
 }
 
 void
-AutomationList::stop_touch (bool mark, double)
+AutomationList::stop_touch (double)
 {
        if (g_atomic_int_get (&_touching) == 0) {
                /* this touch has already been stopped (probably by Automatable::transport_stopped),
@@ -256,16 +270,6 @@ AutomationList::stop_touch (bool mark, double)
        }
 
        g_atomic_int_set (&_touching, 0);
-
-        if (_state == Touch) {
-
-                if (mark) {
-
-                       /* XXX need to mark the last added point with the
-                        * current time
-                        */
-                }
-        }
 }
 
 /* _before may be owned by the undo stack,
@@ -280,6 +284,17 @@ AutomationList::clear_history ()
        _before = 0;
 }
 
+void
+AutomationList::snapshot_history (bool need_lock)
+{
+       if (!in_new_write_pass ()) {
+               return;
+       }
+       delete _before;
+       _before = &state (true, need_lock);
+}
+
+
 void
 AutomationList::thaw ()
 {
@@ -292,7 +307,7 @@ AutomationList::thaw ()
 }
 
 bool
-AutomationList::paste (const ControlList& alist, double pos, DoubleBeatsFramesConverter const& bfc)
+AutomationList::paste (const ControlList& alist, double pos, DoubleBeatsSamplesConverter const& bfc)
 {
        AutomationType src_type = (AutomationType)alist.parameter().type();
        AutomationType dst_type = (AutomationType)_parameter.type();
@@ -300,13 +315,13 @@ AutomationList::paste (const ControlList& alist, double pos, DoubleBeatsFramesCo
        if (parameter_is_midi (src_type) == parameter_is_midi (dst_type)) {
                return ControlList::paste (alist, pos);
        }
-       bool to_frame = parameter_is_midi (src_type);
+       bool to_sample = parameter_is_midi (src_type);
 
        ControlList cl (alist);
        cl.clear ();
        for (const_iterator i = alist.begin ();i != alist.end (); ++i) {
                double when = (*i)->when;
-               if (to_frame) {
+               if (to_sample) {
                        when = bfc.to ((*i)->when);
                } else {
                        when = bfc.from ((*i)->when);
@@ -325,23 +340,19 @@ AutomationList::memento_command (XMLNode* before, XMLNode* after)
 XMLNode&
 AutomationList::get_state ()
 {
-       return state (true);
+       return state (true, true);
 }
 
 XMLNode&
-AutomationList::state (bool full)
+AutomationList::state (bool save_auto_state, bool need_lock)
 {
        XMLNode* root = new XMLNode (X_("AutomationList"));
-       LocaleGuard lg;
 
        root->set_property ("automation-id", EventTypeMap::instance().to_symbol(_parameter));
        root->set_property ("id", id());
-       root->set_property ("default", _default_value);
-       root->set_property ("min-yval", _min_yval);
-       root->set_property ("max-yval", _max_yval);
        root->set_property ("interpolation-style", _interpolation);
 
-       if (full) {
+       if (save_auto_state) {
                /* never serialize state with Write enabled - too dangerous
                   for the user's data
                */
@@ -359,21 +370,23 @@ AutomationList::state (bool full)
                root->set_property ("state", Off);
        }
 
-       root->set_property ("style", _style);
-
        if (!_events.empty()) {
-               root->add_child_nocopy (serialize_events());
+               root->add_child_nocopy (serialize_events (need_lock));
        }
 
        return *root;
 }
 
 XMLNode&
-AutomationList::serialize_events ()
+AutomationList::serialize_events (bool need_lock)
 {
        XMLNode* node = new XMLNode (X_("events"));
        stringstream str;
 
+       Glib::Threads::RWLock::ReaderLock lm (Evoral::ControlList::_lock, Glib::Threads::NOT_LOCK);
+       if (need_lock) {
+               lm.acquire ();
+       }
        for (iterator xx = _events.begin(); xx != _events.end(); ++xx) {
                str << PBD::to_string ((*xx)->when);
                str << ' ';
@@ -425,6 +438,7 @@ AutomationList::deserialize_events (const XMLNode& node)
                        ok = false;
                        break;
                }
+               y = std::min ((double)_desc.upper, std::max ((double)_desc.lower, y));
                fast_simple_add (x, y);
        }
 
@@ -444,7 +458,6 @@ AutomationList::deserialize_events (const XMLNode& node)
 int
 AutomationList::set_state (const XMLNode& node, int version)
 {
-       LocaleGuard lg;
        XMLNodeList nlist = node.children();
        XMLNode* nsos;
        XMLNodeIterator niter;
@@ -483,6 +496,7 @@ AutomationList::set_state (const XMLNode& node, int version)
                                continue;
                        }
 
+                       y = std::min ((double)_desc.upper, std::max ((double)_desc.lower, y));
                        fast_simple_add (x, y);
                }
 
@@ -509,15 +523,7 @@ AutomationList::set_state (const XMLNode& node, int version)
        }
 
        if (!node.get_property (X_("interpolation-style"), _interpolation)) {
-               _interpolation = Linear;
-       }
-
-       if (!node.get_property (X_("default"), _default_value)) {
-               _default_value = 0.0;
-       }
-
-       if (!node.get_property (X_("style"), _style)) {
-               _style = Absolute;
+               _interpolation = default_interpolation ();
        }
 
        if (node.get_property (X_("state"), _state)) {
@@ -529,14 +535,6 @@ AutomationList::set_state (const XMLNode& node, int version)
                _state = Off;
        }
 
-       if (!node.get_property (X_("min-yval"), _min_yval)) {
-               _min_yval = FLT_MIN;
-       }
-
-       if (!node.get_property (X_("max-yval"), _max_yval)) {
-               _max_yval = FLT_MAX;
-       }
-
        bool have_events = false;
 
        for (niter = nlist.begin(); niter != nlist.end(); ++niter) {
@@ -564,7 +562,6 @@ AutomationList::operator!= (AutomationList const & other) const
        return (
                static_cast<ControlList const &> (*this) != static_cast<ControlList const &> (other) ||
                _state != other._state ||
-               _style != other._style ||
                _touching != other._touching
                );
 }
@@ -578,4 +575,3 @@ AutomationListProperty::clone () const
                boost::shared_ptr<AutomationList> (new AutomationList (*this->_current.get()))
                );
 }
-