Initialize uninitialized variable
[ardour.git] / libs / ardour / automation_control.cc
index 274e84f9f072f2848d08dbd054d4e5416e84bf8a..b8d5226b297d4ee286d88141ccb91fcb188f55a4 100644 (file)
@@ -1,22 +1,23 @@
 /*
-    Copyright (C) 2007 Paul Davis
-    Author: David Robillard
-
-    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
-    the Free Software Foundation; either version 2 of the License, or
-    (at your option) any later version.
-
-    This program is distributed in the hope that it will be useful,
-    but WITHOUT ANY WARRANTY; without even the implied warranty of
-    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-    GNU General Public License for more details.
-
-    You should have received a copy of the GNU General Public License
-    along with this program; if not, write to the Free Software
-    Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
-
-*/
+ * Copyright (C) 2007-2014 David Robillard <d@drobilla.net>
+ * Copyright (C) 2008-2017 Paul Davis <paul@linuxaudiosystems.com>
+ * Copyright (C) 2014-2019 Robin Gareus <robin@gareus.org>
+ * Copyright (C) 2015 Nick Mainsbridge <mainsbridge@gmail.com>
+ *
+ * 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
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License along
+ * with this program; if not, write to the Free Software Foundation, Inc.,
+ * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
+ */
 
 #include <math.h>
 #include <iostream>
