slightly less incomplete work on mixer window treeview port

git-svn-id: svn://localhost/trunk/ardour2@46 d708f5d6-7413-0410-9779-e7cbd77b26cf

6 files changed, 198 insertions, 178 deletions

diff --git a/gtk2_ardour/io_selector.h b/gtk2_ardour/io_selector.h
index 9485eacd02..abc902ba4c 100644
--- a/gtk2_ardour/io_selector.h
+++ b/gtk2_ardour/io_selector.h
@@ -85,13 +85,13 @@ class IOSelector : public Gtk::VBox {
 	Gtk::ScrolledWindow port_display_scroller;
 
 	PBD::Lock port_display_lock;
-	slist<Gtk::CList *> port_displays;
+	slist<Gtk::TreeView *> port_displays;
 	void display_ports ();
 
 	void rescan ();
 	void clear_connections ();
 
-	void port_selection_handler (gint row, gint col, GdkEvent *ev, Gtk::CList *);
+	void port_selection_changed();
 
 	void ports_changed (ARDOUR::IOChange, void *);
 	void name_changed (void*);
@@ -100,10 +100,10 @@ class IOSelector : public Gtk::VBox {
 	void remove_port ();
 	gint remove_port_when_idle (ARDOUR::Port *);
 
-	gint port_column_button_release (GdkEventButton*, Gtk::CList*);
-	gint connection_click (GdkEventButton *, Gtk::CList *clist);
+	gint port_column_button_release (GdkEventButton*, Gtk::TreeView*);
+	gint connection_click (GdkEventButton *, Gtk::TreeView*);
 	
-	void select_clist(Gtk::CList*);
+	void select_clist(Gtk::TreeView*);
 	void select_next_clist ();
 };
 
diff --git a/gtk2_ardour/mixer_strip.h b/gtk2_ardour/mixer_strip.h
index 93538265fc..6e36573929 100644
--- a/gtk2_ardour/mixer_strip.h
+++ b/gtk2_ardour/mixer_strip.h
@@ -138,7 +138,7 @@ class MixerStrip : public RouteUI, public Gtk::EventBox
 	Gtk::Button   name_button;
 
 	Gtk::Window*  comment_window;
-	Gtk::Text     comment_area;
+	Gtk::TextView comment_area;
 	Gtk::Button   comment_button;
 
 	void setup_comment_editor ();
diff --git a/gtk2_ardour/mixer_ui.cc b/gtk2_ardour/mixer_ui.cc
index 9015c1da1d..2d25126757 100644
--- a/gtk2_ardour/mixer_ui.cc
+++ b/gtk2_ardour/mixer_ui.cc
@@ -48,103 +48,81 @@ using namespace Gtk;
 using namespace Gtkmm2ext;
 using namespace sigc;
 
-static const gchar* group_list_titles[] = { 
-	N_("***"), 
-	N_("Bar"),
-	0
-};
-
-GdkPixmap* Mixer_UI::check_pixmap = 0;
-GdkBitmap* Mixer_UI::check_mask = 0;
-GdkPixmap* Mixer_UI::empty_pixmap = 0;
-GdkBitmap* Mixer_UI::empty_mask = 0;
-
-
 Mixer_UI::Mixer_UI (AudioEngine& eng)
-	: Gtk::Window (GTK_WINDOW_TOPLEVEL),
+	: Window (Gtk::WINDOW_TOPLEVEL),
 	  KeyboardTarget (*this, "mixer"),
-	  engine (eng),
+	  engine (eng)
 {
 	_strip_width = Wide;
 	track_menu = 0;
 
-	check_pixmap = gdk_pixmap_colormap_create_from_xpm_d (NULL, 
-		      gtk_widget_get_colormap (GTK_WIDGET(group_list.gobj())),
-		      &check_mask, NULL, (gchar **) check_xpm);
-	empty_pixmap = gdk_pixmap_colormap_create_from_xpm_d (NULL, 
-		      gtk_widget_get_colormap (GTK_WIDGET(group_list.gobj())),
-		      &empty_mask, NULL, (gchar **) empty_xpm);
-
 	XMLNode* node = ARDOUR_UI::instance()->mixer_settings();
 	set_state (*node);
 
-
- 	scroller_base.signal_add_event()s (Gdk::BUTTON_PRESS_MASK|Gdk::BUTTON_RELEASE_MASK);
+ 	scroller_base.add_events (Gdk::BUTTON_PRESS_MASK|Gdk::BUTTON_RELEASE_MASK);
  	scroller_base.set_name ("MixerWindow");
  	scroller_base.signal_button_release_event().connect (mem_fun(*this, &Mixer_UI::strip_scroller_button_release));
 	// add as last item of strip packer
 	strip_packer.pack_end (scroller_base, true, true);
 
-	scroller.add_with_viewport (strip_packer);
+	scroller.add (strip_packer);
 	scroller.set_policy (Gtk::POLICY_AUTOMATIC, Gtk::POLICY_AUTOMATIC);
 
 	track_display_model = ListStore::create (track_display_columns);
 	track_display.set_model (track_display_model);
-	track_display.append_column (_("Strips"));
+	track_display.append_column (_("Strips"), track_display_columns.text);
 	track_display.set_name (X_("MixerTrackDisplayList"));
-	track_display.set_selection_mode (GTK_SELECTION_MULTIPLE);
+	track_display.get_selection()->set_mode (Gtk::SELECTION_MULTIPLE);
 	track_display.set_reorderable (true);
 	track_display.set_size_request (75, -1);
 	track_display.set_headers_visible (true);
 	track_display.set_headers_clickable (true);
-	track_display_scroller.add (track_display_list);
+	track_display_scroller.add (track_display);
 	track_display_scroller.set_policy (Gtk::POLICY_NEVER, Gtk::POLICY_AUTOMATIC);
 
 	group_display_model = ListStore::create (group_display_columns);
 	group_display.set_model (group_display_model);
-	group_display.append_column (_("Mix Groups"));
+	group_display.append_column (_("active"), group_display_columns.active);
+	group_display.append_column (_("groupname"), group_display_columns.text);
+	group_display.get_column (0)->set_data (X_("colnum"), GUINT_TO_POINTER(0));
+	group_display.get_column (1)->set_data (X_("colnum"), GUINT_TO_POINTER(1));
+
+	/* use checkbox for the active column */
+
+	CellRendererToggle *active_cell = dynamic_cast<CellRendererToggle*>(group_display.get_column_cell_renderer (0));
+	active_cell->property_activatable() = true;
+	active_cell->property_radio() = false;
+	
 	group_display.set_name ("MixerGroupList");
 	group_display.set_reorderable (true);
 	group_display.set_size_request (true);
-	group_display.set_headers_visible (true);
-	group_display.set_headers_clickable (true);
-	group_display_scroller.add (group_list);
+	group_display_scroller.add (group_display);
 	group_display_scroller.set_policy (Gtk::POLICY_NEVER, Gtk::POLICY_AUTOMATIC);
 
 	snapshot_display_model = ListStore::create (group_display_columns);
 	snapshot_display.set_model (snapshot_display_model);
-	snapshot_display.append_column (_("Mix Groups"));
+	snapshot_display.append_column (X_("mixgroups"), snapshot_display_columns.visible_name);
 	snapshot_display.set_name ("MixerSnapshotDisplayList");
 	snapshot_display.set_size_request (75, -1);
 	snapshot_display.set_reorderable (true);
 	snapshot_display_scroller.add (snapshot_display);
 	snapshot_display_scroller.set_policy (Gtk::POLICY_NEVER, Gtk::POLICY_AUTOMATIC);
 
-	group_list_vbox.pack_start (group_list_button, false, false);
-	group_list_vbox.pack_start (group_list_scroller, true, true);
-
-	list<string> stupid_list;
-
-	stupid_list.push_back ("");
-	stupid_list.push_back (_("-all-"));
-
-	group_list.rows().push_back (stupid_list);
-	group_list.rows().back().set_data (0);
-	group_list.rows().back().select();
-	group_list.cell(0,0).set_pixmap (check_pixmap);
+	group_display_vbox.pack_start (group_display_button, false, false);
+	group_display_vbox.pack_start (group_display_scroller, true, true);
 
 	track_display_frame.set_name("BaseFrame");
 	track_display_frame.set_shadow_type (Gtk::SHADOW_IN);
 	track_display_frame.add(track_display_scroller);
 
-	group_list_frame.set_name ("BaseFrame");
-	group_list_frame.set_shadow_type (Gtk::SHADOW_IN);
-	group_list_frame.add (group_list_vbox);
+	group_display_frame.set_name ("BaseFrame");
+	group_display_frame.set_shadow_type (Gtk::SHADOW_IN);
+	group_display_frame.add (group_display_vbox);
 
 	rhs_pane1.add1 (track_display_frame);
 	rhs_pane1.add2 (rhs_pane2);
 
-	rhs_pane2.add1 (group_list_frame);
+	rhs_pane2.add1 (group_display_frame);
 	rhs_pane2.add2 (snapshot_display_scroller);
 
 	list_vpacker.pack_start (rhs_pane1, true, true);
@@ -353,7 +331,7 @@ Mixer_UI::disconnect_from_session ()
 {
 	ENSURE_GUI_THREAD(mem_fun(*this, &Mixer_UI::disconnect_from_session));
 	
-	group_list.clear ();
+	group_display.clear ();
 	set_title (_("ardour: mixer"));
 	stop_updating ();
 	hide_all_strips (false);
@@ -515,24 +493,29 @@ Mixer_UI::fast_update_strips ()
 }
 
 void
-Mixer_UI::snapshot_display_selected (gint row, gint col, GdkEvent* ev)
+Mixer_UI::snapshot_display_selection_changed (gint row, gint col, GdkEvent* ev)
 {
-	string* snap_name;
+	TreeModel::iterator i = snapshot_display.get_selection()->get_selected();
+
+	string snap_name = (*i)[snapshot_display_columns.real_name]
 	
-	if ((snap_name = (string*) snapshot_display.get_row_data(row)) != 0){
-		if (session->snap_name() == *snap_name){
-			return;
-		}
-		string path = session->path();
-		ARDOUR_UI::instance()->load_session(path, *snap_name);
+	if (session->snap_name() == snap_name) {
+		return;
 	}
+	
+	ARDOUR_UI::instance()->load_session(session->path(), snap_name);
+}
+
+bool
+Mixer_UI::snapshot_display_button_press (GdkEventButton* ev)
+{
+	return false;
 }
 
 void
 Mixer_UI::track_display_selection_changed ()
 {
 	MixerStrip* strip;
-
 	TreeModel::Children rows = track_display_model->children();
 	TreeModel::Children::iterator i;
 	Glib::RefPtr<TreeViewSelection> selection = track_display.get_selection();
@@ -551,23 +534,33 @@ Mixer_UI::track_display_selection_changed ()
 }
 
 void
-Mixer_UI::unselect_strip_in_display (MixerStrip *strip)
+Mixer_UI::select_strip_op (MixerStrip *strip, bool yn)
 {
-	CList_Helpers::RowIterator i;
-
-	if ((i = track_display_list.rows().find_data (strip)) != track_display_list.rows().end()) {
-		(*i).unselect ();
+	TreeModel::Children rows = track_display_model->children();
+	TreeModel::Children::iterator i;
+	Glib::RefPtr<TreeViewSelection> selection = track_display.get_selection();
+	
+	for (i = rows.begin(); i != rows.end(); ++i) {
+		if ((*i)[rows.data] == strip) {
+			if (yn) {
+				track_display.get_selection()->select (*i);
+			} else {
+				track_display.get_selection()->unselect (*i);
+			}
+			break;
+		}
 	}
 }
 
 void
+Mixer_UI::unselect_strip_in_display (MixerStrip *strip)
+{
+	select_strip_op (strip, true);
+}
+void
 Mixer_UI::select_strip_in_display (MixerStrip *strip)
 {
-	CList_Helpers::RowIterator i;
-
-	if ((i = track_display_list.rows().find_data (strip)) != track_display_list.rows().end()) {
-		(*i).select ();
-	}
+	select_strip_op (strip, false);
 }
 
 void
@@ -585,13 +578,14 @@ Mixer_UI::queue_track_display_reordered (gint arg1, gint arg2)
 int
 Mixer_UI::track_display_reordered ()
 {
-	CList_Helpers::RowList::iterator i;
+	TreeModel::Children rows = track_display_model->children();
+	TreeModel::Children::iterator i;
+	Glib::RefPtr<TreeViewSelection> selection = track_display.get_selection();
 	long order;
+	
+	for (order = 0, i = rows.begin(); i != rows.end(); ++i, ++order) {
+		MixerStrip* strip = (*i)[columns.data];
 
-	// hide_all_strips (false);
-
-	for (order = 0, i  = track_display_list.rows().begin(); i != track_display_list.rows().end(); ++i, ++order) {
-		MixerStrip* strip = (MixerStrip *) (*i)->get_data ();
 		if (strip->marked_for_display()) {
 			strip->route().set_order_key (N_("signal"), order);
 			strip_packer.reorder_child (*strip, -1); /* put at end */
@@ -636,14 +630,18 @@ Mixer_UI::strip_name_changed (void* src, MixerStrip* mx)
 {
 	ENSURE_GUI_THREAD(bind (mem_fun(*this, &Mixer_UI::strip_name_changed), src, mx));
 	
-	CList_Helpers::RowList::iterator i;
-
-	if ((i = track_display_list.rows().find_data (mx)) == track_display_list.rows().end()) {
-		error << _("track display list item for renamed strip not found!") << endmsg;
-		return;
-	}
+	TreeModel::Children rows = track_display_model->children();
+	TreeModel::Children::iterator i;
+	Glib::RefPtr<TreeViewSelection> selection = track_display.get_selection();
+	
+	for (i = rows.begin(); i != rows.end(); ++i) {
+		if ((*i)[columns.data] == mx) {
+			(*i)[columns.text] = mx->route().name();
+			return;
+		}
+	} 
 
-	track_display_list.cell ((*i)->get_row_num(), 0).set_text (mx->route().name());
+	error << _("track display list item for renamed strip not found!") << endmsg;
 }
 
 void
@@ -671,81 +669,76 @@ Mixer_UI::new_mix_group ()
 }
 
 void
-Mixer_UI::group_list_button_clicked ()
+Mixer_UI::group_display_button_clicked ()
 {
 	if (session) {
 		new_mix_group ();
 	}
 }
 
-gint
-Mixer_UI::group_list_button_press_event (GdkEventButton* ev)
+bool
+Mixer_UI::group_display_button_press (GdkEventButton* ev)
 {
 	gint row, col;
 	RouteGroup* group;
 
-	if (group_list.get_selection_info ((int)ev->x, (int)ev->y, &row, &col) == 0) {
-		return FALSE;
+	TreeIter iter;
+	TreeModel::Path path;
+	TreeViewColumn* column;
+	int cellx;
+	int celly;
+
+	if (!group_display.get_path_at_pos ((int)ev->x, (int)ev->y, path, column, cellx, celly)) {
+		return false;
 	}
+	
+	switch (GPOINTER_TO_UINT (column.get_data (X_("colnum")))) {
+	case 0:
+		/* active column click */
+		
+		if ((iter = group_display.get_iter (path))) {
+			/* path points to a valid node */
+			if ((group = (*iter)[group_display_columns.group]) != 0) {
+				group->set_active (!group->is_active (), this);
+			}
+		}
+		break;
 
-	switch (col) {
 	case 1:
 		if (Keyboard::is_edit_event (ev)) {
-			// RouteGroup* group = (RouteGroup *) group_list.row(row).get_data ();
-
+			// RouteGroup* group = (RouteGroup *) group_display.row(row).get_data ();
 			// edit_mix_group (group);
 
-			return stop_signal (group_list, "button_press_event");
-
 		} else {
 			/* allow regular select to occur */
-			return FALSE;
-		}
-
-	case 0:
-		stop_signal (group_list, "button_press_event");
-		if ((group = (RouteGroup *) group_list.row(row).get_data ()) != 0) {
-			group->set_active (!group->is_active (), this);
+			return false;
 		}
 		break;
 	}
 		
-	return TRUE;
+	return true;
 }
 
 void
-Mixer_UI::group_selected (gint row, gint col, GdkEvent* ev)
+Mixer_UI::group_selection_changed ()
 {
-	RouteGroup* group = (RouteGroup *) group_list.row(row).get_data ();
-
-	if (group) {
-		group->set_hidden (false, this);
-	} else {
-		/* the master group */
-
-		for (list<MixerStrip *>::iterator i = strips.begin(); i != strips.end(); ++i) {
-			show_strip (*i);
-		}
+	TreeModel::iterator i;
+	TreeModel::Children rows = group_display_model->children();
+	TreeViewSelection* selection = group_display.get_selection();
 
-		track_display_reordered ();
-	}
-}
-
-void
-Mixer_UI::group_unselected (gint row, gint col, GdkEvent* ev)
-
-{
-	RouteGroup* group = (RouteGroup *) group_list.row(row).get_data ();
+	for (i = rows.begin(); i != rows.end(); ++i) {
+		RouteGroup* group;
 
-	if (group) {
-		group->set_hidden (true, this);
-	} else {
-		/* the master group */
+		group = (*iter)[group_display_columns.group];
 
-		for (list<MixerStrip *>::iterator i = strips.begin(); i != strips.end(); ++i) {
-			hide_strip (*i);
+		if (selection->is_selected (i)) {
+			group->set_hidden (true, this);
+		} else {
+			group->set_hidden (true, this);
 		}
 	}
+
+	track_display_reordered ();
 }
 
 void
@@ -757,24 +750,38 @@ Mixer_UI::group_flags_changed (void* src, RouteGroup* group)
 		// select row
 	}
 
-	CList_Helpers::RowIterator ri = group_list.rows().find_data (group);
+	TreeModel::Children rows;
+	TreeModel::Children::iterator gi;
+	TreeModel::Children::iterator ti;
+	MixGroup* group;
+
+	CList_Helpers::RowIterator ri = group_display.rows().find_data (group);
+
+	rows = group_display_model->children();
+
+	for (gi = rows.begin(); gi != grows.end(); ++gi) {
+		if ((*gi)[columns.data] == group) {
+			break;
+		}
+	}
+
+	if (gi == rows.end()) {
+		return;
+	}
+		
+	rows = track_display_model->children();
 
 	for (list<MixerStrip *>::iterator i = strips.begin(); i != strips.end(); ++i) {
 		if ((*i)->route().mix_group() == group) {
 			if (group->is_hidden ()) {
 				unselect_strip_in_display(*i);
+				group_display.get_selection()->unselect (*gi);
 			} else {
 				select_strip_in_display(*i);
-				(*ri)->select();
+				group_display.get_selection()->select (*gi);
 			}
 		}
 	}
-
-	if (group->is_active()) {
-		group_list.cell (ri->get_row_num(),0).set_pixmap (check_pixmap, check_mask);
-	} else {
-		group_list.cell (ri->get_row_num(),0).set_pixmap (empty_pixmap, empty_mask);
-	}
 }
 
 void
@@ -815,8 +822,8 @@ Mixer_UI::redisplay_snapshots ()
 	for (vector<string*>::iterator i = states->begin(); i != states->end(); ++i) {
 		string statename = *(*i);
 		row = *(snapshot_display_model->append());
-		row[visible] = statename;
-		row[hidden] = statename;
+		row[snapshot_columns.visible_name] = statename;
+		row[snapshot_columns.real_name] = statename;
 	}
 
 	delete states;
diff --git a/gtk2_ardour/mixer_ui.h b/gtk2_ardour/mixer_ui.h
index d8600c19c7..504074642b 100644
--- a/gtk2_ardour/mixer_ui.h
+++ b/gtk2_ardour/mixer_ui.h
@@ -88,14 +88,14 @@ class Mixer_UI : public Gtk::Window, public KeyboardTarget
 	Gtk::HBox                scroller_hpacker;
 	Gtk::VBox                mixer_scroller_vpacker;
 	Gtk::VBox                list_vpacker;
-	Gtk::Label               group_list_button_label;
-	Gtk::Button              group_list_button;
+	Gtk::Label               group_display_button_label;
+	Gtk::Button              group_display_button;
 	Gtk::ScrolledWindow      track_display_scroller;
-	Gtk::ScrolledWindow      group_list_scroller;
+	Gtk::ScrolledWindow      group_display_scroller;
 	Gtk::ScrolledWindow      snapshot_display_scroller;
-	Gtk::VBox		 group_list_vbox;
+	Gtk::VBox		 group_display_vbox;
 	Gtk::Frame 		 track_display_frame;
-	Gtk::Frame		 group_list_frame;
+	Gtk::Frame		 group_display_frame;
 	Gtk::VPaned		 rhs_pane1;
 	Gtk::VPaned		 rhs_pane2;
 	Gtk::HBox                strip_packer;
@@ -122,6 +122,8 @@ class Mixer_UI : public Gtk::Window, public KeyboardTarget
 	void unselect_all_audiobus_strips ();
 	void select_all_audiobus_strips ();
 
+	void select_strip_op (bool);
+
 	void follow_strip_selection ();
 
 	gint start_updating ();
@@ -144,8 +146,7 @@ class Mixer_UI : public Gtk::Window, public KeyboardTarget
 
 	void group_selected (gint row, gint col, GdkEvent *ev);
 	void group_unselected (gint row, gint col, GdkEvent *ev);
-	gint group_list_button_press_event (GdkEventButton *);
-	void group_list_button_clicked();
+	void group_display_active_clicked();
 	void new_mix_group ();
 	void add_mix_group (ARDOUR::RouteGroup *);
 
@@ -166,27 +167,38 @@ class Mixer_UI : public Gtk::Window, public KeyboardTarget
 
 	/* various treeviews */
 	
-	template<class T> struct TextDataModelColumns : public Gtk::TreeModel::ColumnRecord {
-	    TrackListModelColumns() { 
+	struct TrackDisplayModelColumns : public Gtk::TreeModel::ColumnRecord {
+	    TrackDisplayModelColumns() { 
+		    add (text);
+		    add (data);
+	    }
+	    Gtk::TreeModelColumn<Glib::ustring> text;
+	    Gtk::TreeModelColumn<ARDOUR::Route*> data;
+	};
+
+	struct GroupDisplayModelColumns : public Gtk::TreeModel::ColumnRecord {
+	    GroupDisplayModelColumns() { 
+		    add (active);
 		    add (text);
 		    add (data);
 	    }
-	    Gtk::TreeModelColumn<std::string> text;
-	    Gtk::TreeModelColumn<T*> data;
+	    Gtk::TreeModelColumn<bool>                active;
+	    Gtk::TreeModelColumn<Glib::ustring>       text;
+	    Gtk::TreeModelColumn<ARDOUR::RouteGroup*> data;
 	};
 
-	struct TextPairModelColumns : public Gtk::TreeModel::ColumnRecord {
-	    TrackListModelColumns() { 
-		    add (visible);
-		    add (hidden);
+	struct SnapshotDisplayModelColumns : public Gtk::TreeModel::ColumnRecord {
+	    SnapshotDisplayModelColumns() { 
+		    add (visible_name);
+		    add (real_name);
 	    }
-	    Gtk::TreeModelColumn<std::string> visible;
-	    Gtk::TreeModelColumn<std::string> hidden;
+	    Gtk::TreeModelColumn<Glib::ustring> visible_name;
+	    Gtk::TreeModelColumn<Glib::ustring> real_name;
 	};
 
-	TextDataModelColumns<ARDOUR::Route>                track_display_columns;
-	TextDataGroupListModelColumns<ARDOUR::RouteGroup>  group_list_columns;
-	TextPairModelColumns                               snapshot_display_columns;
+	TrackDisplayModelColumns    track_display_columns;
+	GroupDisplayModelColumns    group_display_columns;
+	SnapshotDisplayModelColumns snapshot_display_columns;
 
 	Gtk::TreeView track_display;
 	Gtk::TreeView group_display;
diff --git a/gtk2_ardour/plugin_selector.h b/gtk2_ardour/plugin_selector.h
index 22aa2a872b..cc482ffbf0 100644
--- a/gtk2_ardour/plugin_selector.h
+++ b/gtk2_ardour/plugin_selector.h
@@ -20,8 +20,9 @@
 #ifndef __ardour_plugin_selector_h__
 #define __ardour_plugin_selector_h__
 
-#include <gtkmm.h>
-#include <gtkmm/ctree.h>
+#include <gtkmm/notebook.h>
+#include <gtkmm/notebook.h>
+#include <gtkmm/treeview.h>
 #include <gtkmm2ext/selector.h>
 
 #include <ardour_dialog.h>
@@ -65,19 +66,19 @@ class PluginSelector : public ArdourDialog
 	ARDOUR::PluginManager *manager;
 	list<ARDOUR::PluginInfo*> added_plugins;
 
-	static void _input_refiller (Gtk::CList &, void *);
-	static void _output_refiller (Gtk::CList &, void *);
-
-	void input_refiller (Gtk::CList &);
-	void output_refiller (Gtk::CList &);
-	void i_plugin_selected (Gtkmm2ext::Selector *selector,
-			      Gtkmm2ext::SelectionResult *res);
-	void i_plugin_chosen (Gtkmm2ext::Selector *selector,
-			    Gtkmm2ext::SelectionResult *res);
-	void o_plugin_selected (Gtkmm2ext::Selector *selector,
-			      Gtkmm2ext::SelectionResult *res);
-	void o_plugin_chosen (Gtkmm2ext::Selector *selector,
-			    Gtkmm2ext::SelectionResult *res);
+	//static void _input_refiller (Gtk::CList &, void *);
+	//static void _output_refiller (Gtk::CList &, void *);
+
+	//void input_refiller (Gtk::CList &);
+	//void output_refiller (Gtk::CList &);
+	//void i_plugin_selected (Gtkmm2ext::Selector *selector,
+	//			Gtkmm2ext::SelectionResult *res);
+        //void i_plugin_chosen (Gtkmm2ext::Selector *selector,
+	//		    Gtkmm2ext::SelectionResult *res);
+	//void o_plugin_selected (Gtkmm2ext::Selector *selector,
+	//		      Gtkmm2ext::SelectionResult *res);
+	//void o_plugin_chosen (Gtkmm2ext::Selector *selector,
+	//		    Gtkmm2ext::SelectionResult *res);
 	
 	void btn_add_clicked();
 	void btn_remove_clicked();
diff --git a/gtk2_ardour/redirect_box.h b/gtk2_ardour/redirect_box.h
index 913990e3ac..55769cd4e1 100644
--- a/gtk2_ardour/redirect_box.h
+++ b/gtk2_ardour/redirect_box.h
@@ -94,7 +94,7 @@ class RedirectBox : public Gtk::HBox
 	
 	Gtk::EventBox	    redirect_eventbox;
 	Gtk::HBox           redirect_hpacker;
-	Gtk::CList          redirect_display;
+	Gtk::TreeView       redirect_display;
 	Gtk::ScrolledWindow redirect_scroller;
 
 	Width _width;
@@ -111,8 +111,8 @@ class RedirectBox : public Gtk::HBox
 	vector<Gtk::MenuItem*> selection_dependent_items;
 	Gtk::MenuItem* redirect_paste_item;
 	gint redirect_menu_map_handler (GdkEventAny *ev);
-	Gtk::Menu * build_redirect_menu (Gtk::CList&);
-	void build_redirect_tooltip (Gtk::CList&, Gtk::EventBox&, string);
+	Gtk::Menu * build_redirect_menu (Gtk::TreeView&);
+	void build_redirect_tooltip (Gtk::TreeView&, Gtk::EventBox&, string);
 	void show_redirect_menu (gint arg);
 
 	void choose_send ();