From c104c9d4726f3ba1ecd352d13b88a57f2f964510 Mon Sep 17 00:00:00 2001 From: Paul Davis Date: Wed, 12 Oct 2016 17:26:46 -0400 Subject: change return type of AutomationControl::actually_set_value() from void to bool, to indicate if value was changed. Don't call Session::set_dirty() when no change occurs --- libs/ardour/pan_controllable.cc | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) (limited to 'libs/ardour/pan_controllable.cc') diff --git a/libs/ardour/pan_controllable.cc b/libs/ardour/pan_controllable.cc index 89842b124b..e4ad360910 100644 --- a/libs/ardour/pan_controllable.cc +++ b/libs/ardour/pan_controllable.cc @@ -34,15 +34,14 @@ PanControllable::lower () const } } -void +bool PanControllable::actually_set_value (double v, Controllable::GroupControlDisposition group_override) { boost::shared_ptr p = owner->panner(); if (!p) { /* no panner: just do it */ - AutomationControl::actually_set_value (v, group_override); - return; + return AutomationControl::actually_set_value (v, group_override); } bool can_set = false; @@ -62,8 +61,10 @@ PanControllable::actually_set_value (double v, Controllable::GroupControlDisposi } if (can_set) { - AutomationControl::actually_set_value (v, group_override); + return AutomationControl::actually_set_value (v, group_override); } + + return false; } std::string -- cgit v1.2.3