cleanup up cleanup at session destruction; clarify the meaning of 3 signals (DropRefe...
[ardour.git] / gtk2_ardour / route_time_axis.cc
index ef948307a2846ce85f36597009bf89a917edb2a0..3e6edd65e1b631335322bccd021804ae4240d641 100644 (file)
@@ -88,7 +88,6 @@ using namespace PBD;
 using namespace Gtkmm2ext;
 using namespace Gtk;
 using namespace Editing;
-using namespace sigc;
 using namespace std;
 
 Glib::RefPtr<Gdk::Pixbuf> RouteTimeAxisView::slider;
@@ -101,7 +100,7 @@ RouteTimeAxisView::setup_slider_pix ()
        }
 }
 
-RouteTimeAxisView::RouteTimeAxisView (PublicEditor& ed, Session& sess, boost::shared_ptr<Route> rt, Canvas& canvas)
+RouteTimeAxisView::RouteTimeAxisView (PublicEditor& ed, Session* sess, boost::shared_ptr<Route> rt, Canvas& canvas)
        : AxisView(sess)
        , RouteUI(rt, sess)
        , TimeAxisView(sess,ed,(TimeAxisView*) 0, canvas)
@@ -156,17 +155,17 @@ RouteTimeAxisView::RouteTimeAxisView (PublicEditor& ed, Session& sess, boost::sh
        hide_button.add (*(manage (new Image (::get_icon("hide")))));
        hide_button.show_all ();
 
-       route_group_button.signal_button_release_event().connect (mem_fun(*this, &RouteTimeAxisView::edit_click), false);
-       playlist_button.signal_clicked().connect (mem_fun(*this, &RouteTimeAxisView::playlist_click));
-       automation_button.signal_clicked().connect (mem_fun(*this, &RouteTimeAxisView::automation_click));
-       size_button.signal_button_release_event().connect (mem_fun(*this, &RouteTimeAxisView::size_click), false);
-       visual_button.signal_clicked().connect (mem_fun(*this, &RouteTimeAxisView::visual_click));
-       hide_button.signal_clicked().connect (mem_fun(*this, &RouteTimeAxisView::hide_click));
+       route_group_button.signal_button_release_event().connect (sigc::mem_fun(*this, &RouteTimeAxisView::edit_click), false);
+       playlist_button.signal_clicked().connect (sigc::mem_fun(*this, &RouteTimeAxisView::playlist_click));
+       automation_button.signal_clicked().connect (sigc::mem_fun(*this, &RouteTimeAxisView::automation_click));
+       size_button.signal_button_release_event().connect (sigc::mem_fun(*this, &RouteTimeAxisView::size_click), false);
+       visual_button.signal_clicked().connect (sigc::mem_fun(*this, &RouteTimeAxisView::visual_click));
+       hide_button.signal_clicked().connect (sigc::mem_fun(*this, &RouteTimeAxisView::hide_click));
 
-       solo_button->signal_button_press_event().connect (mem_fun(*this, &RouteUI::solo_press), false);
-       solo_button->signal_button_release_event().connect (mem_fun(*this, &RouteUI::solo_release), false);
-       mute_button->signal_button_press_event().connect (mem_fun(*this, &RouteUI::mute_press), false);
-       mute_button->signal_button_release_event().connect (mem_fun(*this, &RouteUI::mute_release), false);
+       solo_button->signal_button_press_event().connect (sigc::mem_fun(*this, &RouteUI::solo_press), false);
+       solo_button->signal_button_release_event().connect (sigc::mem_fun(*this, &RouteUI::solo_release), false);
+       mute_button->signal_button_press_event().connect (sigc::mem_fun(*this, &RouteUI::mute_press), false);
+       mute_button->signal_button_release_event().connect (sigc::mem_fun(*this, &RouteUI::mute_release), false);
 
        if (is_track()) {
 
@@ -185,18 +184,18 @@ RouteTimeAxisView::RouteTimeAxisView (PublicEditor& ed, Session& sess, boost::sh
                }
                rec_enable_button->show_all ();
 
-               rec_enable_button->signal_button_press_event().connect (mem_fun(*this, &RouteUI::rec_enable_press), false);
-               rec_enable_button->signal_button_release_event().connect (mem_fun(*this, &RouteUI::rec_enable_release), false);
+               rec_enable_button->signal_button_press_event().connect (sigc::mem_fun(*this, &RouteUI::rec_enable_press), false);
+               rec_enable_button->signal_button_release_event().connect (sigc::mem_fun(*this, &RouteUI::rec_enable_release), false);
                controls_table.attach (*rec_enable_button, 5, 6, 0, 1, Gtk::FILL|Gtk::EXPAND, Gtk::FILL|Gtk::EXPAND, 0, 0);
                ARDOUR_UI::instance()->tooltips().set_tip(*rec_enable_button, _("Record"));
 
-               rec_enable_button->set_sensitive (_session.writable());
+               rec_enable_button->set_sensitive (_session->writable());
        }
 
        controls_hbox.pack_start(gm.get_level_meter(), false, false);
-       _route->meter_change.connect (mem_fun(*this, &RouteTimeAxisView::meter_changed));
-       _route->input()->changed.connect (mem_fun(*this, &RouteTimeAxisView::io_changed));
-       _route->output()->changed.connect (mem_fun(*this, &RouteTimeAxisView::io_changed));
+       _route->meter_change.connect (*this, ui_bind (&RouteTimeAxisView::meter_changed, this, _1), gui_context());
+       _route->input()->changed.connect (*this, ui_bind (&RouteTimeAxisView::io_changed, this, _1, _2), gui_context());
+       _route->output()->changed.connect (*this, ui_bind (&RouteTimeAxisView::io_changed, this, _1, _2), gui_context());
 
        controls_table.attach (*mute_button, 6, 7, 0, 1, Gtk::FILL|Gtk::EXPAND, Gtk::FILL|Gtk::EXPAND, 0, 0);
        controls_table.attach (*solo_button, 7, 8, 0, 1, Gtk::FILL|Gtk::EXPAND, Gtk::FILL|Gtk::EXPAND, 0, 0);
@@ -235,42 +234,42 @@ RouteTimeAxisView::RouteTimeAxisView (PublicEditor& ed, Session& sess, boost::sh
 
        _y_position = -1;
 
-       _route->mute_changed.connect (mem_fun(*this, &RouteUI::mute_changed));
-       _route->solo_changed.connect (mem_fun(*this, &RouteUI::solo_changed));
-       _route->processors_changed.connect (mem_fun(*this, &RouteTimeAxisView::processors_changed));
-       _route->NameChanged.connect (mem_fun(*this, &RouteTimeAxisView::route_name_changed));
-       _route->solo_isolated_changed.connect (mem_fun(*this, &RouteUI::solo_changed));
+       _route->mute_changed.connect (*this, ui_bind (&RouteUI::mute_changed, this, _1), gui_context());
+       _route->solo_changed.connect (*this, ui_bind (&RouteUI::solo_changed, this, _1), gui_context());
+       _route->processors_changed.connect (*this, ui_bind (&RouteTimeAxisView::processors_changed, this, _1), gui_context());
+       _route->NameChanged.connect (*this, boost::bind (&RouteTimeAxisView::route_name_changed, this), gui_context());
+       _route->solo_isolated_changed.connect (*this, ui_bind (&RouteUI::solo_changed, this, _1), gui_context());
 
 
        if (is_track()) {
 
-               track()->TrackModeChanged.connect (mem_fun(*this, &RouteTimeAxisView::track_mode_changed));
-               track()->FreezeChange.connect (mem_fun(*this, &RouteTimeAxisView::map_frozen));
-               track()->DiskstreamChanged.connect (mem_fun(*this, &RouteTimeAxisView::diskstream_changed));
-               get_diskstream()->SpeedChanged.connect (mem_fun(*this, &RouteTimeAxisView::speed_changed));
+               track()->TrackModeChanged.connect (*this, boost::bind (&RouteTimeAxisView::track_mode_changed, this), gui_context());
+               track()->FreezeChange.connect (*this, boost::bind (&RouteTimeAxisView::map_frozen, this), gui_context());
+               track()->DiskstreamChanged.connect (*this, boost::bind (&RouteTimeAxisView::diskstream_changed, this), gui_context());
+               get_diskstream()->SpeedChanged.connect (*this, boost::bind (&RouteTimeAxisView::speed_changed, this), gui_context());
 
                /* pick up the correct freeze state */
                map_frozen ();
 
        }
 
-       _editor.ZoomChanged.connect (mem_fun(*this, &RouteTimeAxisView::reset_samples_per_unit));
-       ColorsChanged.connect (mem_fun (*this, &RouteTimeAxisView::color_handler));
+       _editor.ZoomChanged.connect (sigc::mem_fun(*this, &RouteTimeAxisView::reset_samples_per_unit));
+       ColorsChanged.connect (sigc::mem_fun (*this, &RouteTimeAxisView::color_handler));
 
        route_group_menu = new RouteGroupMenu (
                _session,
                (RouteGroup::Property) (RouteGroup::Mute | RouteGroup::Solo | RouteGroup::Edit)
                );
 
-       route_group_menu->GroupSelected.connect (mem_fun (*this, &RouteTimeAxisView::set_route_group_from_menu));
+       route_group_menu->GroupSelected.connect (sigc::mem_fun (*this, &RouteTimeAxisView::set_route_group_from_menu));
 
-       gm.get_gain_slider().signal_scroll_event().connect(mem_fun(*this, &RouteTimeAxisView::controls_ebox_scroll), false);
+       gm.get_gain_slider().signal_scroll_event().connect(sigc::mem_fun(*this, &RouteTimeAxisView::controls_ebox_scroll), false);
        gm.get_gain_slider().set_name ("TrackGainFader");
 }
 
 RouteTimeAxisView::~RouteTimeAxisView ()
 {
-       GoingAway (); /* EMIT_SIGNAL */
+       CatchDeletion (this);
 
        for (list<ProcessorAutomationInfo*>::iterator i = processor_automation.begin(); i != processor_automation.end(); ++i) {
                delete *i;
@@ -302,8 +301,8 @@ RouteTimeAxisView::post_construct ()
        update_diskstream_display ();
 
        subplugin_menu.items().clear ();
-       _route->foreach_processor (mem_fun (*this, &RouteTimeAxisView::add_processor_to_subplugin_menu));
-       _route->foreach_processor (mem_fun (*this, &RouteTimeAxisView::add_existing_processor_automation_curves));
+       _route->foreach_processor (sigc::mem_fun (*this, &RouteTimeAxisView::add_processor_to_subplugin_menu));
+       _route->foreach_processor (sigc::mem_fun (*this, &RouteTimeAxisView::add_existing_processor_automation_curves));
        reset_processor_automation_curves ();
 }
 
@@ -311,20 +310,28 @@ gint
 RouteTimeAxisView::edit_click (GdkEventButton *ev)
 {
        if (Keyboard::modifier_state_equals (ev->state, Keyboard::PrimaryModifier)) {
-               _route->set_route_group (0, this);
-               return FALSE;
+               if (_route->route_group()) {
+                       _route->route_group()->remove (_route);
+               }
+               return false;
        }
 
        route_group_menu->rebuild (_route->route_group ());
        route_group_menu->popup (ev->button, ev->time);
 
-       return FALSE;
+       return false;
 }
 
 void
 RouteTimeAxisView::set_route_group_from_menu (RouteGroup *eg)
 {
-       _route->set_route_group (eg, this);
+       if (eg) {
+               eg->add (_route);
+       } else {
+               if (_route->route_group()) {
+                       _route->route_group()->remove (_route);
+               }
+       }
 }
 
 void
@@ -423,13 +430,13 @@ RouteTimeAxisView::build_automation_action_menu ()
        automation_action_menu->set_name ("ArdourContextMenu");
 
        automation_items.push_back (MenuElem (_("Show all automation"),
-                                             mem_fun(*this, &RouteTimeAxisView::show_all_automation)));
+                                             sigc::mem_fun(*this, &RouteTimeAxisView::show_all_automation)));
 
        automation_items.push_back (MenuElem (_("Show existing automation"),
-                                             mem_fun(*this, &RouteTimeAxisView::show_existing_automation)));
+                                             sigc::mem_fun(*this, &RouteTimeAxisView::show_existing_automation)));
 
        automation_items.push_back (MenuElem (_("Hide all automation"),
-                                             mem_fun(*this, &RouteTimeAxisView::hide_all_automation)));
+                                             sigc::mem_fun(*this, &RouteTimeAxisView::hide_all_automation)));
 
        if (subplugin_menu.get_attach_widget())
                subplugin_menu.detach();
