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, 8 insertions, 0 deletions

diff --git a/libs/ardour/amp.cc b/libs/ardour/amp.cc
index c62bb7ff69..8b55b8ca8d 100644
--- a/libs/ardour/amp.cc
+++ b/libs/ardour/amp.cc
@@ -403,6 +403,14 @@ Amp::set_state (const XMLNode& node, int version)
 void
 Amp::GainControl::set_value (double val)
 {
+	if (writable()) {
+		set_value_unchecked (val);
+	}
+}
+
+void
+Amp::GainControl::set_value_unchecked (double val)
+{
 	AutomationControl::set_value (std::max (std::min (val, (double)_desc.upper), (double)_desc.lower));
 	_amp->session().set_dirty ();
 }