changes to help strp silence
[ardour.git] / gtk2_ardour / mixer_ui.cc
index 0c88b7b2db09600108069ae7089d73259cda6017..4b4a5b41b18fa135cc9b26ca5dd41c748e3a152f 100644 (file)
 */
 
 #include <algorithm>
+#include <map>
 #include <sigc++/bind.h>
 
 #include <gtkmm/accelmap.h>
 
 #include "pbd/convert.h"
+#include "pbd/stacktrace.h"
 #include <glibmm/thread.h>
 
 #include <gtkmm2ext/gtk_ui.h>
@@ -55,16 +57,13 @@ using namespace PBD;
 using namespace Gtk;
 using namespace Glib;
 using namespace Gtkmm2ext;
-using namespace sigc;
 using namespace std;
 
 using PBD::atoi;
 
 Mixer_UI::Mixer_UI ()
        : Window (Gtk::WINDOW_TOPLEVEL)
-       , _plugin_selector (0)
 {
-       session = 0;
        _strip_width = Config->get_default_narrow_ms() ? Narrow : Wide;
        track_menu = 0;
        route_group_context_menu = 0;
@@ -75,11 +74,11 @@ Mixer_UI::Mixer_UI ()
        strip_redisplay_does_not_sync_order_keys = false;
        ignore_sync = false;
 
-       Route::SyncOrderKeys.connect (mem_fun (*this, &Mixer_UI::sync_order_keys));
+       Route::SyncOrderKeys.connect (*this, ui_bind (&Mixer_UI::sync_order_keys, this, _1), gui_context());
 
        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));
+       scroller_base.signal_button_release_event().connect (sigc::mem_fun(*this, &Mixer_UI::strip_scroller_button_release));
        // add as last item of strip packer
        strip_packer.pack_end (scroller_base, true, true);
 
@@ -105,15 +104,15 @@ Mixer_UI::Mixer_UI ()
        track_display.set_reorderable (true);
        track_display.set_headers_visible (true);
 
-       track_model->signal_row_deleted().connect (mem_fun (*this, &Mixer_UI::track_list_delete));
-       track_model->signal_row_changed().connect (mem_fun (*this, &Mixer_UI::track_list_change));
-       track_model->signal_rows_reordered().connect (mem_fun (*this, &Mixer_UI::track_list_reorder));
+       track_model->signal_row_deleted().connect (sigc::mem_fun (*this, &Mixer_UI::track_list_delete));
+       track_model->signal_row_changed().connect (sigc::mem_fun (*this, &Mixer_UI::track_list_change));
+       track_model->signal_rows_reordered().connect (sigc::mem_fun (*this, &Mixer_UI::track_list_reorder));
 
        CellRendererToggle* track_list_visible_cell = dynamic_cast<CellRendererToggle*>(track_display.get_column_cell_renderer (1));
        track_list_visible_cell->property_activatable() = true;
        track_list_visible_cell->property_radio() = false;
 
-       track_display.signal_button_press_event().connect (mem_fun (*this, &Mixer_UI::track_display_button_press), false);
+       track_display.signal_button_press_event().connect (sigc::mem_fun (*this, &Mixer_UI::track_display_button_press), false);
 
        track_display_scroller.add (track_display);
        track_display_scroller.set_policy (Gtk::POLICY_NEVER, Gtk::POLICY_AUTOMATIC);
@@ -136,7 +135,7 @@ Mixer_UI::Mixer_UI ()
 
        CellRendererText* name_cell = dynamic_cast<CellRendererText*>(group_display.get_column_cell_renderer (0));
        name_cell->property_editable() = true;
-       name_cell->signal_edited().connect (mem_fun (*this, &Mixer_UI::route_group_name_edit));
+       name_cell->signal_edited().connect (sigc::mem_fun (*this, &Mixer_UI::route_group_name_edit));
 
        /* use checkbox for the active column */
 
@@ -144,9 +143,9 @@ Mixer_UI::Mixer_UI ()
        active_cell->property_activatable() = true;
        active_cell->property_radio() = false;
 
-       group_model->signal_row_changed().connect (mem_fun (*this, &Mixer_UI::route_group_row_change));
+       group_model->signal_row_changed().connect (sigc::mem_fun (*this, &Mixer_UI::route_group_row_change));
 