@@ -445,7 +452,7 @@ RouteTimeAxisView::build_automation_action_menu ()
                delete i->second->menu_item;
 
                automation_items.push_back(CheckMenuElem (_route->describe_parameter(i->second->param),
-                               bind (mem_fun(*this, &RouteTimeAxisView::toggle_automation_track), i->second->param)));
+                               sigc::bind (sigc::mem_fun(*this, &RouteTimeAxisView::toggle_automation_track), i->second->param)));
 
                i->second->menu_item = static_cast<Gtk::CheckMenuItem*>(&automation_items.back());
 
@@ -473,12 +480,12 @@ RouteTimeAxisView::build_display_menu ()
        display_menu->set_name ("ArdourContextMenu");
 
        items.push_back (MenuElem (_("Height"), *size_menu));
-       items.push_back (MenuElem (_("Color"), mem_fun(*this, &RouteTimeAxisView::select_track_color)));
+       items.push_back (MenuElem (_("Color"), sigc::mem_fun(*this, &RouteTimeAxisView::select_track_color)));
 
        items.push_back (SeparatorElem());
 
        if (!Profile->get_sae()) {
-               items.push_back (MenuElem (_("Remote Control ID..."), mem_fun (*this, &RouteUI::open_remote_control_id_dialog)));
+               items.push_back (MenuElem (_("Remote Control ID..."), sigc::mem_fun (*this, &RouteUI::open_remote_control_id_dialog)));
                /* rebuild this every time */
                build_automation_action_menu ();
                items.push_back (MenuElem (_("Automation"), *automation_action_menu));
@@ -498,9 +505,9 @@ RouteTimeAxisView::build_display_menu ()
                RadioMenuItem::Group layers_group;
 
                layers_items.push_back(RadioMenuElem (layers_group, _("Overlaid"),
-                               bind (mem_fun (*this, &RouteTimeAxisView::set_layer_display), Overlaid)));
+                               sigc::bind (sigc::mem_fun (*this, &RouteTimeAxisView::set_layer_display), Overlaid)));
                layers_items.push_back(RadioMenuElem (layers_group, _("Stacked"),
-                               bind (mem_fun (*this, &RouteTimeAxisView::set_layer_display), Stacked)));
+                               sigc::bind (sigc::mem_fun (*this, &RouteTimeAxisView::set_layer_display), Stacked)));
 
                items.push_back (MenuElem (_("Layers"), *layers_menu));
 
