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/solo_control.cc | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'libs/ardour/solo_control.cc') diff --git a/libs/ardour/solo_control.cc b/libs/ardour/solo_control.cc index 087e5d04ab..ca41ee7aac 100644 --- a/libs/ardour/solo_control.cc +++ b/libs/ardour/solo_control.cc @@ -157,11 +157,11 @@ SoloControl::mod_solo_by_others_upstream (int32_t delta) Changed (false, Controllable::NoGroup); /* EMIT SIGNAL */ } -void +bool SoloControl::actually_set_value (double val, PBD::Controllable::GroupControlDisposition group_override) { if (_soloable.is_safe() || !_soloable.can_solo()) { - return; + return false; } set_self_solo (val == 1.0); @@ -170,7 +170,7 @@ SoloControl::actually_set_value (double val, PBD::Controllable::GroupControlDisp be retrieved by AutomationControl::get_value (), and emits Changed */ - SlavableAutomationControl::actually_set_value (val, group_override); + return SlavableAutomationControl::actually_set_value (val, group_override); } double -- cgit v1.2.3