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

1 files changed, 8 insertions, 4 deletions

diff --git a/libs/ardour/solo_isolate_control.cc b/libs/ardour/solo_isolate_control.cc
index 7964f7eb66..ebd046fba6 100644
--- a/libs/ardour/solo_isolate_control.cc
+++ b/libs/ardour/solo_isolate_control.cc
@@ -88,11 +88,11 @@ SoloIsolateControl::mod_solo_isolated_by_upstream (int32_t delta)
 	}
 }
 
-void
+bool
 SoloIsolateControl::actually_set_value (double val, PBD::Controllable::GroupControlDisposition gcd)
 {
 	if (!_soloable.can_solo()) {
-		return;
+		return false;
 	}
 
 	set_solo_isolated (val, gcd);
@@ -101,8 +101,12 @@ SoloIsolateControl::actually_set_value (double val, PBD::Controllable::GroupCont
 	   be retrieved by AutomationControl::get_value (), and emits Changed
 	*/
 
-	AutomationControl::actually_set_value (val, gcd);
-	_session.set_dirty ();
+	if (AutomationControl::actually_set_value (val, gcd)) {
+		_session.set_dirty ();
+		return true;
+	}
+
+	return false;
 }
 
 void