- bool to_list = _list && boost::dynamic_pointer_cast<AutomationList>(_list)->automation_write();
- const double old_value = Control::user_double ();
+ 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
+ to return a scalar value that in some way reflects the state of the
+ control (with semantics defined by the control itself, since it's
+ internal state may be more complex than can be fully represented by
+ a single scalar).
+
+ This method's only job is to set the "user_double()" value of the
+ underlying Evoral::Control object, and so we should compare the new
+ value we're being given to the current user_double().
+
+ Unless ... we're doing automation playback, in which case the
+ current effective value of the control (used to determine if
+ anything has changed) is the one derived from the automation event
+ list.
+ */
+ float old_value = Control::user_double();