@@ -56,8 +57,9 @@ AutomationControl::AutomationControl(ARDOUR::Session&                          s
 
        : Controllable (name.empty() ? EventTypeMap::instance().to_symbol(parameter) : name, flags)
        , Evoral::Control(parameter, desc, list)
-       , _session(session)
+       , SessionHandleRef (session)
        , _desc(desc)
+       , _no_session(false)
 {
        if (_desc.toggled) {
                set_flags (Controllable::Toggle);
@@ -70,8 +72,18 @@ AutomationControl::AutomationControl(ARDOUR::Session&                          s
 
 AutomationControl::~AutomationControl ()
 {
-       _session.selection().remove_control_by_id (id());
+       if (!_no_session && !_session.deletion_in_progress ()) {
+               _session.selection().remove_control_by_id (id());
+               DropReferences (); /* EMIT SIGNAL */
+       }
+}
+
+void
+AutomationControl::session_going_away ()
+{
+       SessionHandleRef::session_going_away ();
        DropReferences (); /* EMIT SIGNAL */
+       _no_session = true;
 }
 
 bool
@@ -88,8 +100,8 @@ AutomationControl::writable() const
 double
 AutomationControl::get_value() const
 {
-       bool from_list = _list && boost::dynamic_pointer_cast<AutomationList>(_list)->automation_playback();
-       return Control::get_double (from_list, _session.transport_frame());
+       bool from_list = alist() && alist()->automation_playback();
+       return Control::get_double (from_list, _session.transport_sample());
 }
 
 double
@@ -130,7 +142,7 @@ AutomationControl::set_value (double val, PBD::Controllable::GroupControlDisposi
        }
 
        if (_group && _group->use_me (gcd)) {
-               _group->set_group_value (shared_from_this(), val);
+               _group->set_group_value (boost::dynamic_pointer_cast<AutomationControl>(shared_from_this()), val);
        } else {
                actually_set_value (val, gcd);
        }
@@ -147,7 +159,7 @@ AutomationControl::grouped_controls () const
 }
 
 void
-AutomationControl::automation_run (framepos_t start, pframes_t nframes)
+AutomationControl::automation_run (samplepos_t start, pframes_t nframes)
 {
        if (!automation_playback ()) {
                return;
@@ -174,8 +186,8 @@ AutomationControl::automation_run (framepos_t start, pframes_t nframes)
 void
 AutomationControl::actually_set_value (double value, PBD::Controllable::GroupControlDisposition gcd)
 {
-       boost::shared_ptr<AutomationList> al = boost::dynamic_pointer_cast<AutomationList> (_list);
-       const framepos_t pos = _session.transport_frame();
+       boost::shared_ptr<AutomationList> al = alist ();
+       const samplepos_t pos = _session.transport_sample();
        bool to_list;
 
        /* We cannot use ::get_value() here since that is virtual, and intended
@@ -230,7 +242,7 @@ AutomationControl::set_automation_state (AutoState as)
        if (flags() & NotAutomatable) {
                return;
        }
-       if (_list && as != alist()->automation_state()) {
+       if (alist() && as != alist()->automation_state()) {
 
                const double val = get_value ();
 
@@ -241,67 +253,70 @@ AutomationControl::set_automation_state (AutoState as)
                }
 
                if (as == Write) {
-                       AutomationWatch::instance().add_automation_watch (shared_from_this());
-               } else if (as == Touch) {
+                       AutomationWatch::instance().add_automation_watch (boost::dynamic_pointer_cast<AutomationControl>(shared_from_this()));
+               } else if (as & (Touch | Latch)) {
                        if (alist()->empty()) {
-                               Control::set_double (val, _session.current_start_frame (), true);
-                               Control::set_double (val, _session.current_end_frame (), true);
+                               Control::set_double (val, _session.current_start_sample (), true);
+                               Control::set_double (val, _session.current_end_sample (), true);
                                Changed (true, Controllable::NoGroup);
                        }
                        if (!touching()) {
-                               AutomationWatch::instance().remove_automation_watch (shared_from_this());
+                               AutomationWatch::instance().remove_automation_watch (boost::dynamic_pointer_cast<AutomationControl>(shared_from_this()));
                        } else {
                                /* this seems unlikely, but the combination of
                                 * a control surface and the mouse could make
                                 * it possible to put the control into Touch
                                 * mode *while* touching it.
                                 */
-                               AutomationWatch::instance().add_automation_watch (shared_from_this());
+                               AutomationWatch::instance().add_automation_watch (boost::dynamic_pointer_cast<AutomationControl>(shared_from_this()));
                        }
                } else {
-                       AutomationWatch::instance().remove_automation_watch (shared_from_this());
+                       AutomationWatch::instance().remove_automation_watch (boost::dynamic_pointer_cast<AutomationControl>(shared_from_this()));
                        Changed (false, Controllable::NoGroup);
                }
        }
 }
 
 void
-AutomationControl::start_touch(double when)
+AutomationControl::start_touch (double when)
 {
-       if (!_list) {
+       if (!_list || touching ()) {
                return;
        }
 
-       if (!touching()) {
-               if (alist()->automation_state() == Touch) {
-                       /* subtle. aligns the user value with the playback and
-                        * use take actual value (incl masters).
-                        *
-                        * Touch + hold writes inverse curve of master-automation
-                        * using AutomationWatch::timer ()
-                        */
-                       AutomationControl::actually_set_value (get_value (), Controllable::NoGroup);
-                       alist()->start_touch (when);
-                       if (!_desc.toggled) {
-                               AutomationWatch::instance().add_automation_watch (shared_from_this());
-                       }
+       if (alist()->automation_state() & (Touch | Latch)) {
+               /* subtle. aligns the user value with the playback and
+                * use take actual value (incl masters).
+                *
+                * Touch + hold writes inverse curve of master-automation
+                * using AutomationWatch::timer ()
+                */
+               AutomationControl::actually_set_value (get_value (), Controllable::NoGroup);
+               alist()->start_touch (when);
+               if (!_desc.toggled) {
+                       AutomationWatch::instance().add_automation_watch (boost::dynamic_pointer_cast<AutomationControl>(shared_from_this()));
                }
                set_touching (true);
        }
 }
 
 void
-AutomationControl::stop_touch(double when)
+AutomationControl::stop_touch (double when)
 {
-       if (!_list) return;
-       if (touching()) {
-               set_touching (false);
-
-               if (alist()->automation_state() == Touch) {
-                       alist()->stop_touch (when);
-                       if (!_desc.toggled) {
-                               AutomationWatch::instance().remove_automation_watch (shared_from_this());
-                       }
+       if (!_list || !touching ()) {
+               return;
+       }
+
+       if (alist()->automation_state() == Latch && _session.transport_rolling ()) {
+               return;
+       }
+
+       set_touching (false);
+
+       if (alist()->automation_state() & (Touch | Latch)) {
+               alist()->stop_touch (when);
+               if (!_desc.toggled) {
+                       AutomationWatch::instance().remove_automation_watch (boost::dynamic_pointer_cast<AutomationControl>(shared_from_this()));
                }
        }
 }
@@ -310,9 +325,10 @@ void
 AutomationControl::commit_transaction (bool did_write)
 {
        if (did_write) {
-               if (alist ()->before ()) {
+               XMLNode* before = alist ()->before ();
+               if (before) {
                        _session.begin_reversible_command (string_compose (_("record %1 automation"), name ()));
-                       _session.commit_reversible_command (new MementoCommand<AutomationList> (*alist ().get (), alist ()->before (), &alist ()->get_state ()));
+                       _session.commit_reversible_command (alist ()->memento_command (before, &alist ()->get_state ()));
                }
        } else {
                alist ()->clear_history ();
@@ -363,7 +379,7 @@ AutomationControl::check_rt (double val, Controllable::GroupControlDisposition g
 {
        if (!_session.loading() && (flags() & Controllable::RealTime) && !AudioEngine::instance()->in_process_thread()) {
                /* queue change in RT context */
-               _session.set_control (shared_from_this(), val, gcd);
+               _session.set_control (boost::dynamic_pointer_cast<AutomationControl>(shared_from_this()), val, gcd);
                return true;
        }