X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=gtk2_ardour%2Fport_group.cc;h=1598c5933c2b11ade888562586f1df2512d3f75b;hb=7110f48619af3b93fd2b1e2893462a63f6bf96fd;hp=25897222644e8f37f7108acd9aa93016fc855caa;hpb=dc86434283718b7c4a4ed6af52a950e6cfc5e86f;p=ardour.git

diff --git a/gtk2_ardour/port_group.cc b/gtk2_ardour/port_group.cc
index 2589722264..1598c5933c 100644
--- a/gtk2_ardour/port_group.cc
+++ b/gtk2_ardour/port_group.cc
@@ -20,6 +20,9 @@
 #include <cstring>
 #include <boost/shared_ptr.hpp>
 
+#include "midi++/manager.h"
+#include "midi++/mmc.h"
+
 #include "ardour/audio_track.h"
 #include "ardour/audioengine.h"
 #include "ardour/bundle.h"
@@ -29,7 +32,10 @@
 #include "ardour/port.h"
 #include "ardour/session.h"
 #include "ardour/auditioner.h"
+#include "ardour/control_protocol_manager.h"
+#include "control_protocol/control_protocol.h"
 
+#include "gui_thread.h"
 #include "port_group.h"
 #include "port_matrix.h"
 #include "time_axis_view.h"
@@ -50,6 +56,14 @@ PortGroup::PortGroup (std::string const & 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.
@@ -80,6 +94,14 @@ PortGroup::add_bundle (boost::shared_ptr<Bundle> b, boost::shared_ptr<IO> io, Gd
 	add_bundle_internal (b, io, true, c, false);
 }
 
+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)
 {
@@ -90,7 +112,7 @@ PortGroup::add_bundle_internal (boost::shared_ptr<Bundle> b, boost::shared_ptr<I
 		/* 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) {
+		while (i != _bundles.end() && b->has_same_ports ((*i)->bundle) == false) {
 			++i;
 		}
 		
@@ -99,14 +121,9 @@ PortGroup::add_bundle_internal (boost::shared_ptr<Bundle> b, boost::shared_ptr<I
 		}
 	}
 
-	BundleRecord r;
-	r.bundle = b;
-	r.io = io;
-	r.colour = colour;
-	r.has_colour = has_colour;
-	r.changed_connection = b->Changed.connect (sigc::mem_fun (*this, &PortGroup::bundle_changed));
-
-	_bundles.push_back (r);
+	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 ();	
 }
@@ -117,7 +134,7 @@ PortGroup::remove_bundle (boost::shared_ptr<Bundle> b)
 	assert (b.get());
 
 	BundleList::iterator i = _bundles.begin ();
-	while (i != _bundles.end() && i->bundle != b) {
+	while (i != _bundles.end() && (*i)->bundle != b) {
 		++i;
 	}
 
@@ -125,7 +142,7 @@ PortGroup::remove_bundle (boost::shared_ptr<Bundle> b)
 		return;
 	}
 
-	i->changed_connection.disconnect ();
+	delete *i;
 	_bundles.erase (i);
 
 	Changed ();
@@ -142,7 +159,7 @@ void
 PortGroup::clear ()
 {
 	for (BundleList::iterator i = _bundles.begin(); i != _bundles.end(); ++i) {
-		i->changed_connection.disconnect ();
+		delete *i;
 	}
 
 	_bundles.clear ();
@@ -153,7 +170,7 @@ bool
 PortGroup::has_port (std::string const& p) const
 {
 	for (BundleList::const_iterator i = _bundles.begin(); i != _bundles.end(); ++i) {
-		if (i->bundle->offers_port_alone (p)) {
+		if ((*i)->bundle->offers_port_alone (p)) {
 			return true;
 		}
 	}
@@ -165,16 +182,16 @@ boost::shared_ptr<Bundle>
 PortGroup::only_bundle ()
 {
 	assert (_bundles.size() == 1);
-	return _bundles.front().bundle;
+	return _bundles.front()->bundle;
 }
 
 
-uint32_t
+ChanCount
 PortGroup::total_channels () const
 {
-	uint32_t n = 0;
+	ChanCount n;
 	for (BundleList::const_iterator i = _bundles.begin(); i != _bundles.end(); ++i) {
-		n += i->bundle->nchannels ();
+		n += (*i)->bundle->nchannels ();
 	}
 
 	return n;
@@ -184,7 +201,7 @@ 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) {
+	while (i != _bundles.end() && (*i)->bundle != b) {
 		++i;
 	}
 
@@ -192,28 +209,75 @@ PortGroup::io_from_bundle (boost::shared_ptr<ARDOUR::Bundle> b) const
 		return boost::shared_ptr<IO> ();
 	}
 
-	return i->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().n_total()) {
+					/* see if this channel on *i has an equivalent on *j */
+					uint32_t l = 0;
+					while (l < (*j)->bundle->nchannels().n_total() && (*i)->bundle->channel_ports (k) != (*j)->bundle->channel_ports (l)) {
+						++l;
+					}
+
+					if (l == (*j)->bundle->nchannels().n_total()) {
+						/* it does not */
+						break;
+					}
+
+					++k;
+				}
+
+				if (k == (*i)->bundle->nchannels().n_total()) {
+					/* 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 (DataType::AUDIO), _signals_suspended (false), _pending_change (false), _pending_bundle_change ((Bundle::Change) 0)
+	: _signals_suspended (false), _pending_change (false), _pending_bundle_change ((Bundle::Change) 0)
 {
 
 }
 
-void
-PortGroupList::set_type (DataType t)
+PortGroupList::~PortGroupList() 
 {
-	_type = t;
-	clear ();
+	/* XXX need to clean up bundles, but ownership shared with PortGroups */
 }
 
 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::weak_ptr<Processor> wp, list<boost::shared_ptr<IO> >* route_ios, bool inputs, set<boost::shared_ptr<IO> >& used_io
 	)
 {
 	boost::shared_ptr<Processor> p (wp.lock());
@@ -229,16 +293,34 @@ PortGroupList::maybe_add_processor_to_list (
 		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 ());
+			route_ios->push_back (io);
 			used_io.insert (io);
 		}
 	}
 }
 
