switching to "touch" implies existing automation.
authorRobin Gareus <robin@gareus.org>
Mon, 27 Jun 2016 17:04:38 +0000 (19:04 +0200)
committerRobin Gareus <robin@gareus.org>
Mon, 27 Jun 2016 17:04:38 +0000 (19:04 +0200)
This fixes an issue of Fader jumping to unity when switching from
manual to touch if there is no prior automation.
Likewise for plugin parameters jumping to "default"

libs/ardour/automation_control.cc

index 2be0538229e89ca92754d4d455d9a086634996d8..e0ebc00062e1b3dc751e9df7f796956fc0185eb8 100644 (file)
@@ -144,6 +144,8 @@ AutomationControl::set_automation_state (AutoState as)
        }
        if (_list && as != alist()->automation_state()) {
 
+               const double val = get_value ();
+
                alist()->set_automation_state (as);
                if (_desc.toggled) {
                        return;  // No watch for boolean automation
@@ -152,6 +154,10 @@ AutomationControl::set_automation_state (AutoState as)
                if (as == Write) {
                        AutomationWatch::instance().add_automation_watch (shared_from_this());
                } else if (as == Touch) {
+                       if (alist()->empty()) {
+                               Control::set_double (val, _session.transport_frame(), true);
+                               Changed (true, Controllable::NoGroup);
+                       }
                        if (!touching()) {
                                AutomationWatch::instance().remove_automation_watch (shared_from_this());
                        } else {