PBD::Controllable::Flag flags=PBD::Controllable::Flag (0)
);
- ~AutomationControl ();
+ virtual ~AutomationControl ();
boost::shared_ptr<AutomationList> alist() const {
return boost::dynamic_pointer_cast<AutomationList>(_list);
PBD::Controllable::Flag flags=PBD::Controllable::Flag (0)
);
- ~SlavableAutomationControl ();
+ virtual ~SlavableAutomationControl ();
double get_value () const;
mutable Glib::Threads::RWLock master_lock;
typedef std::map<PBD::ID,MasterRecord> Masters;
Masters _masters;
- PBD::ScopedConnectionList masters_connections;
+ std::map<boost::weak_ptr<AutomationControl>, PBD::ScopedConnection> masters_connections;
void master_going_away (boost::weak_ptr<AutomationControl>);
double get_value_locked() const;
avoiding holding a reference to the control in the binding
itself.
*/
-
- m->DropReferences.connect_same_thread (masters_connections, boost::bind (&SlavableAutomationControl::master_going_away, this, boost::weak_ptr<AutomationControl>(m)));
+ assert (masters_connections.find (boost::weak_ptr<AutomationControl>(m)) == masters_connections.end());
+ PBD::ScopedConnection con;
+ m->DropReferences.connect_same_thread (con, boost::bind (&SlavableAutomationControl::master_going_away, this, boost::weak_ptr<AutomationControl>(m)));
+ masters_connections[boost::weak_ptr<AutomationControl>(m)] = con;
/* Store the connection inside the MasterRecord, so
that when we destroy it, the connection is destroyed
void
SlavableAutomationControl::remove_master (boost::shared_ptr<AutomationControl> m)
{
+ masters_connections.erase (boost::weak_ptr<AutomationControl>(m));
pre_remove_master (m);
{
VCA::~VCA ()
{
DEBUG_TRACE (DEBUG::Destruction, string_compose ("delete VCA %1\n", number()));
+ {
+ Glib::Threads::Mutex::Lock lm (_control_lock);
+ for (Controls::const_iterator li = _controls.begin(); li != _controls.end(); ++li) {
+ boost::dynamic_pointer_cast<AutomationControl>(li->second)->drop_references ();
+ }
+ }
{
Glib::Threads::Mutex::Lock lm (number_lock);
if (_number == next_number - 1) {