-       group_display.signal_button_press_event().connect (mem_fun (*this, &Mixer_UI::group_display_button_press), false);
+       group_display.signal_button_press_event().connect (sigc::mem_fun (*this, &Mixer_UI::group_display_button_press), false);
 
        group_display_scroller.add (group_display);
        group_display_scroller.set_policy (Gtk::POLICY_NEVER, Gtk::POLICY_AUTOMATIC);
@@ -168,8 +167,8 @@ Mixer_UI::Mixer_UI ()
 
        route_group_display_button_box->set_homogeneous (true);
 
-       route_group_add_button->signal_clicked().connect (mem_fun (*this, &Mixer_UI::new_route_group));
-       route_group_remove_button->signal_clicked().connect (mem_fun (*this, &Mixer_UI::remove_selected_route_group));
+       route_group_add_button->signal_clicked().connect (sigc::mem_fun (*this, &Mixer_UI::new_route_group));
+       route_group_remove_button->signal_clicked().connect (sigc::mem_fun (*this, &Mixer_UI::remove_selected_route_group));
 
        route_group_display_button_box->add (*route_group_remove_button);
        route_group_display_button_box->add (*route_group_add_button);
@@ -200,9 +199,9 @@ Mixer_UI::Mixer_UI ()
        list_hpane.add1(list_vpacker);
        list_hpane.add2(global_hpacker);
 
