Add AutomationControl::set_value_unchecked() and AutomationControl::writable() and use them.

Classes derived from AutomationControl now check ::writable() in their ::set_value() methods to ensure that they
do not attempt to overwrite data sent to them while automation playback is underway.

1 files changed, 16 insertions, 0 deletions

diff --git a/libs/ardour/route.cc b/libs/ardour/route.cc
index 60f4908768..56d1f69ebe 100644
--- a/libs/ardour/route.cc
+++ b/libs/ardour/route.cc
@@ -3848,6 +3848,14 @@ Route::SoloControllable::SoloControllable (std::string name, boost::shared_ptr<R
 void
 Route::SoloControllable::set_value (double val)
 {
+	if (writable()) {
+		set_value_unchecked (val);
+	}
+}
+
+void
+Route::SoloControllable::set_value_unchecked (double val)
+{
 	const bool bval = ((val >= 0.5) ? true : false);
 
 	boost::shared_ptr<RouteList> rl (new RouteList);
@@ -3921,6 +3929,14 @@ Route::MuteControllable::set_superficial_value(bool muted)
 void
 Route::MuteControllable::set_value (double val)
 {
+	if (writable()) {
+		set_value_unchecked (val);
+	}
+}
+
+void
+Route::MuteControllable::set_value_unchecked (double val)
+{
 	const bool bval = ((val >= 0.5) ? true : false);
 
 	boost::shared_ptr<Route> r = _route.lock ();