pay attention to MIDI channel selector when creating notes with GUI (fixes #3233)
[ardour.git] / gtk2_ardour / port_group.cc
index 4bf565a3e9c0de84dab86ebe60cb86fd5fe9999b..a010b3c184b4be91743ca6820d020e178142c003 100644 (file)
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2002-2009 Paul Davis 
+    Copyright (C) 2002-2009 Paul Davis
 
     This program is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
 
 */
 
-#include "port_group.h"
-#include "port_matrix.h"
-#include "i18n.h"
-#include "ardour/session.h"
+#include <cstring>
+#include <boost/shared_ptr.hpp>
+
 #include "ardour/audio_track.h"
-#include "ardour/midi_track.h"
 #include "ardour/audioengine.h"
-#include "ardour/port.h"
 #include "ardour/bundle.h"
-#include <boost/shared_ptr.hpp>
-#include <cstring>
+#include "ardour/user_bundle.h"
+#include "ardour/io_processor.h"
+#include "ardour/midi_track.h"
+#include "ardour/port.h"
+#include "ardour/session.h"
+#include "ardour/auditioner.h"
+
+#include "gui_thread.h"
+#include "port_group.h"
+#include "port_matrix.h"
+#include "time_axis_view.h"
+#include "public_editor.h"
+
+#include "i18n.h"
 
 using namespace std;
 using namespace Gtk;
+using namespace ARDOUR;
+
+/** PortGroup constructor.
+ * @param n Name.
+ */
+PortGroup::PortGroup (std::string const & n)
+       : name (n)
+{
+
+}
+
+PortGroup::~PortGroup()
+{
+       for (BundleList::iterator i = _bundles.begin(); i != _bundles.end(); ++i) {
+               delete *i;
+       }
+       _bundles.clear ();
+}
 
 /** Add a bundle to a group.
  *  @param b Bundle.
+ *  @param allow_dups true to allow the group to contain more than one bundle with the same port, otherwise false.
  */
 void
-PortGroup::add_bundle (boost::shared_ptr<ARDOUR::Bundle> b)
+PortGroup::add_bundle (boost::shared_ptr<Bundle> b, bool allow_dups)
 {
-       assert (b.get());
-       _bundles.push_back (b);
+       add_bundle_internal (b, boost::shared_ptr<IO> (), false, Gdk::Color (), allow_dups);
+}
 
-       Modified ();
+/** Add a bundle to a group.
+ *  @param b Bundle.
+ *  @param io IO whose ports are in the bundle.
+ */
+void
+PortGroup::add_bundle (boost::shared_ptr<Bundle> b, boost::shared_ptr<IO> io)
+{
+       add_bundle_internal (b, io, false, Gdk::Color (), false);
 }
 
-/** Add a port to a group.
- *  @param p Port.
+/** Add a bundle to a group.
+ *  @param b Bundle.
+ *  @param c Colour to represent the bundle with.
  */
 void
-PortGroup::add_port (std::string const &p)
+PortGroup::add_bundle (boost::shared_ptr<Bundle> b, boost::shared_ptr<IO> io, Gdk::Color c)
 {
-       ports.push_back (p);
+       add_bundle_internal (b, io, true, c, false);
+}
 
-       Modified ();
+PortGroup::BundleRecord::BundleRecord (boost::shared_ptr<ARDOUR::Bundle> b, boost::shared_ptr<ARDOUR::IO> iop, Gdk::Color c, bool has_c)
+       : bundle (b)
+       , io (iop)
+       , colour (c)
+       , has_colour (has_c)
+{
 }
 
+void
+PortGroup::add_bundle_internal (boost::shared_ptr<Bundle> b, boost::shared_ptr<IO> io, bool has_colour, Gdk::Color colour, bool allow_dups)
+{
+       assert (b.get());
+
+       if (!allow_dups) {
+               
+               /* don't add this bundle if we already have one with the same ports */
+               
+               BundleList::iterator i = _bundles.begin ();
+               while (i != _bundles.end() && b->has_same_ports ((*i)->bundle) == false) {
+                       ++i;
+               }
+               
+               if (i != _bundles.end ()) {
+                       return;
+               }
+       }
+
+       BundleRecord* br = new BundleRecord (b, io, colour, has_colour);
+       b->Changed.connect (br->changed_connection, invalidator (*this), ui_bind (&PortGroup::bundle_changed, this, _1), gui_context());
+       _bundles.push_back (br);
+
+       Changed ();     
+}
+
+void
+PortGroup::remove_bundle (boost::shared_ptr<Bundle> b)
+{
+       assert (b.get());
+
+       BundleList::iterator i = _bundles.begin ();
+       while (i != _bundles.end() && (*i)->bundle != b) {
+               ++i;
+       }
+
+       if (i == _bundles.end()) {
+               return;
+       }
+
+       delete *i;
+       _bundles.erase (i);
+
+       Changed ();
+}
+
+void
+PortGroup::bundle_changed (Bundle::Change c)
+{
+       BundleChanged (c);
+}
+
+
 void
 PortGroup::clear ()
 {
-       _bundles.clear ();
-       ports.clear ();
+       for (BundleList::iterator i = _bundles.begin(); i != _bundles.end(); ++i) {
+               delete *i;
+       }
 
-       Modified ();
+       _bundles.clear ();
+       Changed ();
 }
 
 bool
 PortGroup::has_port (std::string const& p) const
 {
-       for (ARDOUR::BundleList::const_iterator i = _bundles.begin(); i != _bundles.end(); ++i) {
-               if ((*i)->offers_port_alone (p)) {
-                       return true;
-               }
-       }
-
-       for (vector<std::string>::const_iterator i = ports.begin(); i != ports.end(); ++i) {
-               if (*i == p) {
+       for (BundleList::const_iterator i = _bundles.begin(); i != _bundles.end(); ++i) {
+               if ((*i)->bundle->offers_port_alone (p)) {
                        return true;
                }
        }
@@ -82,160 +173,319 @@ PortGroup::has_port (std::string const& p) const
        return false;
 }
 
-boost::shared_ptr<ARDOUR::Bundle>
+boost::shared_ptr<Bundle>
 PortGroup::only_bundle ()
 {
        assert (_bundles.size() == 1);
-       return _bundles.front();
+       return _bundles.front()->bundle;
 }
 
 
 uint32_t
-PortGroup::total_ports () const
+PortGroup::total_channels () const
 {
        uint32_t n = 0;
-       for (ARDOUR::BundleList::const_iterator i = _bundles.begin(); i != _bundles.end(); ++i) {
-               n += (*i)->nchannels ();
+       for (BundleList::const_iterator i = _bundles.begin(); i != _bundles.end(); ++i) {
+               n += (*i)->bundle->nchannels ();
        }
 
-       n += ports.size();
-
        return n;
 }
 
-       
+boost::shared_ptr<IO>
+PortGroup::io_from_bundle (boost::shared_ptr<ARDOUR::Bundle> b) const
+{
+       BundleList::const_iterator i = _bundles.begin ();
+       while (i != _bundles.end() && (*i)->bundle != b) {
+               ++i;
+       }
+
+       if (i == _bundles.end()) {
+               return boost::shared_ptr<IO> ();
+       }
+
+       return (*i)->io;
+}
+
+/** Remove bundles whose channels are already represented by other, larger bundles */
+void
+PortGroup::remove_duplicates ()
+{
+       BundleList::iterator i = _bundles.begin();
+       while (i != _bundles.end()) {
+
+               BundleList::iterator tmp = i;
+               ++tmp;
+
+               bool remove = false;
+
+               for (BundleList::iterator j = _bundles.begin(); j != _bundles.end(); ++j) {
+
+                       if ((*j)->bundle->nchannels() > (*i)->bundle->nchannels()) {
+                               /* this bundle is larger */
+
+                               uint32_t k = 0;
+                               while (k < (*i)->bundle->nchannels()) {
+                                       /* see if this channel on *i has an equivalent on *j */
+                                       uint32_t l = 0;
+                                       while (l < (*j)->bundle->nchannels() && (*i)->bundle->channel_ports (k) != (*j)->bundle->channel_ports (l)) {
+                                               ++l;
+                                       }
+
+                                       if (l == (*j)->bundle->nchannels()) {
+                                               /* it does not */
+                                               break;
+                                       }
+
+                                       ++k;
+                               }
+
+                               if (k == (*i)->bundle->nchannels ()) {
+                                       /* all channels on *i are represented by the larger bundle *j, so remove *i */
+                                       remove = true;
+                                       break;
+                               }
+                       }
+               }
+               
+               if (remove) {
+                       _bundles.erase (i);
+               }
+               
+               i = tmp;
+       }
+}
+
+
 /** PortGroupList constructor.
  */
-
 PortGroupList::PortGroupList ()
-       : _type (ARDOUR::DataType::AUDIO), _bundles_dirty (true)
+       : _type (DataType::AUDIO), _signals_suspended (false), _pending_change (false), _pending_bundle_change ((Bundle::Change) 0)
 {
-       
+
+}
+
+PortGroupList::~PortGroupList() 
+{
+       /* XXX need to clean up bundles, but ownership shared with PortGroups */
 }
 
 void
-PortGroupList::set_type (ARDOUR::DataType t)
+PortGroupList::set_type (DataType t)
 {
        _type = t;
        clear ();
 }
 
+void
+PortGroupList::maybe_add_processor_to_list (
+       boost::weak_ptr<Processor> wp, list<boost::shared_ptr<Bundle> >* route_bundles, bool inputs, set<boost::shared_ptr<IO> >& used_io
+       )
+{
+       boost::shared_ptr<Processor> p (wp.lock());
+
+       if (!p) {
+               return;
+       }
+
+       boost::shared_ptr<IOProcessor> iop = boost::dynamic_pointer_cast<IOProcessor> (p);
+
+       if (iop) {
+
+               boost::shared_ptr<IO> io = inputs ? iop->input() : iop->output();
+
+               if (io && used_io.find (io) == used_io.end()) {
+                       route_bundles->push_back (io->bundle ());
+                       used_io.insert (io);
+               }
+       }
+}
+
+
 /** Gather bundles from around the system and put them in this PortGroupList */
 void
-PortGroupList::gather (ARDOUR::Session& session, bool inputs)
+PortGroupList::gather (ARDOUR::Session* session, bool inputs, bool allow_dups)
 {
        clear ();
 
-       boost::shared_ptr<PortGroup> buss (new PortGroup (_("Buss")));
+       if (session == 0) {
+               return;
+       }
+
+       boost::shared_ptr<PortGroup> bus (new PortGroup (_("Bus")));
        boost::shared_ptr<PortGroup> track (new PortGroup (_("Track")));
        boost::shared_ptr<PortGroup> system (new PortGroup (_("System")));
+       boost::shared_ptr<PortGroup> ardour (new PortGroup (_("Ardour")));
        boost::shared_ptr<PortGroup> other (new PortGroup (_("Other")));
 
-       /* Find the bundles for routes.  We take their bundles, copy them,
-          and add ports from the route's processors */
-
-       boost::shared_ptr<ARDOUR::RouteList> routes = session.get_routes ();
-
-       for (ARDOUR::RouteList::const_iterator i = routes->begin(); i != routes->end(); ++i) {
-               /* Copy the appropriate bundle from the route */
-               boost::shared_ptr<ARDOUR::Bundle> bundle (
-                       new ARDOUR::Bundle (
-                               inputs ? (*i)->bundle_for_inputs() : (*i)->bundle_for_outputs ()
-                               )
-                       );
-
-               /* Add ports from the route's processors */
-               uint32_t n = 0;
-               while (1) {
-                       boost::shared_ptr<ARDOUR::Processor> p = (*i)->nth_processor (n);
-                       if (p == 0) {
-                               break;
-                       }
+       /* Find the bundles for routes.  We use the RouteBundle class to join
+          the route's input/output and processor bundles together so that they
+          are presented as one bundle in the matrix. */
 
-                       boost::shared_ptr<ARDOUR::IOProcessor> iop = boost::dynamic_pointer_cast<ARDOUR::IOProcessor> (p);
+       boost::shared_ptr<RouteList> routes = session->get_routes ();
 
-                       if (iop) {
-                               boost::shared_ptr<ARDOUR::Bundle> pb = inputs ?
-                                       iop->io()->bundle_for_inputs() : iop->io()->bundle_for_outputs();
-                               bundle->add_channels_from_bundle (pb);
-                       }
+       for (RouteList::const_iterator i = routes->begin(); i != routes->end(); ++i) {
 
-                       ++n;
-               }
-                       
-               /* Work out which group to put this bundle in */
+               list<boost::shared_ptr<Bundle> > route_bundles;
+
+               /* keep track of IOs that we have taken bundles from,
+                  so that we can avoid taking the same IO from both
+                  Route::output() and the main_outs Delivery */
+
+               set<boost::shared_ptr<IO> > used_io;
+               boost::shared_ptr<IO> io = inputs ? (*i)->input() : (*i)->output();
+               used_io.insert (io);
+
+               route_bundles.push_back (io->bundle ());
+
+               (*i)->foreach_processor (boost::bind (&PortGroupList::maybe_add_processor_to_list, this, _1, &route_bundles, inputs, used_io));
+
+               /* Work out which group to put these bundles in */
                boost::shared_ptr<PortGroup> g;
-               if (_type == ARDOUR::DataType::AUDIO) {
+               if (_type == DataType::AUDIO) {
 
-                       if (boost::dynamic_pointer_cast<ARDOUR::AudioTrack> (*i)) {
+                       if (boost::dynamic_pointer_cast<AudioTrack> (*i)) {
                                g = track;
-                       } else if (!boost::dynamic_pointer_cast<ARDOUR::MidiTrack>(*i)) {
-                               g = buss;
-                       } 
+                       } else if (!boost::dynamic_pointer_cast<MidiTrack>(*i)) {
+                               g = bus;
+                       }
 
 
-               } else if (_type == ARDOUR::DataType::MIDI) {
+               } else if (_type == DataType::MIDI) {
 
-                       if (boost::dynamic_pointer_cast<ARDOUR::MidiTrack> (*i)) {
+                       if (boost::dynamic_pointer_cast<MidiTrack> (*i)) {
                                g = track;
                        }
 
                        /* No MIDI busses yet */
-               } 
-                       
+               }
+
                if (g) {
-                       g->add_bundle (bundle);
+
+                       TimeAxisView* tv = PublicEditor::instance().axis_view_from_route (*i);
+                       for (list<boost::shared_ptr<Bundle> >::iterator i = route_bundles.begin(); i != route_bundles.end(); ++i) {
+                               if (tv) {
+                                       g->add_bundle (*i, io, tv->color ());
+                               } else {
+                                       g->add_bundle (*i, io);
+                               }
+                       }
                }
        }
 
-       /* Bundles created by the session */
-       
-       boost::shared_ptr<ARDOUR::BundleList> b = session.bundles ();
-       for (ARDOUR::BundleList::iterator i = b->begin(); i != b->end(); ++i) {
-               if ((*i)->ports_are_inputs() == inputs && (*i)->type() == _type) {
-                       system->add_bundle (*i);
+       /* Bundles owned by the session; add user bundles first, then normal ones, so
+          that UserBundles that offer the same ports as a normal bundle get priority
+       */
+
+       boost::shared_ptr<BundleList> b = session->bundles ();
+
+       for (BundleList::iterator i = b->begin(); i != b->end(); ++i) {
+               if (boost::dynamic_pointer_cast<UserBundle> (*i) && (*i)->ports_are_inputs() == inputs && (*i)->type() == _type) {
+                       system->add_bundle (*i, allow_dups);
                }
        }
 
+       for (BundleList::iterator i = b->begin(); i != b->end(); ++i) {
+               if (boost::dynamic_pointer_cast<UserBundle> (*i) == 0 && (*i)->ports_are_inputs() == inputs && (*i)->type() == _type) {
+                       system->add_bundle (*i, allow_dups);
+               }
+       }
+       
+       /* Ardour stuff */
+
+       if (!inputs && _type == DataType::AUDIO) {
+               ardour->add_bundle (session->the_auditioner()->output()->bundle());
+               ardour->add_bundle (session->click_io()->bundle());
+       }
+
        /* Now find all other ports that we haven't thought of yet */
 
-       const char **ports = session.engine().get_ports ("", _type.to_jack_type(), inputs ? 
-                                                         JackPortIsInput : JackPortIsOutput);
+       std::vector<std::string> extra_system;
+       std::vector<std::string> extra_other;
+
+       const char **ports = session->engine().get_ports ("", _type.to_jack_type(), inputs ?
+                                                        JackPortIsInput : JackPortIsOutput);
        if (ports) {
 
                int n = 0;
                string client_matching_string;
 
-               client_matching_string = session.engine().client_name();
+               client_matching_string = session->engine().client_name();
                client_matching_string += ':';
 
                while (ports[n]) {
-                       
+
                        std::string const p = ports[n];
 
-                       if (!system->has_port(p) && !buss->has_port(p) && !track->has_port(p) && !other->has_port(p)) {
-                               
+                       if (!system->has_port(p) &&
+                           !bus->has_port(p) &&
+                           !track->has_port(p) &&
+                           !ardour->has_port(p) &&
+                           !other->has_port(p)) {
+                                
+                                /* special hack: ignore MIDI ports labelled Midi-Through. these
+                                   are basically useless and mess things up for default
+                                   connections.
+                                */
+
+                                if (p.find ("MIDI-Through") != string::npos) {
+                                        ++n;
+                                        continue;
+                                }
+
                                if (port_has_prefix (p, "system:") ||
                                    port_has_prefix (p, "alsa_pcm") ||
                                    port_has_prefix (p, "ardour:")) {
-                                       system->add_port (p);
+                                       extra_system.push_back (p);
                                } else {
-                                       other->add_port (p);
+                                       extra_other.push_back (p);
                                }
                        }
-                       
+
                        ++n;
                }
 
                free (ports);
        }
 
-       add_group (system);
-       add_group (buss);
-       add_group (track);
-       add_group (other);
+       if (!extra_system.empty()) {
+               boost::shared_ptr<Bundle> b = make_bundle_from_ports (extra_system, inputs);
+               system->add_bundle (b);
+       }
+
+       if (!extra_other.empty()) {
+               other->add_bundle (make_bundle_from_ports (extra_other, inputs));
+       }
+
+       if (!allow_dups) {
+               system->remove_duplicates ();
+       }
+
+       add_group_if_not_empty (other);
+       add_group_if_not_empty (bus);
+       add_group_if_not_empty (track);
+       add_group_if_not_empty (ardour);
+       add_group_if_not_empty (system);
+
+       emit_changed ();
+}
+
+boost::shared_ptr<Bundle>
+PortGroupList::make_bundle_from_ports (std::vector<std::string> const & p, bool inputs) const
+{
+       boost::shared_ptr<Bundle> b (new Bundle ("", _type, inputs));
+
+       std::string const pre = common_prefix (p);
+       if (!pre.empty()) {
+               b->set_name (pre.substr (0, pre.length() - 1));
+       }
+
+       for (uint32_t j = 0; j < p.size(); ++j) {
+               b->add_channel (p[j].substr (pre.length()));
+               b->set_port (j, p[j]);
+       }
 
-       _bundles_dirty = true;
+       return b;
 }
 
 bool
@@ -243,75 +493,47 @@ PortGroupList::port_has_prefix (const std::string& n, const std::string& p) cons
 {
        return n.substr (0, p.length()) == p;
 }
-       
 
-void
-PortGroupList::update_bundles () const
+std::string
+PortGroupList::common_prefix_before (std::vector<std::string> const & p, std::string const & s) const
 {
-       _bundles.clear ();
-               
-       for (PortGroupList::List::const_iterator i = begin (); i != end (); ++i) {
-               if ((*i)->visible()) {
-                       
-                       std::copy ((*i)->bundles().begin(), (*i)->bundles().end(), std::back_inserter (_bundles));
-
-                       /* make a bundle for the ports, if there are any */
-                       if (!(*i)->ports.empty()) {
-
-                               boost::shared_ptr<ARDOUR::Bundle> b (new ARDOUR::Bundle ("", _type, !_offer_inputs));
-                               
-                               std::string const pre = common_prefix ((*i)->ports);
-                               if (!pre.empty()) {
-                                       b->set_name (pre.substr (0, pre.length() - 1));
-                               }
+       /* we must have some strings and the first must contain the separator string */
+       if (p.empty() || p[0].find_first_of (s) == std::string::npos) {
+               return "";
+       }
 
-                               for (uint32_t j = 0; j < (*i)->ports.size(); ++j) {
-                                       std::string const p = (*i)->ports[j];
-                                       b->add_channel (p.substr (pre.length()));
-                                       b->set_port (j, p);
-                               }
-                                       
-                               _bundles.push_back (b);
-                       }
+       /* prefix of the first string */
+       std::string const fp = p[0].substr (0, p[0].find_first_of (s) + 1);
+
+       /* see if the other strings also start with fp */
+       uint32_t j = 1;
+       while (j < p.size()) {
+               if (p[j].substr (0, fp.length()) != fp) {
+                       break;
                }
+               ++j;
+       }
+
+       if (j != p.size()) {
+               return "";
        }
 
-       _bundles_dirty = false;
+       return fp;
 }
 
+
 std::string
 PortGroupList::common_prefix (std::vector<std::string> const & p) const
 {
        /* common prefix before '/' ? */
-       if (p[0].find_first_of ("/") != std::string::npos) {
-               std::string const fp = p[0].substr (0, (p[0].find_first_of ("/") + 1));
-               uint32_t j = 1;
-               while (j < p.size()) {
-                       if (p[j].substr (0, fp.length()) != fp) {
-                               break;
-                       }
-                       ++j;
-               }
-               
-               if (j == p.size()) {
-                       return fp;
-               }
+       std::string cp = common_prefix_before (p, "/");
+       if (!cp.empty()) {
+               return cp;
        }
-       
-       /* or before ':' ? */
-       if (p[0].find_first_of (":") != std::string::npos) {
-               std::string const fp = p[0].substr (0, (p[0].find_first_of (":") + 1));
-               uint32_t j = 1;
-               while (j < p.size()) {
-                       if (p[j].substr (0, fp.length()) != fp) {
-                               break;
-                       }
-                       ++j;
-               }
-               
-               if (j == p.size()) {
-                       return fp;
-               }
+
+       cp = common_prefix_before (p, ":");
+       if (!cp.empty()) {
+               return cp;
        }
 
        return "";
@@ -321,43 +543,128 @@ void
 PortGroupList::clear ()
 {
        _groups.clear ();
-       _bundles_dirty = true;
+       _bundle_changed_connections.drop_connections ();
+       emit_changed ();
 }
 
-ARDOUR::BundleList const &
+
+PortGroup::BundleList const &
 PortGroupList::bundles () const
 {
-       if (_bundles_dirty) {
-               update_bundles ();
+       _bundles.clear ();
+
+       for (PortGroupList::List::const_iterator i = begin (); i != end (); ++i) {
+               std::copy ((*i)->bundles().begin(), (*i)->bundles().end(), std::back_inserter (_bundles));
        }
 
        return _bundles;
 }
 
 uint32_t
-PortGroupList::total_visible_ports () const
+PortGroupList::total_channels () const
 {
        uint32_t n = 0;
-       
+
        for (PortGroupList::List::const_iterator i = begin(); i != end(); ++i) {
-               if ((*i)->visible()) {
-                       n += (*i)->total_ports ();
-               }
+               n += (*i)->total_channels ();
        }
 
        return n;
 }
 
 void
-PortGroupList::group_modified ()
+PortGroupList::add_group_if_not_empty (boost::shared_ptr<PortGroup> g)
 {
-       _bundles_dirty = true;
+       if (!g->bundles().empty ()) {
+               add_group (g);
+       }
 }
 
 void
 PortGroupList::add_group (boost::shared_ptr<PortGroup> g)
 {
        _groups.push_back (g);
-       g->Modified.connect (sigc::mem_fun (*this, &PortGroupList::group_modified));
-       _bundles_dirty = true;
+
+       g->Changed.connect (_changed_connections, invalidator (*this), boost::bind (&PortGroupList::emit_changed, this), gui_context());
+       g->BundleChanged.connect (_bundle_changed_connections, invalidator (*this), ui_bind (&PortGroupList::emit_bundle_changed, this, _1), gui_context());
+
+       emit_changed ();
 }
+
+void
+PortGroupList::remove_bundle (boost::shared_ptr<Bundle> b)
+{
+       for (List::iterator i = _groups.begin(); i != _groups.end(); ++i) {
+               (*i)->remove_bundle (b);
+       }
+
+       emit_changed ();
+}
+
+void
+PortGroupList::emit_changed ()
+{
+       if (_signals_suspended) {
+               _pending_change = true;
+       } else {
+               Changed ();
+       }
+}
+
+void
+PortGroupList::emit_bundle_changed (Bundle::Change c)
+{
+       if (_signals_suspended) {
+               _pending_bundle_change = c;
+       } else {
+               BundleChanged (c);
+       }
+}
+void
+PortGroupList::suspend_signals ()
+{
+       _signals_suspended = true;
+}
+
+void
+PortGroupList::resume_signals ()
+{
+       if (_pending_change) {
+               Changed ();
+               _pending_change = false;
+       }
+
+       if (_pending_bundle_change != 0) {
+               BundleChanged (_pending_bundle_change);
+               _pending_bundle_change = (ARDOUR::Bundle::Change) 0;
+       }
+
+       _signals_suspended = false;
+}
+
+boost::shared_ptr<IO>
+PortGroupList::io_from_bundle (boost::shared_ptr<ARDOUR::Bundle> b) const
+{
+       List::const_iterator i = _groups.begin ();
+       while (i != _groups.end()) {
+               boost::shared_ptr<IO> io = (*i)->io_from_bundle (b);
+               if (io) {
+                       return io;
+               }
+               ++i;
+       }
+
+       return boost::shared_ptr<IO> ();
+}
+
+bool
+PortGroupList::empty () const
+{
+       List::const_iterator i = _groups.begin ();
+       while (i != _groups.end() && (*i)->total_channels() == 0) {
+               ++i;
+       }
+
+       return (i == _groups.end());
+}
+