From a448b5ea1268b55fd1870384eed163946cbf0495 Mon Sep 17 00:00:00 2001 From: Paul Davis Date: Sun, 22 May 2016 09:47:47 -0400 Subject: no-op: explanatory comments --- libs/ardour/slavable_automation_control.cc | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) (limited to 'libs/ardour/slavable_automation_control.cc') diff --git a/libs/ardour/slavable_automation_control.cc b/libs/ardour/slavable_automation_control.cc index 71769ad155..4da324515a 100644 --- a/libs/ardour/slavable_automation_control.cc +++ b/libs/ardour/slavable_automation_control.cc @@ -141,8 +141,16 @@ SlavableAutomationControl::add_master (boost::shared_ptr m) m->DropReferences.connect_same_thread (masters_connections, boost::bind (&SlavableAutomationControl::master_going_away, this, boost::weak_ptr(m))); - /* Store the connection inside the MasterRecord, so that when we destroy it, the connection is destroyed - and we no longer hear about changes to the AutomationControl. + /* Store the connection inside the MasterRecord, so + that when we destroy it, the connection is destroyed + and we no longer hear about changes to the + AutomationControl. + + Note that this also makes it safe to store a + boost::shared_ptr in the functor, + since we know we will destroy the functor when the + connection is destroyed, which happens when we + disconnect from the master (for any reason). Note that we fix the "from_self" argument that will be given to our own Changed signal to "false", -- cgit v1.2.3