+struct RouteIOs {
+	RouteIOs (boost::shared_ptr<Route> r, boost::shared_ptr<IO> i) {
+		route = r;
+		ios.push_back (i);
+	}
+	
+	boost::shared_ptr<Route> route;
+	std::list<boost::shared_ptr<IO> > ios;
+};
+
+class RouteIOsComparator {
+public:
+	bool operator() (RouteIOs const & a, RouteIOs const & b) {
+		return a.route->order_key (X_("editor")) < b.route->order_key (X_("editor"));
+	}
+};
 
-/** Gather bundles from around the system and put them in this PortGroupList */
+/** Gather ports from around the system and put them in this PortGroupList.
+ *  @param type Type of ports to collect, or NIL for all types.
+ */
 void
-PortGroupList::gather (ARDOUR::Session* session, bool inputs, bool allow_dups)
+PortGroupList::gather (ARDOUR::Session* session, ARDOUR::DataType type, bool inputs, bool allow_dups)
 {
 	clear ();
 
@@ -252,16 +334,16 @@ PortGroupList::gather (ARDOUR::Session* session, bool inputs, bool allow_dups)
 	boost::shared_ptr<PortGroup> ardour (new PortGroup (_("Ardour")));
 	boost::shared_ptr<PortGroup> other (new PortGroup (_("Other")));
 
-	/* 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. */
+	/* Find the IOs which have bundles for routes and their processors.  We store
+	   these IOs in a RouteIOs class so that we can then sort the results by route
+	   order key.
+	*/
 
 	boost::shared_ptr<RouteList> routes = session->get_routes ();
+	list<RouteIOs> route_ios;
 
 	for (RouteList::const_iterator i = routes->begin(); i != routes->end(); ++i) {
 
-		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 */
@@ -270,38 +352,38 @@ PortGroupList::gather (ARDOUR::Session* session, bool inputs, bool allow_dups)
 		boost::shared_ptr<IO> io = inputs ? (*i)->input() : (*i)->output();
 		used_io.insert (io);
 
-		route_bundles.push_back (io->bundle ());
-
-		(*i)->foreach_processor (bind (mem_fun (*this, &PortGroupList::maybe_add_processor_to_list), &route_bundles, inputs, used_io));
-
-		/* Work out which group to put these bundles in */
-		boost::shared_ptr<PortGroup> g;
-		if (_type == DataType::AUDIO) {
-
-			if (boost::dynamic_pointer_cast<AudioTrack> (*i)) {
-				g = track;
-			} else if (!boost::dynamic_pointer_cast<MidiTrack>(*i)) {
-				g = bus;
-			}
+		RouteIOs rb (*i, io);
+		(*i)->foreach_processor (boost::bind (&PortGroupList::maybe_add_processor_to_list, this, _1, &rb.ios, inputs, used_io));
 
+		route_ios.push_back (rb);
+	}
 
-		} else if (_type == DataType::MIDI) {
+	/* Sort RouteIOs by the routes' editor order keys */
+	route_ios.sort (RouteIOsComparator ());
 
-			if (boost::dynamic_pointer_cast<MidiTrack> (*i)) {
-				g = track;
-			}
+	/* Now put the bundles that belong to these sorted RouteIOs into the PortGroup.
+	   Note that if the RouteIO's bundles are multi-type, we may make new Bundles
+	   with only the ports of one type.
+	*/
+	
+	for (list<RouteIOs>::iterator i = route_ios.begin(); i != route_ios.end(); ++i) {
+		TimeAxisView* tv = PublicEditor::instance().axis_view_from_route (i->route);
 
-			/* No MIDI busses yet */
+		/* Work out which group to put these IOs' bundles in */
+		boost::shared_ptr<PortGroup> g;
+		if (boost::dynamic_pointer_cast<Track> (i->route)) {
+			g = track;
+		} else {
+			g = bus;
 		}
 
-		if (g) {
-
-			TimeAxisView* tv = PublicEditor::instance().axis_view_from_route (i->get());
-			for (list<boost::shared_ptr<Bundle> >::iterator i = route_bundles.begin(); i != route_bundles.end(); ++i) {
+		for (list<boost::shared_ptr<IO> >::iterator j = i->ios.begin(); j != i->ios.end(); ++j) {
+			boost::shared_ptr<Bundle> b = bundle_for_type ((*j)->bundle(), type);
+			if (b->nchannels() != ChanCount::ZERO) {
 				if (tv) {
-					g->add_bundle (*i, io, tv->color ());
+					g->add_bundle (b, *j, tv->color ());
 				} else {
-					g->add_bundle (*i, io);
+					g->add_bundle (b, *j);
 				}
 			}
 		}
@@ -314,57 +396,137 @@ PortGroupList::gather (ARDOUR::Session* session, bool inputs, bool allow_dups)
 	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);
+		if (boost::dynamic_pointer_cast<UserBundle> (*i) && (*i)->ports_are_inputs() == inputs) {
+			boost::shared_ptr<Bundle> b = bundle_for_type (*i, type);
+			if (b->nchannels() != ChanCount::ZERO) {
+				system->add_bundle (b, 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);
+		if (boost::dynamic_pointer_cast<UserBundle> (*i) == 0 && (*i)->ports_are_inputs() == inputs) {
+			boost::shared_ptr<Bundle> b = bundle_for_type (*i, type);
+			if (b->nchannels() != ChanCount::ZERO) {
+				system->add_bundle (b, allow_dups);
+			}
 		}
 	}
 	
 	/* Ardour stuff */
 
-	if (!inputs && _type == DataType::AUDIO) {
-		ardour->add_bundle (session->the_auditioner()->output()->bundle());
-		ardour->add_bundle (session->click_io()->bundle());
+	if (!inputs) {
+		boost::shared_ptr<Bundle> b = bundle_for_type (session->the_auditioner()->output()->bundle(), type);
+		if (b->nchannels() != ChanCount::ZERO) {
+			ardour->add_bundle (b);
+		}
+
+		b = bundle_for_type (session->click_io()->bundle(), type);
+		if (b->nchannels() != ChanCount::ZERO) {
+			ardour->add_bundle (b);
+		}
+	}
+
+	/* Ardour's surfaces */
+
+	ControlProtocolManager& m = ControlProtocolManager::instance ();
+	for (list<ControlProtocolInfo*>::iterator i = m.control_protocol_info.begin(); i != m.control_protocol_info.end(); ++i) {
+		if ((*i)->protocol) {
+			list<boost::shared_ptr<Bundle> > b = (*i)->protocol->bundles ();
+			for (list<boost::shared_ptr<Bundle> >::iterator j = b.begin(); j != b.end(); ++j) {
+				if ((*j)->ports_are_inputs() == inputs) {
+					ardour->add_bundle (*j);
+				}
+			}
+		}
+	}
+
+	/* Ardour's sync ports */
+
+	MIDI::Manager* midi_manager = MIDI::Manager::instance ();
+	if (midi_manager && (type == DataType::MIDI || type == DataType::NIL)) {
+		boost::shared_ptr<Bundle> sync (new Bundle (_("Sync"), inputs));
+		MIDI::MachineControl* mmc = midi_manager->mmc ();
+		AudioEngine& ae = session->engine ();
+		if (inputs) {
+			sync->add_channel (
+				_("MTC in"), DataType::MIDI, ae.make_port_name_non_relative (midi_manager->mtc_input_port()->name())
+				);
+			sync->add_channel (
+				_("MIDI control in"), DataType::MIDI, ae.make_port_name_non_relative (midi_manager->midi_input_port()->name())
+				);
+			sync->add_channel (
+				_("MIDI clock in"), DataType::MIDI, ae.make_port_name_non_relative (midi_manager->midi_clock_input_port()->name())
+				);
+			sync->add_channel (
+				_("MMC in"), DataType::MIDI, ae.make_port_name_non_relative (mmc->input_port()->name())
+				);
+		} else {
+			sync->add_channel (
+				_("MTC out"), DataType::MIDI, ae.make_port_name_non_relative (midi_manager->mtc_output_port()->name())
+				);
+			sync->add_channel (
+				_("MIDI control out"), DataType::MIDI, ae.make_port_name_non_relative (midi_manager->midi_output_port()->name())
+				);
+			sync->add_channel (
+				_("MIDI clock out"), DataType::MIDI, ae.make_port_name_non_relative (midi_manager->midi_clock_output_port()->name())
+				);
+			sync->add_channel (
+				_("MMC out"), DataType::MIDI, ae.make_port_name_non_relative (mmc->output_port()->name())
+				);
+		}
+		
+		ardour->add_bundle (sync);
 	}
 
 	/* Now find all other ports that we haven't thought of yet */
 
-	std::vector<std::string> extra_system;
-	std::vector<std::string> extra_other;
+	std::vector<std::string> extra_system[DataType::num_types];
+	std::vector<std::string> extra_other[DataType::num_types];
 
- 	const char **ports = session->engine().get_ports ("", _type.to_jack_type(), inputs ?
-							 JackPortIsInput : JackPortIsOutput);
+	const char ** ports = 0;
+	if (type == DataType::NIL) {
+		ports = session->engine().get_ports ("", "", inputs ? JackPortIsInput : JackPortIsOutput);
+	} else {
+		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 += ':';
 
 		while (ports[n]) {
 
 			std::string const p = ports[n];
 
-			cout << p << "\n";
-
 			if (!system->has_port(p) &&
 			    !bus->has_port(p) &&
 			    !track->has_port(p) &&
 			    !ardour->has_port(p) &&
 			    !other->has_port(p)) {
-
-				if (port_has_prefix (p, "system:") ||
-				    port_has_prefix (p, "alsa_pcm") ||
-				    port_has_prefix (p, "ardour:")) {
-					extra_system.push_back (p);
-				} else {
-					extra_other.push_back (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;
+                                }
+
+				/* can't use the audio engine for this as we are looking at non-Ardour ports */
+
+				jack_port_t* jp = jack_port_by_name (session->engine().jack(), p.c_str());
+				if (jp) {
+					DataType t (jack_port_type (jp));
+					if (t != DataType::NIL) {
+						if (port_has_prefix (p, "system:") || port_has_prefix (p, "alsa_pcm") || port_has_prefix (p, "ardour:")) {
+							extra_system[t].push_back (p);
+						} else {
+							extra_other[t].push_back (p);
+						}
+					}
 				}
 			}
 
@@ -374,28 +536,43 @@ PortGroupList::gather (ARDOUR::Session* session, bool inputs, bool allow_dups)
 		free (ports);
 	}
 
-	if (!extra_system.empty()) {
-		boost::shared_ptr<Bundle> b = make_bundle_from_ports (extra_system, inputs);
-		system->add_bundle (b);
+	for (DataType::iterator i = DataType::begin(); i != DataType::end(); ++i) {
+		if (!extra_system[*i].empty()) {
+			boost::shared_ptr<Bundle> b = make_bundle_from_ports (extra_system[*i], *i, inputs);
+			boost::shared_ptr<Bundle> bt = bundle_for_type (b, type);
+			if (bt->nchannels() != ChanCount::ZERO) {
+				system->add_bundle (bt);
+			}
+		}
+	}
+
+	for (DataType::iterator i = DataType::begin(); i != DataType::end(); ++i) {
+		if (!extra_other[*i].empty()) {
+			boost::shared_ptr<Bundle> b = make_bundle_from_ports (extra_other[*i], *i, inputs);
+			boost::shared_ptr<Bundle> bt = bundle_for_type (b, type);
+			if (bt->nchannels() != ChanCount::ZERO) {
+				other->add_bundle (bt);
+			}
+		}
 	}
 
-	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 (system);
+	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 (other);
+	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
+PortGroupList::make_bundle_from_ports (std::vector<std::string> const & p, ARDOUR::DataType type, bool inputs) const
 {
-	boost::shared_ptr<Bundle> b (new Bundle ("", _type, inputs));
+	boost::shared_ptr<Bundle> b (new Bundle ("", inputs));
 
 	std::string const pre = common_prefix (p);
 	if (!pre.empty()) {
@@ -403,7 +580,7 @@ PortGroupList::make_bundle_from_ports (std::vector<std::string> const & p, bool
 	}
 
 	for (uint32_t j = 0; j < p.size(); ++j) {
-		b->add_channel (p[j].substr (pre.length()));
+		b->add_channel (p[j].substr (pre.length()), type);
 		b->set_port (j, p[j]);
 	}
 
@@ -465,13 +642,7 @@ void
 PortGroupList::clear ()
 {
 	_groups.clear ();
-
-	for (std::vector<sigc::connection>::iterator i = _bundle_changed_connections.begin(); i != _bundle_changed_connections.end(); ++i) {
-		i->disconnect ();
-	}
-
-	_bundle_changed_connections.clear ();
-
+	_bundle_changed_connections.drop_connections ();
 	emit_changed ();
 }
 
@@ -488,10 +659,10 @@ PortGroupList::bundles () const
 	return _bundles;
 }
 
-uint32_t
+ChanCount
 PortGroupList::total_channels () const
 {
-	uint32_t n = 0;
+	ChanCount n;
 
 	for (PortGroupList::List::const_iterator i = begin(); i != end(); ++i) {
 		n += (*i)->total_channels ();
@@ -513,11 +684,8 @@ PortGroupList::add_group (boost::shared_ptr<PortGroup> g)
 {
 	_groups.push_back (g);
 
-	g->Changed.connect (sigc::mem_fun (*this, &PortGroupList::emit_changed));
-
-	_bundle_changed_connections.push_back (
-		g->BundleChanged.connect (sigc::mem_fun (*this, &PortGroupList::emit_bundle_changed))
-		);
+	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 ();
 }
@@ -592,10 +760,37 @@ bool
 PortGroupList::empty () const
 {
 	List::const_iterator i = _groups.begin ();
-	while (i != _groups.end() && (*i)->total_channels() == 0) {
+	while (i != _groups.end() && (*i)->total_channels() == ChanCount::ZERO) {
 		++i;
 	}
 
 	return (i == _groups.end());
 }
 
+/** Take a bundle, and either return it, if it contains only ports of type \a t,
+ *  or return a new bundle with those ports from \a b which are of type \a t.
+ *  Note that t == NIL is taken to mean "all types".
+ */
+boost::shared_ptr<Bundle>
+PortGroupList::bundle_for_type (boost::shared_ptr<Bundle> b, DataType t) const
+{
+	/* We are asked for a bundle with all types, so that's easy */
+	if (t == DataType::NIL) {
+		return b;
+	}
+
+	if (b->nchannels().get(t) == b->nchannels().n_total()) {
+		/* All channels on b are of the correct type, so just return b */
+		return b;
+	}
+
+	/* We must build a new bundle */
+	boost::shared_ptr<Bundle> n (new ARDOUR::Bundle (b->name(), b->ports_are_inputs()));
+	for (uint32_t i = 0; i < b->nchannels().n_total(); ++i) {
+		if (b->channel_type(i) == t) {
+			n->add_channel (b->channel_name (i), t, b->channel_ports (i));
+		}
+	}
+
+	return n;
+}