keep port-matrix in sync when deleting ports. and fix segfault on session close with...
authorRobin Gareus <robin@gareus.org>
Fri, 2 Aug 2013 15:40:58 +0000 (17:40 +0200)
committerRobin Gareus <robin@gareus.org>
Fri, 2 Aug 2013 15:41:48 +0000 (17:41 +0200)
gtk2_ardour/port_matrix.cc

index bac415d72f63760b06973c50f8f9b39bb8f3482f..0df5d2214d7f5b024ea7065cfb6bb25c7d62e2dc 100644 (file)
@@ -152,6 +152,7 @@ PortMatrix::init ()
 
        /* and also bundles */
        _session->BundleAdded.connect (_session_connections, invalidator (*this), boost::bind (&PortMatrix::setup_global_ports, this), gui_context());
+       _session->BundleRemoved.connect (_session_connections, invalidator (*this), boost::bind (&PortMatrix::setup_global_ports, this), gui_context());
 
        /* and also ports */
        _session->engine().PortRegisteredOrUnregistered.connect (_session_connections, invalidator (*this), boost::bind (&PortMatrix::setup_global_ports, this), gui_context());
@@ -180,6 +181,7 @@ PortMatrix::reconnect_to_routes ()
        boost::shared_ptr<RouteList> routes = _session->get_routes ();
        for (RouteList::iterator i = routes->begin(); i != routes->end(); ++i) {
                (*i)->processors_changed.connect (_route_connections, invalidator (*this), boost::bind (&PortMatrix::route_processors_changed, this, _1), gui_context());
+               (*i)->DropReferences.connect (_route_connections, invalidator (*this), boost::bind (&PortMatrix::routes_changed, this), gui_context());
        }
 }
 
@@ -198,6 +200,7 @@ PortMatrix::route_processors_changed (RouteProcessorChange c)
 void
 PortMatrix::routes_changed ()
 {
+       if (!_session) return;
        reconnect_to_routes ();
        setup_global_ports ();
 }
@@ -206,7 +209,10 @@ PortMatrix::routes_changed ()
 void
 PortMatrix::setup ()
 {
-       if (!_session) return; // session went away
+       if (!_session) {
+               _route_connections.drop_connections ();
+               return; // session went away
+       }
 
        /* this needs to be done first, as the visible_ports() method uses the
           notebook state to decide which ports are being shown */