@@ -511,35 +518,34 @@ RouteTimeAxisView::build_display_menu ()
                RadioMenuItem::Group align_group;
 
                alignment_items.push_back (RadioMenuElem (align_group, _("Align with existing material"),
-                                       bind (mem_fun(*this, &RouteTimeAxisView::set_align_style), ExistingMaterial)));
+                                       sigc::bind (sigc::mem_fun(*this, &RouteTimeAxisView::set_align_style), ExistingMaterial)));
                align_existing_item = dynamic_cast<RadioMenuItem*>(&alignment_items.back());
                if (get_diskstream()->alignment_style() == ExistingMaterial)
                        align_existing_item->set_active();
 
                alignment_items.push_back (RadioMenuElem (align_group, _("Align with capture time"),
-                                       bind (mem_fun(*this, &RouteTimeAxisView::set_align_style), CaptureTime)));
+                                       sigc::bind (sigc::mem_fun(*this, &RouteTimeAxisView::set_align_style), CaptureTime)));
                align_capture_item = dynamic_cast<RadioMenuItem*>(&alignment_items.back());
                if (get_diskstream()->alignment_style() == CaptureTime)
                        align_capture_item->set_active();
 
                if (!Profile->get_sae()) {
                        items.push_back (MenuElem (_("Alignment"), *alignment_menu));
-                       get_diskstream()->AlignmentStyleChanged.connect (
-                                       mem_fun(*this, &RouteTimeAxisView::align_style_changed));
+                       get_diskstream()->AlignmentStyleChanged.connect (route_connections, boost::bind (&RouteTimeAxisView::align_style_changed, this), gui_context());
 
                        RadioMenuItem::Group mode_group;
-                       items.push_back (RadioMenuElem (mode_group, _("Normal mode"), bind (
-                                       mem_fun (*this, &RouteTimeAxisView::set_track_mode),
+                       items.push_back (RadioMenuElem (mode_group, _("Normal mode"), sigc::bind (
+                                       sigc::mem_fun (*this, &RouteTimeAxisView::set_track_mode),
                                        ARDOUR::Normal)));
                        normal_track_mode_item = dynamic_cast<RadioMenuItem*>(&items.back());
 
-                       items.push_back (RadioMenuElem (mode_group, _("Tape mode"), bind (
-                                       mem_fun (*this, &RouteTimeAxisView::set_track_mode),
+                       items.push_back (RadioMenuElem (mode_group, _("Tape mode"), sigc::bind (
+                                       sigc::mem_fun (*this, &RouteTimeAxisView::set_track_mode),
                                        ARDOUR::Destructive)));
                        destructive_track_mode_item = dynamic_cast<RadioMenuItem*>(&items.back());
 
                        items.push_back (RadioMenuElem (mode_group, _("No layering mode"),
-                                                       bind (mem_fun (*this, &RouteTimeAxisView::set_track_mode), ARDOUR::NonLayered)));
+                                                       sigc::bind (sigc::mem_fun (*this, &RouteTimeAxisView::set_track_mode), ARDOUR::NonLayered)));
                        non_layered_track_mode_item = dynamic_cast<RadioMenuItem*>(&items.back());
 
 
@@ -557,8 +563,7 @@ RouteTimeAxisView::build_display_menu ()
                        }
                }
 
-               get_diskstream()->AlignmentStyleChanged.connect (
-                               mem_fun(*this, &RouteTimeAxisView::align_style_changed));
+               get_diskstream()->AlignmentStyleChanged.connect (route_connections, boost::bind (&RouteTimeAxisView::align_style_changed, this), gui_context());
 
                mode_menu = build_mode_menu();
                if (mode_menu)
@@ -571,17 +576,17 @@ RouteTimeAxisView::build_display_menu ()
                items.push_back (SeparatorElem());
        }
 
-       items.push_back (CheckMenuElem (_("Active"), mem_fun(*this, &RouteUI::toggle_route_active)));
+       items.push_back (CheckMenuElem (_("Active"), sigc::mem_fun(*this, &RouteUI::toggle_route_active)));
        route_active_menu_item = dynamic_cast<CheckMenuItem *> (&items.back());
        route_active_menu_item->set_active (_route->active());
 
        items.push_back (SeparatorElem());
-       items.push_back (MenuElem (_("Hide"), mem_fun(*this, &RouteTimeAxisView::hide_click)));
+       items.push_back (MenuElem (_("Hide"), sigc::mem_fun(*this, &RouteTimeAxisView::hide_click)));
        if (!Profile->get_sae()) {
-               items.push_back (MenuElem (_("Remove"), mem_fun(*this, &RouteUI::remove_this_route)));
+               items.push_back (MenuElem (_("Remove"), sigc::mem_fun(*this, &RouteUI::remove_this_route)));
        } else {
                items.push_front (SeparatorElem());
-               items.push_front (MenuElem (_("Delete"), mem_fun(*this, &RouteUI::remove_this_route)));
+               items.push_front (MenuElem (_("Delete"), sigc::mem_fun(*this, &RouteUI::remove_this_route)));
        }
 }
 
@@ -635,7 +640,7 @@ RouteTimeAxisView::_set_track_mode (Track* track, TrackMode mode, RadioMenuItem*
 
                if (!needs_bounce) {
                        /* cannot be done */
-                       Glib::signal_idle().connect (bind (sigc::ptr_fun (__reset_item), reset_item, reset_item_2));
+                       Glib::signal_idle().connect (sigc::bind (sigc::ptr_fun (__reset_item), reset_item, reset_item_2));
                        return;
                } else {
                        cerr << "would bounce this one\n";
@@ -1037,8 +1042,8 @@ RouteTimeAxisView::use_copy_playlist (bool prompt, vector<boost::shared_ptr<Play
                name = resolve_new_group_playlist_name(name, playlists_before_op);
        }
 
-       while (_session.playlists->by_name(name)) {
-               name = Playlist::bump_name (name, _session);
+       while (_session->playlists->by_name(name)) {
+               name = Playlist::bump_name (name, *_session);
        }
 
        // TODO: The prompter "new" button should be de-activated if the user
@@ -1089,8 +1094,8 @@ RouteTimeAxisView::use_new_playlist (bool prompt, vector<boost::shared_ptr<Playl
                name = resolve_new_group_playlist_name(name,playlists_before_op);
        }
 
-       while (_session.playlists->by_name(name)) {
-               name = Playlist::bump_name (name, _session);
+       while (_session->playlists->by_name(name)) {
+               name = Playlist::bump_name (name, *_session);
        }
 
 
@@ -1136,13 +1141,13 @@ RouteTimeAxisView::clear_playlist ()
 void
 RouteTimeAxisView::speed_changed ()
 {
-       Gtkmm2ext::UI::instance()->call_slot (mem_fun(*this, &RouteTimeAxisView::reset_samples_per_unit));
+       Gtkmm2ext::UI::instance()->call_slot (boost::bind (&RouteTimeAxisView::reset_samples_per_unit, this));
 }
 
 void
 RouteTimeAxisView::diskstream_changed ()
 {
-       Gtkmm2ext::UI::instance()->call_slot (mem_fun(*this, &RouteTimeAxisView::update_diskstream_display));
+       Gtkmm2ext::UI::instance()->call_slot (boost::bind (&RouteTimeAxisView::update_diskstream_display, this));
 }
 
 void
@@ -1169,33 +1174,23 @@ RouteTimeAxisView::selection_click (GdkEventButton* ev)
                return;
        }
 
-       PublicEditor::TrackViewList* tracks = _editor.get_valid_views (this, _route->route_group());
-
        switch (ArdourKeyboard::selection_type (ev->state)) {
        case Selection::Toggle:
-               _editor.get_selection().toggle (*tracks);
+               _editor.get_selection().toggle (this);
                break;
 
        case Selection::Set:
-               _editor.get_selection().set (*tracks);
+               _editor.get_selection().set (this);
                break;
 
        case Selection::Extend:
-               if (tracks->size() > 1) {
-                       /* add each one, do not "extend" */
-                       _editor.get_selection().add (*tracks);
-               } else {
-                       /* extend to the single track */
-                       _editor.extend_selection_to_track (*tracks->front());
-               }
+               _editor.extend_selection_to_track (*this);
                break;
 
        case Selection::Add:
-               _editor.get_selection().add (*tracks);
+               _editor.get_selection().add (this);
                break;
        }
-
-       delete tracks;
 }
 
 void
@@ -1328,10 +1323,10 @@ RouteTimeAxisView::name_entry_changed ()
                return;
        }
 
-       if (!_session.route_name_unique (x)) {
+       if (!_session->route_name_unique (x)) {
                ARDOUR_UI::instance()->popup_error (_("A track already exists with that name"));
                name_entry.set_text (_route->name());
-       } else if (_session.route_name_internal (x)) {
+       } else if (_session->route_name_internal (x)) {
                ARDOUR_UI::instance()->popup_error (_("You cannot create a track with that name as it is reserved for Ardour"));
                name_entry.set_text (_route->name());
        } else {
@@ -1411,7 +1406,7 @@ RouteTimeAxisView::cut_copy_clear (Selection& selection, CutCopyOp op)
        case Cut:
                if ((what_we_got = playlist->cut (time)) != 0) {
                        _editor.get_cut_buffer().add (what_we_got);
-                       _session.add_command( new MementoCommand<Playlist>(*playlist.get(), &before, &playlist->get_state()));
+                       _session->add_command( new MementoCommand<Playlist>(*playlist.get(), &before, &playlist->get_state()));
                        ret = true;
                }
                break;
@@ -1423,7 +1418,7 @@ RouteTimeAxisView::cut_copy_clear (Selection& selection, CutCopyOp op)
 
        case Clear:
                if ((what_we_got = playlist->cut (time)) != 0) {
-                       _session.add_command( new MementoCommand<Playlist>(*playlist, &before, &playlist->get_state()));
+                       _session->add_command( new MementoCommand<Playlist>(*playlist, &before, &playlist->get_state()));
                        what_we_got->release ();
                        ret = true;
                }
@@ -1455,7 +1450,7 @@ RouteTimeAxisView::paste (nframes_t pos, float times, Selection& selection, size
 
        XMLNode &before = playlist->get_state();
        playlist->paste (*p, pos, times);
-       _session.add_command( new MementoCommand<Playlist>(*playlist, &before, &playlist->get_state()));
+       _session->add_command( new MementoCommand<Playlist>(*playlist, &before, &playlist->get_state()));
 
        return true;
 }
@@ -1497,19 +1492,19 @@ RouteTimeAxisView::build_playlist_menu (Gtk::Menu * menu)
        boost::shared_ptr<Diskstream> ds = get_diskstream();
        RadioMenuItem::Group playlist_group;
 
-       _session.playlists->get (playlists);
+       _session->playlists->get (playlists);
 
        for (vector<boost::shared_ptr<Playlist> >::iterator i = playlists.begin(); i != playlists.end(); ++i) {
 
                if ((*i)->get_orig_diskstream_id() == ds->id()) {
-                       playlist_items.push_back (RadioMenuElem (playlist_group, (*i)->name(), bind (mem_fun (*this, &RouteTimeAxisView::use_playlist),
+                       playlist_items.push_back (RadioMenuElem (playlist_group, (*i)->name(), sigc::bind (sigc::mem_fun (*this, &RouteTimeAxisView::use_playlist),
                                                                                                     boost::weak_ptr<Playlist> (*i))));
 
                        if (ds->playlist()->id() == (*i)->id()) {
                                static_cast<RadioMenuItem*>(&playlist_items.back())->set_active();
                        }
                } else if (ds->playlist()->id() == (*i)->id()) {
-                       playlist_items.push_back (RadioMenuElem (playlist_group, (*i)->name(), bind (mem_fun (*this, &RouteTimeAxisView::use_playlist),
+                       playlist_items.push_back (RadioMenuElem (playlist_group, (*i)->name(), sigc::bind (sigc::mem_fun (*this, &RouteTimeAxisView::use_playlist),
                                                                                                     boost::weak_ptr<Playlist>(*i))));
                        static_cast<RadioMenuItem*>(&playlist_items.back())->set_active();
 
@@ -1517,25 +1512,25 @@ RouteTimeAxisView::build_playlist_menu (Gtk::Menu * menu)
        }
 
        playlist_items.push_back (SeparatorElem());
-       playlist_items.push_back (MenuElem (_("Rename"), mem_fun(*this, &RouteTimeAxisView::rename_current_playlist)));
+       playlist_items.push_back (MenuElem (_("Rename"), sigc::mem_fun(*this, &RouteTimeAxisView::rename_current_playlist)));
        playlist_items.push_back (SeparatorElem());
 
        if (!route_group() || !route_group()->is_active()) {
-               playlist_items.push_back (MenuElem (_("New"), bind(mem_fun(_editor, &PublicEditor::new_playlists), this)));
-               playlist_items.push_back (MenuElem (_("New Copy"), bind(mem_fun(_editor, &PublicEditor::copy_playlists), this)));
+               playlist_items.push_back (MenuElem (_("New"), sigc::bind(sigc::mem_fun(_editor, &PublicEditor::new_playlists), this)));
+               playlist_items.push_back (MenuElem (_("New Copy"), sigc::bind(sigc::mem_fun(_editor, &PublicEditor::copy_playlists), this)));
 
        } else {
                // Use a label which tells the user what is happening
-               playlist_items.push_back (MenuElem (_("New Take"), bind(mem_fun(_editor, &PublicEditor::new_playlists), this)));
-               playlist_items.push_back (MenuElem (_("Copy Take"), bind(mem_fun(_editor, &PublicEditor::copy_playlists), this)));
+               playlist_items.push_back (MenuElem (_("New Take"), sigc::bind(sigc::mem_fun(_editor, &PublicEditor::new_playlists), this)));
+               playlist_items.push_back (MenuElem (_("Copy Take"), sigc::bind(sigc::mem_fun(_editor, &PublicEditor::copy_playlists), this)));
 
        }
 
        playlist_items.push_back (SeparatorElem());
-       playlist_items.push_back (MenuElem (_("Clear Current"), bind(mem_fun(_editor, &PublicEditor::clear_playlists), this)));
+       playlist_items.push_back (MenuElem (_("Clear Current"), sigc::bind(sigc::mem_fun(_editor, &PublicEditor::clear_playlists), this)));
        playlist_items.push_back (SeparatorElem());
 
-       playlist_items.push_back (MenuElem(_("Select from all ..."), mem_fun(*this, &RouteTimeAxisView::show_playlist_selector)));
+       playlist_items.push_back (MenuElem(_("Select from all ..."), sigc::mem_fun(*this, &RouteTimeAxisView::show_playlist_selector)));
 }
 
 void
@@ -1572,20 +1567,22 @@ RouteTimeAxisView::use_playlist (boost::weak_ptr<Playlist> wpl)
 
                        take_name = take_name.substr(idx + group_string.length()); // find the bit containing the take number / name
 
-                       for (list<Route*>::const_iterator i = route_group()->route_list().begin(); i != route_group()->route_list().end(); ++i) {
-                               if ( (*i) == this->route().get()) {
+                       boost::shared_ptr<RouteList> rl (route_group()->route_list());
+
+                       for (RouteList::const_iterator i = rl->begin(); i != rl->end(); ++i) {
+                               if ( (*i) == this->route()) {
                                        continue;
                                }
 
                                std::string playlist_name = (*i)->name()+group_string+take_name;
 
-                               Track *track = dynamic_cast<Track *>(*i);
+                               boost::shared_ptr<Track> track = boost::dynamic_pointer_cast<Track>(*i);
                                if (!track) {
                                        std::cerr << "route " << (*i)->name() << " is not a Track" << std::endl;
                                        continue;
                                }
 
-                               boost::shared_ptr<Playlist> ipl = session().playlists->by_name(playlist_name);
+                               boost::shared_ptr<Playlist> ipl = session()->playlists->by_name(playlist_name);
                                if (!ipl) {
                                        // No playlist for this track for this take yet, make it
                                        track->diskstream()->use_new_playlist();
@@ -1593,8 +1590,6 @@ RouteTimeAxisView::use_playlist (boost::weak_ptr<Playlist> wpl)
                                } else {
                                        track->diskstream()->use_playlist(ipl);
                                }
-
-                               //(*i)->get_dis
                        }
                }
        }
@@ -1613,7 +1608,7 @@ RouteTimeAxisView::map_frozen ()
                return;
        }
 
-       ENSURE_GUI_THREAD (mem_fun(*this, &RouteTimeAxisView::map_frozen));
+       ENSURE_GUI_THREAD (*this, &RouteTimeAxisView::map_frozen)
 
        switch (track()->freeze_state()) {
        case Track::Frozen:
@@ -1687,7 +1682,9 @@ RouteTimeAxisView::automation_track_hidden (Evoral::Parameter param)
                ran->menu_item->set_active (false);
        }
 
-        _route->gui_changed ("visible_tracks", (void *) 0); /* EMIT_SIGNAL */
+       if (_route) {
+               _route->gui_changed ("visible_tracks", (void *) 0); /* EMIT_SIGNAL */
+       }
 }
 
 
@@ -1900,7 +1897,7 @@ RouteTimeAxisView::add_processor_automation_curve (boost::shared_ptr<Processor>
                        new AutomationTimeAxisView (_session, _route, processor, control,
                                _editor, *this, false, parent_canvas, name, state_name));
 
-       pan->view->Hiding.connect (bind (mem_fun(*this, &RouteTimeAxisView::processor_automation_track_hidden), pan, processor));
+       pan->view->Hiding.connect (sigc::bind (sigc::mem_fun(*this, &RouteTimeAxisView::processor_automation_track_hidden), pan, processor));
 
        if (!pan->view->marked_for_display()) {
                pan->view->hide ();
@@ -1911,7 +1908,7 @@ RouteTimeAxisView::add_processor_automation_curve (boost::shared_ptr<Processor>
        add_child (pan->view);
 
        if (_view) {
-               _view->foreach_regionview (mem_fun(*pan->view.get(), &TimeAxisView::add_ghost));
+               _view->foreach_regionview (sigc::mem_fun(*pan->view.get(), &TimeAxisView::add_ghost));
        }
 
        processor->mark_automation_visible (what, true);
@@ -1962,7 +1959,7 @@ RouteTimeAxisView::add_automation_child(Evoral::Parameter param, boost::shared_p
 
        add_child (track);
 
-       track->Hiding.connect (bind (mem_fun (*this, &RouteTimeAxisView::automation_track_hidden), param));
+       track->Hiding.connect (sigc::bind (sigc::mem_fun (*this, &RouteTimeAxisView::automation_track_hidden), param));
 
        bool hideit = (!show);
 
@@ -2068,7 +2065,7 @@ RouteTimeAxisView::add_processor_to_subplugin_menu (boost::weak_ptr<Processor> p
 
                }
 
-               mitem->signal_toggled().connect (bind (mem_fun(*this, &RouteTimeAxisView::processor_menu_item_toggled), rai, pan));
+               mitem->signal_toggled().connect (sigc::bind (sigc::mem_fun(*this, &RouteTimeAxisView::processor_menu_item_toggled), rai, pan));
        }
 
        /* add the menu for this processor, because the subplugin
@@ -2134,8 +2131,8 @@ RouteTimeAxisView::processors_changed (RouteProcessorChange c)
 
        subplugin_menu.items().clear ();
 
-       _route->foreach_processor (mem_fun (*this, &RouteTimeAxisView::add_processor_to_subplugin_menu));
-       _route->foreach_processor (mem_fun (*this, &RouteTimeAxisView::add_existing_processor_automation_curves));
+       _route->foreach_processor (sigc::mem_fun (*this, &RouteTimeAxisView::add_processor_to_subplugin_menu));
+       _route->foreach_processor (sigc::mem_fun (*this, &RouteTimeAxisView::add_existing_processor_automation_curves));
 
        bool deleted_processor_automation = false;
 
@@ -2263,7 +2260,7 @@ RouteTimeAxisView::clear_meter ()
 void
 RouteTimeAxisView::meter_changed (void *src)
 {
-       ENSURE_GUI_THREAD (bind (mem_fun(*this, &RouteTimeAxisView::meter_changed), src));
+       ENSURE_GUI_THREAD (*this, &RouteTimeAxisView::meter_changed, src)
        reset_meter();
 }
 
@@ -2287,7 +2284,7 @@ RouteTimeAxisView::build_underlay_menu(Gtk::Menu* parent_menu) {
 
                for(UnderlayList::iterator it = _underlay_streams.begin(); it != _underlay_streams.end(); ++it) {
                        gs_items.push_back(MenuElem(string_compose(_("Remove \"%1\""), (*it)->trackview().name()),
-                                                   bind(mem_fun(*this, &RouteTimeAxisView::remove_underlay), *it)));
+                                                   sigc::bind(sigc::mem_fun(*this, &RouteTimeAxisView::remove_underlay), *it)));
                }
        }
 }
@@ -2343,7 +2340,7 @@ RouteTimeAxisView::add_underlay(StreamView* v, bool update_xml)
                _underlay_streams.push_back(v);
                other._underlay_mirrors.push_back(this);
 
-               v->foreach_regionview(mem_fun(*this, &RouteTimeAxisView::add_ghost));
+               v->foreach_regionview(sigc::mem_fun(*this, &RouteTimeAxisView::add_ghost));
 
                if(update_xml) {
                        if(!underlay_xml_node) {
@@ -2376,7 +2373,7 @@ RouteTimeAxisView::remove_underlay(StreamView* v)
                        /*NOTREACHED*/
                }
 
-               v->foreach_regionview(mem_fun(*this, &RouteTimeAxisView::remove_ghost));
+               v->foreach_regionview(sigc::mem_fun(*this, &RouteTimeAxisView::remove_ghost));
 
                _underlay_streams.erase(it);
                other._underlay_mirrors.erase(gm);