-       rhs_pane1.signal_size_allocate().connect (bind (mem_fun(*this, &Mixer_UI::pane_allocation_handler),
+       rhs_pane1.signal_size_allocate().connect (sigc::bind (sigc::mem_fun(*this, &Mixer_UI::pane_allocation_handler),
                                                        static_cast<Gtk::Paned*> (&rhs_pane1)));
-       list_hpane.signal_size_allocate().connect (bind (mem_fun(*this, &Mixer_UI::pane_allocation_handler),
+       list_hpane.signal_size_allocate().connect (sigc::bind (sigc::mem_fun(*this, &Mixer_UI::pane_allocation_handler),
                                                         static_cast<Gtk::Paned*> (&list_hpane)));
 
        global_vpacker.pack_start (list_hpane, true, true);
@@ -218,12 +217,12 @@ Mixer_UI::Mixer_UI ()
 
        add_accel_group (ActionManager::ui_manager->get_accel_group());
 
-       signal_delete_event().connect (mem_fun (*this, &Mixer_UI::hide_window));
+       signal_delete_event().connect (sigc::mem_fun (*this, &Mixer_UI::hide_window));
        add_events (Gdk::KEY_PRESS_MASK|Gdk::KEY_RELEASE_MASK);
 
-       signal_configure_event().connect (mem_fun (*ARDOUR_UI::instance(), &ARDOUR_UI::configure_handler));
+       signal_configure_event().connect (sigc::mem_fun (*ARDOUR_UI::instance(), &ARDOUR_UI::configure_handler));
 
-       _selection.RoutesChanged.connect (mem_fun(*this, &Mixer_UI::follow_strip_selection));
+       _selection.RoutesChanged.connect (sigc::mem_fun(*this, &Mixer_UI::follow_strip_selection));
 
        route_group_display_button_box->show();
        route_group_add_button->show();
@@ -251,6 +250,12 @@ Mixer_UI::Mixer_UI ()
        group_display.show();
 
        auto_rebinding = FALSE;
+
+       MixerStrip::CatchDeletion.connect (*this, ui_bind (&Mixer_UI::remove_strip, this, _1), gui_context());
+
+#ifndef DEFER_PLUGIN_SELECTOR_LOAD
+       _plugin_selector = new PluginSelector (PluginManager::the_manager ());
+#endif
 }
 
 Mixer_UI::~Mixer_UI ()
@@ -297,7 +302,7 @@ Mixer_UI::hide_window (GdkEventAny *ev)
 void
 Mixer_UI::add_strip (RouteList& routes)
 {
-       ENSURE_GUI_THREAD(bind (mem_fun(*this, &Mixer_UI::add_strip), routes));
+       ENSURE_GUI_THREAD (*this, &Mixer_UI::add_strip, routes)
 
        MixerStrip* strip;
 
@@ -311,7 +316,7 @@ Mixer_UI::add_strip (RouteList& routes)
                        return;
                }
 
-               strip = new MixerStrip (*this, *session, route);
+               strip = new MixerStrip (*this, _session, route);
                strips.push_back (strip);
 
                Config->get_default_narrow_ms() ? _strip_width = Narrow : _strip_width = Wide;
@@ -332,11 +337,10 @@ Mixer_UI::add_strip (RouteList& routes)
                        route->set_order_key (N_("signal"), track_model->children().size()-1);
                }
 
-               route->NameChanged.connect (bind (mem_fun(*this, &Mixer_UI::strip_name_changed), strip));
+               route->PropertyChanged.connect (*this, ui_bind (&Mixer_UI::strip_property_changed, this, _1, strip), gui_context());
 
-               strip->GoingAway.connect (bind (mem_fun(*this, &Mixer_UI::remove_strip), strip));
-               strip->WidthChanged.connect (mem_fun(*this, &Mixer_UI::strip_width_changed));
-               strip->signal_button_release_event().connect (bind (mem_fun(*this, &Mixer_UI::strip_button_release_event), strip));
+               strip->WidthChanged.connect (sigc::mem_fun(*this, &Mixer_UI::strip_width_changed));
+               strip->signal_button_release_event().connect (sigc::bind (sigc::mem_fun(*this, &Mixer_UI::strip_button_release_event), strip));
        }
 
        no_track_list_redisplay = false;
@@ -349,7 +353,14 @@ Mixer_UI::add_strip (RouteList& routes)
 void
 Mixer_UI::remove_strip (MixerStrip* strip)
 {
-       ENSURE_GUI_THREAD(bind (mem_fun(*this, &Mixer_UI::remove_strip), strip));
+       if (_session && _session->deletion_in_progress()) {
+               /* its all being taken care of */
+               return;
+       }
+
+       ENSURE_GUI_THREAD (*this, &Mixer_UI::remove_strip, strip);
+
+       cerr << "Mixer UI removing strip for " << strip << endl;
 
        TreeModel::Children rows = track_model->children();
        TreeModel::Children::iterator ri;
@@ -374,34 +385,42 @@ Mixer_UI::remove_strip (MixerStrip* strip)
 void
 Mixer_UI::sync_order_keys (string const & src)
 {
-       vector<int> neworder;
        TreeModel::Children rows = track_model->children();
        TreeModel::Children::iterator ri;
 
-       if (src == N_("signal") || !session || (session->state_of_the_state() & Session::Loading) || rows.empty()) {
+       if (src == N_("signal") || !_session || (_session->state_of_the_state() & (Session::Loading|Session::Deletion)) || rows.empty()) {
                return;
        }
 
-       for (ri = rows.begin(); ri != rows.end(); ++ri) {
-               neworder.push_back (0);
-       }
+       std::map<int,int> keys;
 
        bool changed = false;
-       unsigned int order;
 
-       for (order = 0, ri = rows.begin(); ri != rows.end(); ++ri, ++order) {
+       unsigned order = 0;
+       for (ri = rows.begin(); ri != rows.end(); ++ri, ++order) {
                boost::shared_ptr<Route> route = (*ri)[track_columns.route];
                unsigned int old_key = order;
                unsigned int new_key = route->order_key (N_("signal"));
 
-               assert (new_key < neworder.size());
-               neworder[new_key] = old_key;
+               keys[new_key] = old_key;
 
                if (new_key != old_key) {
                        changed = true;
                }
        }
 
+       if (keys.size() != rows.size()) {
+               PBD::stacktrace (cerr, 20);
+       }
+       assert(keys.size() == rows.size());
+
+       // Remove any gaps in keys caused by automation children tracks
+       vector<int> neworder;
+       for (std::map<int,int>::const_iterator i = keys.begin(); i != keys.end(); ++i) {
+               neworder.push_back(i->second);
+       }
+       assert(neworder.size() == rows.size());
+
        if (changed) {
                strip_redisplay_does_not_reset_order_keys = true;
                track_model->reorder (neworder);
@@ -442,14 +461,24 @@ Mixer_UI::strip_button_release_event (GdkEventButton *ev, MixerStrip *strip)
 }
 
 void
-Mixer_UI::connect_to_session (Session* sess)
+Mixer_UI::set_session (Session* sess)
 {
-       session = sess;
+       SessionHandlePtr::set_session (sess);
+
+       if (_plugin_selector) {
+               _plugin_selector->set_session (_session);
+       }
+
+       _group_tabs->set_session (sess);
+
+       if (!_session) {
+               return;
+       }
 
        XMLNode* node = ARDOUR_UI::instance()->mixer_settings();
        set_state (*node);
 
-       WindowTitle title(session->name());
+       WindowTitle title(_session->name());
        title += _("Mixer");
        title += Glib::get_application_name();
 
@@ -457,39 +486,41 @@ Mixer_UI::connect_to_session (Session* sess)
 
        initial_track_display ();
 
-       session->GoingAway.connect (mem_fun(*this, &Mixer_UI::disconnect_from_session));
-       session->RouteAdded.connect (mem_fun(*this, &Mixer_UI::add_strip));
-       session->route_group_added.connect (mem_fun(*this, &Mixer_UI::add_route_group));
-       session->route_group_removed.connect (mem_fun(*this, &Mixer_UI::route_groups_changed));
-       session->config.ParameterChanged.connect (mem_fun (*this, &Mixer_UI::parameter_changed));
+       _session->RouteAdded.connect (_session_connections, ui_bind (&Mixer_UI::add_strip, this, _1), gui_context());
+       _session->route_group_added.connect (_session_connections, ui_bind (&Mixer_UI::add_route_group, this, _1), gui_context());
+       _session->route_group_removed.connect (_session_connections, boost::bind (&Mixer_UI::route_groups_changed, this), gui_context());
+       _session->config.ParameterChanged.connect (_session_connections, ui_bind (&Mixer_UI::parameter_changed, this, _1), gui_context());
 
        route_groups_changed ();
 
-       if (_plugin_selector)
-               _plugin_selector->set_session (session);
-
        if (_visible) {
-              show_window();
+               show_window();
        }
 
-       _group_tabs->connect_to_session (sess);
-
        start_updating ();
 }
 
 void
-Mixer_UI::disconnect_from_session ()
+Mixer_UI::session_going_away ()
 {
-       ENSURE_GUI_THREAD(mem_fun(*this, &Mixer_UI::disconnect_from_session));
+       ENSURE_GUI_THREAD (*this, &Mixer_UI::session_going_away)
 
        group_model->clear ();
        _selection.clear ();
+       track_model->clear ();
+
+       for (list<MixerStrip *>::iterator i = strips.begin(); i != strips.end(); ++i) {
+               delete (*i);
+       }
+       strips.clear ();
 
        WindowTitle title(Glib::get_application_name());
        title += _("Mixer");
        set_title (title.get_string());
 
        stop_updating ();
+
+       SessionHandlePtr::session_going_away ();
 }
 
 void
@@ -527,7 +558,7 @@ Mixer_UI::hide_strip (MixerStrip* ms)
 gint
 Mixer_UI::start_updating ()
 {
-    fast_screen_update_connection = ARDOUR_UI::instance()->SuperRapidScreenUpdate.connect (mem_fun(*this, &Mixer_UI::fast_update_strips));
+    fast_screen_update_connection = ARDOUR_UI::instance()->SuperRapidScreenUpdate.connect (sigc::mem_fun(*this, &Mixer_UI::fast_update_strips));
     return 0;
 }
 
@@ -541,7 +572,7 @@ Mixer_UI::stop_updating ()
 void
 Mixer_UI::fast_update_strips ()
 {
-       if (is_mapped () && session) {
+       if (is_mapped () && _session) {
                for (list<MixerStrip *>::iterator i = strips.begin(); i != strips.end(); ++i) {
                        (*i)->fast_update ();
                }
@@ -660,7 +691,7 @@ void
 Mixer_UI::track_list_reorder (const TreeModel::Path&, const TreeModel::iterator&, int* /*new_order*/)
 {
        strip_redisplay_does_not_sync_order_keys = true;
-       session->set_remote_control_ids();
+       _session->set_remote_control_ids();
        redisplay_track_list ();
        strip_redisplay_does_not_sync_order_keys = false;
 }
@@ -670,7 +701,7 @@ Mixer_UI::track_list_change (const Gtk::TreeModel::Path&, const Gtk::TreeModel::
 {
        // never reset order keys because of a property change
        strip_redisplay_does_not_reset_order_keys = true;
-       session->set_remote_control_ids();
+       _session->set_remote_control_ids();
        redisplay_track_list ();
        strip_redisplay_does_not_reset_order_keys = false;
 }
@@ -679,8 +710,10 @@ void
 Mixer_UI::track_list_delete (const Gtk::TreeModel::Path&)
 {
        /* this could require an order sync */
-       session->set_remote_control_ids();
-       redisplay_track_list ();
+       if (_session && !_session->deletion_in_progress()) {
+               _session->set_remote_control_ids();
+               redisplay_track_list ();
+       }
 }
 
 void
@@ -747,10 +780,10 @@ Mixer_UI::redisplay_track_list ()
        }
 
        if (!strip_redisplay_does_not_reset_order_keys && !strip_redisplay_does_not_sync_order_keys) {
-               session->sync_order_keys (N_("signal"));
+               _session->sync_order_keys (N_("signal"));
        }
 
-       // Rebind all of the midi controls automatically
+       // Resigc::bind all of the midi controls automatically
 
        if (auto_rebinding)
                auto_rebind_midi_controls ();
@@ -877,7 +910,7 @@ struct SignalOrderRouteSorter {
 void
 Mixer_UI::initial_track_display ()
 {
-       boost::shared_ptr<RouteList> routes = session->get_routes();
+       boost::shared_ptr<RouteList> routes = _session->get_routes();
        RouteList copy (*routes);
        SignalOrderRouteSorter sorter;
 
@@ -962,19 +995,23 @@ Mixer_UI::build_track_menu ()
        track_menu->set_name ("ArdourContextMenu");
        MenuList& items = track_menu->items();
 
-       items.push_back (MenuElem (_("Show All"), mem_fun(*this, &Mixer_UI::show_all_routes)));
-       items.push_back (MenuElem (_("Hide All"), mem_fun(*this, &Mixer_UI::hide_all_routes)));
-       items.push_back (MenuElem (_("Show All Audio Tracks"), mem_fun(*this, &Mixer_UI::show_all_audiotracks)));
-       items.push_back (MenuElem (_("Hide All Audio Tracks"), mem_fun(*this, &Mixer_UI::hide_all_audiotracks)));
-       items.push_back (MenuElem (_("Show All Audio Busses"), mem_fun(*this, &Mixer_UI::show_all_audiobus)));
-       items.push_back (MenuElem (_("Hide All Audio Busses"), mem_fun(*this, &Mixer_UI::hide_all_audiobus)));
+       items.push_back (MenuElem (_("Show All"), sigc::mem_fun(*this, &Mixer_UI::show_all_routes)));
+       items.push_back (MenuElem (_("Hide All"), sigc::mem_fun(*this, &Mixer_UI::hide_all_routes)));
+       items.push_back (MenuElem (_("Show All Audio Tracks"), sigc::mem_fun(*this, &Mixer_UI::show_all_audiotracks)));
+       items.push_back (MenuElem (_("Hide All Audio Tracks"), sigc::mem_fun(*this, &Mixer_UI::hide_all_audiotracks)));
+       items.push_back (MenuElem (_("Show All Audio Busses"), sigc::mem_fun(*this, &Mixer_UI::show_all_audiobus)));
+       items.push_back (MenuElem (_("Hide All Audio Busses"), sigc::mem_fun(*this, &Mixer_UI::hide_all_audiobus)));
 
 }
 
 void
-Mixer_UI::strip_name_changed (MixerStrip* mx)
+Mixer_UI::strip_property_changed (const PropertyChange& what_changed, MixerStrip* mx)
 {
-       ENSURE_GUI_THREAD(bind (mem_fun(*this, &Mixer_UI::strip_name_changed), mx));
+       if (!what_changed.contains (ARDOUR::Properties::name)) {
+               return;
+       }
+
+       ENSURE_GUI_THREAD (*this, &Mixer_UI::strip_name_changed, what_changed, mx)
 
        TreeModel::Children rows = track_model->children();
        TreeModel::Children::iterator i;
@@ -999,10 +1036,10 @@ Mixer_UI::build_route_group_context_menu ()
        route_group_context_menu->set_name ("ArdourContextMenu");
        MenuList& items = route_group_context_menu->items();
 
-       items.push_back (MenuElem (_("Activate All"), mem_fun(*this, &Mixer_UI::activate_all_route_groups)));
-       items.push_back (MenuElem (_("Disable All"), mem_fun(*this, &Mixer_UI::disable_all_route_groups)));
+       items.push_back (MenuElem (_("Activate All"), sigc::mem_fun(*this, &Mixer_UI::activate_all_route_groups)));
+       items.push_back (MenuElem (_("Disable All"), sigc::mem_fun(*this, &Mixer_UI::disable_all_route_groups)));
        items.push_back (SeparatorElem());
-       items.push_back (MenuElem (_("Add group"), mem_fun(*this, &Mixer_UI::new_route_group)));
+       items.push_back (MenuElem (_("Add group"), sigc::mem_fun(*this, &Mixer_UI::new_route_group)));
 
 }
 
@@ -1066,19 +1103,19 @@ Mixer_UI::group_display_button_press (GdkEventButton* ev)
 void
 Mixer_UI::activate_all_route_groups ()
 {
-       session->foreach_route_group (bind (mem_fun (*this, &Mixer_UI::set_route_group_activation), true));
+       _session->foreach_route_group (sigc::bind (sigc::mem_fun (*this, &Mixer_UI::set_route_group_activation), true));
 }
 
 void
 Mixer_UI::disable_all_route_groups ()
 {
-       session->foreach_route_group (bind (mem_fun (*this, &Mixer_UI::set_route_group_activation), false));
+       _session->foreach_route_group (sigc::bind (sigc::mem_fun (*this, &Mixer_UI::set_route_group_activation), false));
 }
 
 void
 Mixer_UI::route_groups_changed ()
 {
-       ENSURE_GUI_THREAD (mem_fun (*this, &Mixer_UI::route_groups_changed));
+       ENSURE_GUI_THREAD (*this, &Mixer_UI::route_groups_changed)
 
        /* just rebuild the while thing */
 
@@ -1092,13 +1129,23 @@ Mixer_UI::route_groups_changed ()
                row[group_columns.group] = 0;
        }
 
-       session->foreach_route_group (mem_fun (*this, &Mixer_UI::add_route_group));
+       _session->foreach_route_group (sigc::mem_fun (*this, &Mixer_UI::add_route_group));
 }
 
 void
 Mixer_UI::new_route_group ()
 {
-       session->add_route_group (new RouteGroup (*session, "", RouteGroup::Active, (RouteGroup::Property) (RouteGroup::Gain |RouteGroup::Mute | RouteGroup::Solo)));
+       PropertyList plist;
+
+       plist.add (Properties::active, true);
+       plist.add (Properties::gain, true);
+       plist.add (Properties::mute, true);
+       plist.add (Properties::solo, true);
+
+       RouteGroup* g = new RouteGroup (*_session, "");
+       g->set_properties (plist);
+
+       _session->add_route_group (g);
 }
 
 void
@@ -1121,7 +1168,7 @@ Mixer_UI::remove_selected_route_group ()
                RouteGroup* rg = (*iter)[group_columns.group];
 
                if (rg) {
-                       session->remove_route_group (*rg);
+                       _session->remove_route_group (*rg);
                }
        }
 }
@@ -1133,7 +1180,7 @@ Mixer_UI::group_flags_changed (void* src, RouteGroup* group)
                return;
        }
 
-       ENSURE_GUI_THREAD(bind (mem_fun(*this, &Mixer_UI::group_flags_changed), src, group));
+       ENSURE_GUI_THREAD (*this, &Mixer_UI::group_flags_changed, src, group)
 
        /* force an update of any mixer strips that are using this group,
           otherwise mix group names don't change in mixer strips
@@ -1141,7 +1188,7 @@ Mixer_UI::group_flags_changed (void* src, RouteGroup* group)
 
        for (list<MixerStrip *>::iterator i = strips.begin(); i != strips.end(); ++i) {
                if ((*i)->route_group() == group) {
-                       (*i)->route_group_changed(0);
+                       (*i)->route_group_changed();
                }
        }
 
@@ -1220,7 +1267,7 @@ Mixer_UI::route_group_row_change (const Gtk::TreeModel::Path&, const Gtk::TreeMo
 void
 Mixer_UI::add_route_group (RouteGroup* group)
 {
-       ENSURE_GUI_THREAD(bind (mem_fun(*this, &Mixer_UI::add_route_group), group));
+       ENSURE_GUI_THREAD (*this, &Mixer_UI::add_route_group, group)
        bool focus = false;
 
        in_group_row_change = true;
@@ -1235,7 +1282,7 @@ Mixer_UI::add_route_group (RouteGroup* group)
                focus = true;
        }
 
-       group->FlagsChanged.connect (bind (mem_fun(*this, &Mixer_UI::group_flags_changed), group));
+       group->FlagsChanged.connect (*this, ui_bind (&Mixer_UI::group_flags_changed, this, _1, group), gui_context());
 
        if (focus) {
                TreeViewColumn* col = group_display.get_column (0);
@@ -1454,10 +1501,38 @@ Mixer_UI::pane_allocation_handler (Allocation&, Gtk::Paned* which)
                }
        }
 }
+void
+Mixer_UI::scroll_left () 
+{
+       Adjustment* adj = scroller.get_hscrollbar()->get_adjustment();
+       /* stupid GTK: can't rely on clamping across versions */
+       scroller.get_hscrollbar()->set_value (max (adj->get_lower(), adj->get_value() - adj->get_step_increment()));
+}
+
+void
+Mixer_UI::scroll_right ()
+{
+       Adjustment* adj = scroller.get_hscrollbar()->get_adjustment();
+       /* stupid GTK: can't rely on clamping across versions */
+       scroller.get_hscrollbar()->set_value (min (adj->get_upper(), adj->get_value() + adj->get_step_increment()));
+}
 
 bool
 Mixer_UI::on_key_press_event (GdkEventKey* ev)
 {
+       switch (ev->keyval) {
+       case GDK_Left:
+               scroll_left ();
+               return true;
+
+       case GDK_Right:
+               scroll_right ();
+               return true;
+
+       default:
+               break;
+       }
+
        return key_press_focus_accelerator_handler (*this, ev);
 }
 
@@ -1468,11 +1543,42 @@ Mixer_UI::on_key_release_event (GdkEventKey* ev)
        // return key_press_focus_accelerator_handler (*this, ev);
 }
 
+
+bool
+Mixer_UI::on_scroll_event (GdkEventScroll* ev)
+{
+       switch (ev->direction) {
+       case GDK_SCROLL_LEFT:
+               scroll_left ();
+               return true;
+       case GDK_SCROLL_UP:
+               if (ev->state & Keyboard::TertiaryModifier) {
+                       scroll_left ();
+                       return true;
+               }
+               return false;
+
+       case GDK_SCROLL_RIGHT:
+               scroll_right ();
+               return true;
+
+       case GDK_SCROLL_DOWN:
+               if (ev->state & Keyboard::TertiaryModifier) {
+                       scroll_right ();
+                       return true;
+               }
+               return false;
+       }
+
+       return false;
+}
+
+
 void
 Mixer_UI::parameter_changed (string const & p)
 {
        if (p == "show-group-tabs") {
-               bool const s = session->config.get_show_group_tabs ();
+               bool const s = _session->config.get_show_group_tabs ();
                if (s) {
                        _group_tabs->show ();
                } else {
@@ -1487,12 +1593,13 @@ Mixer_UI::set_route_group_activation (RouteGroup* g, bool a)
        g->set_active (a, this);
 }
 
-PluginSelector&
+PluginSelector*
 Mixer_UI::plugin_selector()
 {
-       if (!_plugin_selector) {
-               _plugin_selector = new PluginSelector (PluginManager::the_manager());
-               _plugin_selector->set_session (session);
-       }
-       return *_plugin_selector;
+#ifdef DEFER_PLUGIN_SELECTOR_LOAD
+       if (!_plugin_selector)
+               _plugin_selector = new PluginSelector (PluginManager::the_manager ());
+#endif
+
+       return _plugin_selector;
 }