From dee990103a1fb607ff401ec7c56e8c241074ffb5 Mon Sep 17 00:00:00 2001 From: Robin Gareus Date: Sat, 15 Jul 2017 20:45:49 +0200 Subject: Consistent Automation evaluation: Rule #89: The *owner* of each automation-control is responsible to evaluate automation of automated automation-controls (and emit Changed() signals to notify the GUI and slaved controls). This can happen during run(), when the Processor evaluates automation (eg. PluginInsert does that), but needs to regardless, every cycle. Emit Changed signal for GainControl This follow the same concept as PluginInsert: The Changed signal is called on demand when evaluating automation. --- libs/ardour/route.cc | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) (limited to 'libs/ardour/route.cc') diff --git a/libs/ardour/route.cc b/libs/ardour/route.cc index 34c2b2f823..9bd5c49cea 100644 --- a/libs/ardour/route.cc +++ b/libs/ardour/route.cc @@ -322,7 +322,7 @@ Route::process_output_buffers (BufferSet& bufs, return; } - _mute_control->automation_run (start_frame, nframes); + automation_run (start_frame, nframes); /* figure out if we're going to use gain automation */ if (gain_automation_ok) { @@ -2980,11 +2980,16 @@ Route::silence_unlocked (framecnt_t nframes) _output->silence (nframes); + // update owned automated controllables + automation_run (now, nframes); + for (ProcessorList::iterator i = _processors.begin(); i != _processors.end(); ++i) { boost::shared_ptr pi; if (!_active && (pi = boost::dynamic_pointer_cast (*i)) != 0) { - // skip plugins, they don't need anything when we're not active + /* evaluate automated automation controls */ + pi->automation_run (now, nframes); + /* skip plugins, they don't need anything when we're not active */ continue; } -- cgit v1.2.3