X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=libs%2Fardour%2Fvca.cc;h=36ed09cf258d088f8a8ecf8b5847163d2c70286d;hb=e845b9f9357c4ff471c02b1f63a61275bb4a7d28;hp=f746da8ff38f297d06ebfd582b419494a3422a80;hpb=69250b64ea00671ab05d852833006e483a53cd8b;p=ardour.git

diff --git a/libs/ardour/vca.cc b/libs/ardour/vca.cc
index f746da8ff3..36ed09cf25 100644
--- a/libs/ardour/vca.cc
+++ b/libs/ardour/vca.cc
@@ -19,7 +19,9 @@
 #include "pbd/convert.h"
 
 #include "ardour/automation_control.h"
+#include "ardour/debug.h"
 #include "ardour/gain_control.h"
+#include "ardour/monitor_control.h"
 #include "ardour/rc_configuration.h"
 #include "ardour/route.h"
 #include "ardour/session.h"
@@ -31,7 +33,8 @@ using namespace ARDOUR;
 using namespace PBD;
 using std::string;
 
-gint VCA::next_number = 1;
+Glib::Threads::Mutex VCA::number_lock;
+uint32_t VCA::next_number = 1;
 string VCA::xml_node_name (X_("VCA"));
 
 string
@@ -40,29 +43,32 @@ VCA::default_name_template ()
 	return _("VCA %n");
 }
 
-int
+uint32_t
 VCA::next_vca_number ()
 {
-	/* recall that atomic_int_add() returns the value before the add. We
-	 * start at one, then next one will be two etc.
-	 */
-	return g_atomic_int_add (&next_number, 1);
+	/* we could use atomic inc here, but elsewhere we need more complete
+	   mutex semantics, so we have to do it here also.
+	*/
+	Glib::Threads::Mutex::Lock lm (number_lock);
+	return next_number++;
 }
 
 void
 VCA::set_next_vca_number (uint32_t n)
 {
-	g_atomic_int_set (&next_number, n);
+	Glib::Threads::Mutex::Lock lm (number_lock);
+	next_number = n;
 }
 
 uint32_t
 VCA::get_next_vca_number ()
 {
-	return g_atomic_int_get (&next_number);
+	Glib::Threads::Mutex::Lock lm (number_lock);
+	return next_number;
 }
 
 VCA::VCA (Session& s,  uint32_t num, const string& name)
-	: Stripable (s, name)
+	: Stripable (s, name, PresentationInfo (num, PresentationInfo::VCA))
 	, Muteable (s, name)
 	, Automatable (s)
 	, _number (num)
@@ -85,13 +91,16 @@ VCA::init ()
 
 VCA::~VCA ()
 {
-	DropReferences (); /* emit signal */
-}
-
-uint32_t
-VCA::remote_control_id () const
-{
-	return 9999999 + _number;
+	DEBUG_TRACE (DEBUG::Destruction, string_compose ("delete VCA %1\n", number()));
+	{
+		Glib::Threads::Mutex::Lock lm (number_lock);
+		if (_number == next_number - 1) {
+			/* this was the "last" VCA added, so rewind the next number so
+			 * that future VCAs get numbered as intended
+			 */
+			next_number--;
+		}
+	}
 }
 
 XMLNode&
@@ -101,11 +110,15 @@ VCA::get_state ()
 	node->add_property (X_("name"), _name);
 	node->add_property (X_("number"), _number);
 
+	node->add_child_nocopy (_presentation_info.get_state());
+
 	node->add_child_nocopy (_gain_control->get_state());
 	node->add_child_nocopy (_solo_control->get_state());
 	node->add_child_nocopy (_mute_control->get_state());
 	node->add_child_nocopy (get_automation_xml_state());
 
+	node->add_child_nocopy (Slavable::get_state());
+
 	return *node;
 }
 
@@ -114,6 +127,8 @@ VCA::set_state (XMLNode const& node, int version)
 {
 	XMLProperty const* prop;
 
+	Stripable::set_state (node, version);
+
 	if ((prop = node.property ("name")) != 0) {
 		set_name (prop->value());
 	}
@@ -141,8 +156,45 @@ VCA::set_state (XMLNode const& node, int version)
 			if (prop->value() == _mute_control->name()) {
 				_mute_control->set_state (**i, version);
 			}
+		} else if ((*i)->name() == Slavable::xml_node_name) {
+			Slavable::set_state (**i, version);
 		}
 	}
 
 	return 0;
 }
+
+void
+VCA::clear_all_solo_state ()
+{
+	_solo_control->clear_all_solo_state ();
+}
+
+MonitorState
+VCA::monitoring_state () const
+{
+	/* XXX this has to get more complex but not clear how */
+	return MonitoringInput;
+}
+
+bool
+VCA::slaved () const
+{
+	if (!_gain_control) {
+		return false;
+	}
+	/* just test one particular control, not all of them */
+	return _gain_control->slaved ();
+}
+
+bool
+VCA::slaved_to (boost::shared_ptr<VCA> vca) const
+{
+	if (!vca || !_gain_control) {
+		return false;
+	}
+
+	/* just test one particular control, not all of them */
+
+	return _gain_control->slaved_to (vca->gain_control());
+}