Don't report an error when the user cancels a time stretch operation.
[ardour.git] / gtk2_ardour / route_time_axis.cc
index dc1a463af92e56032a4048f4bb75fe8a7fca3805..0c52b96dc3eb0aff504d403e9de5ee9418e0d219 100644 (file)
@@ -33,7 +33,7 @@
 #include "pbd/whitespace.h"
 #include "pbd/memento_command.h"
 #include "pbd/enumwriter.h"
-#include "pbd/stacktrace.h"
+#include "pbd/stateful_diff_command.h"
 
 #include <gtkmm/menu.h>
 #include <gtkmm/menuitem.h>
@@ -114,9 +114,8 @@ RouteTimeAxisView::RouteTimeAxisView (PublicEditor& ed, Session* sess, boost::sh
        , automation_button (_("a"))
        , visual_button (_("v"))
        , gm (sess, slider, true, 115)
+       , _ignore_track_mode_change (false)
 {
-       set_button_names ();
-
        gm.set_controls (_route, _route->shared_peak_meter(), _route->amp());
        gm.get_level_meter().set_no_show_all();
        gm.get_level_meter().setup_meters(50);
@@ -133,6 +132,9 @@ RouteTimeAxisView::RouteTimeAxisView (PublicEditor& ed, Session* sess, boost::sh
                _marked_for_display = true;
        }
 
+       mute_changed (0);
+        update_solo_display ();
+
        timestretch_rect = 0;
        no_redraw = false;
        destructive_track_mode_item = 0;
@@ -183,30 +185,33 @@ RouteTimeAxisView::RouteTimeAxisView (PublicEditor& ed, Session* sess, boost::sh
                rec_enable_button->show_all ();
 
                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"));
+               ARDOUR_UI::instance()->set_tip(*rec_enable_button, _("Record"));
 
                rec_enable_button->set_sensitive (_session->writable());
        }
 
        controls_hbox.pack_start(gm.get_level_meter(), false, false);
-       _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());
+       _route->meter_change.connect (*this, invalidator (*this), bind (&RouteTimeAxisView::meter_changed, this), gui_context());
+       _route->input()->changed.connect (*this, invalidator (*this), ui_bind (&RouteTimeAxisView::io_changed, this, _1, _2), gui_context());
+       _route->output()->changed.connect (*this, invalidator (*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);
+
+        if (!_route->is_master()) {
+                controls_table.attach (*solo_button, 7, 8, 0, 1, Gtk::FILL|Gtk::EXPAND, Gtk::FILL|Gtk::EXPAND, 0, 0);
+        }
 
        controls_table.attach (route_group_button, 7, 8, 1, 2, Gtk::FILL|Gtk::EXPAND, Gtk::FILL|Gtk::EXPAND, 0, 0);
        controls_table.attach (gm.get_gain_slider(), 0, 5, 1, 2, Gtk::SHRINK, Gtk::SHRINK, 0, 0);
 
-       ARDOUR_UI::instance()->tooltips().set_tip(*solo_button,_("Solo"));
-       ARDOUR_UI::instance()->tooltips().set_tip(*mute_button,_("Mute"));
-       ARDOUR_UI::instance()->tooltips().set_tip(route_group_button, _("Route Group"));
-       ARDOUR_UI::instance()->tooltips().set_tip(size_button,_("Display Height"));
-       ARDOUR_UI::instance()->tooltips().set_tip(playlist_button,_("Playlist"));
-       ARDOUR_UI::instance()->tooltips().set_tip(automation_button, _("Automation"));
-       ARDOUR_UI::instance()->tooltips().set_tip(visual_button, _("Visual options"));
-       ARDOUR_UI::instance()->tooltips().set_tip(hide_button, _("Hide this track"));
+       ARDOUR_UI::instance()->set_tip(*solo_button,_("Solo"));
+       ARDOUR_UI::instance()->set_tip(*mute_button,_("Mute"));
+       ARDOUR_UI::instance()->set_tip(route_group_button, _("Route Group"));
+       ARDOUR_UI::instance()->set_tip(size_button,_("Display Height"));
+       ARDOUR_UI::instance()->set_tip(playlist_button,_("Playlist"));
+       ARDOUR_UI::instance()->set_tip(automation_button, _("Automation"));
+       ARDOUR_UI::instance()->set_tip(visual_button, _("Visual options"));
+       ARDOUR_UI::instance()->set_tip(hide_button, _("Hide this track"));
 
        label_view ();
 
@@ -230,15 +235,14 @@ RouteTimeAxisView::RouteTimeAxisView (PublicEditor& ed, Session* sess, boost::sh
 
        _y_position = -1;
 
-       _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->processors_changed.connect (*this, invalidator (*this), ui_bind (&RouteTimeAxisView::processors_changed, this, _1), gui_context());
+       _route->PropertyChanged.connect (*this, invalidator (*this), ui_bind (&RouteTimeAxisView::route_property_changed, this, _1), gui_context());
 
        if (is_track()) {
 
-               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());
+               track()->TrackModeChanged.connect (*this, invalidator (*this), boost::bind (&RouteTimeAxisView::track_mode_changed, this), gui_context());
+               track()->FreezeChange.connect (*this, invalidator (*this), boost::bind (&RouteTimeAxisView::map_frozen, this), gui_context());
+               track()->SpeedChanged.connect (*this, invalidator (*this), boost::bind (&RouteTimeAxisView::speed_changed, this), gui_context());
 
                /* pick up the correct freeze state */
                map_frozen ();
@@ -248,11 +252,13 @@ RouteTimeAxisView::RouteTimeAxisView (PublicEditor& ed, Session* sess, boost::sh
        _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)
-               );
-
+       PropertyList* plist = new PropertyList();
+       
+       plist->add (ARDOUR::Properties::edit, true);
+       plist->add (ARDOUR::Properties::mute, true);
+       plist->add (ARDOUR::Properties::solo, true);
+       
+       route_group_menu = new RouteGroupMenu (_session, plist);
        route_group_menu->GroupSelected.connect (sigc::mem_fun (*this, &RouteTimeAxisView::set_route_group_from_menu));
 
        gm.get_gain_slider().signal_scroll_event().connect(sigc::mem_fun(*this, &RouteTimeAxisView::controls_ebox_scroll), false);
@@ -341,13 +347,15 @@ RouteTimeAxisView::label_view ()
                name_entry.set_text (x);
        }
 
-       ARDOUR_UI::instance()->tooltips().set_tip (name_entry, x);
+       ARDOUR_UI::instance()->set_tip (name_entry, x);
 }
 
 void
-RouteTimeAxisView::route_name_changed ()
+RouteTimeAxisView::route_property_changed (const PropertyChange& what_changed)
 {
-       label_view ();
+       if (what_changed.contains (ARDOUR::Properties::name)) {
+               label_view ();
+       }
 }
 
 void
@@ -424,13 +432,13 @@ RouteTimeAxisView::build_automation_action_menu ()
 
        automation_action_menu->set_name ("ArdourContextMenu");
        
-       items.push_back (MenuElem (_("Show all automation"),
+       items.push_back (MenuElem (_("Show All Automation"),
                                   sigc::mem_fun(*this, &RouteTimeAxisView::show_all_automation)));
        
-       items.push_back (MenuElem (_("Show existing automation"),
+       items.push_back (MenuElem (_("Show Existing Automation"),
                                   sigc::mem_fun(*this, &RouteTimeAxisView::show_existing_automation)));
        
-       items.push_back (MenuElem (_("Hide all automation"),
+       items.push_back (MenuElem (_("Hide All Automation"),
                                   sigc::mem_fun(*this, &RouteTimeAxisView::hide_all_automation)));
        
        /* attach the plugin submenu. It may have previously been used elsewhere, so we detach it first. */
@@ -497,39 +505,43 @@ RouteTimeAxisView::build_display_menu ()
 
                RadioMenuItem::Group align_group;
 
-               alignment_items.push_back (RadioMenuElem (align_group, _("Align with existing material"),
+               alignment_items.push_back (RadioMenuElem (align_group, _("Align With Existing Material"),
                                        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)
+               if (track()->alignment_style() == ExistingMaterial) {
                        align_existing_item->set_active();
+               }
 
-               alignment_items.push_back (RadioMenuElem (align_group, _("Align with capture time"),
+               alignment_items.push_back (RadioMenuElem (align_group, _("Align With Capture Time"),
                                        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)
+               if (track()->alignment_style() == CaptureTime) {
                        align_capture_item->set_active();
+               }
 
                if (!Profile->get_sae()) {
+
                        items.push_back (MenuElem (_("Alignment"), *alignment_menu));
-                       get_diskstream()->AlignmentStyleChanged.connect (route_connections, boost::bind (&RouteTimeAxisView::align_style_changed, this), gui_context());
+                       track()->AlignmentStyleChanged.connect (route_connections, invalidator (*this), boost::bind (&RouteTimeAxisView::align_style_changed, this), gui_context());
 
                        RadioMenuItem::Group mode_group;
-                       items.push_back (RadioMenuElem (mode_group, _("Normal mode"), sigc::bind (
+                       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"), sigc::bind (
+                       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"),
+                       items.push_back (RadioMenuElem (mode_group, _("Non-Layered Mode"),
                                                        sigc::bind (sigc::mem_fun (*this, &RouteTimeAxisView::set_track_mode), ARDOUR::NonLayered)));
                        non_layered_track_mode_item = dynamic_cast<RadioMenuItem*>(&items.back());
 
 
-
+                       _ignore_track_mode_change = true;
+                       
                        switch (track()->mode()) {
                        case ARDOUR::Destructive:
                                destructive_track_mode_item->set_active ();
@@ -541,17 +553,16 @@ RouteTimeAxisView::build_display_menu ()
                                non_layered_track_mode_item->set_active ();
                                break;
                        }
+                       
+                       _ignore_track_mode_change = false;
                }
 
-               get_diskstream()->AlignmentStyleChanged.connect (route_connections, boost::bind (&RouteTimeAxisView::align_style_changed, this), gui_context());
-
-               mode_menu = build_mode_menu();
-               if (mode_menu)
-                       items.push_back (MenuElem (_("Mode"), *mode_menu));
+               track()->AlignmentStyleChanged.connect (route_connections, invalidator (*this), boost::bind (&RouteTimeAxisView::align_style_changed, this), gui_context());
 
                color_mode_menu = build_color_mode_menu();
-               if (color_mode_menu)
+               if (color_mode_menu) {
                        items.push_back (MenuElem (_("Color Mode"), *color_mode_menu));
+               }
 
                items.push_back (SeparatorElem());
        }
@@ -580,6 +591,10 @@ static bool __reset_item (RadioMenuItem* item, RadioMenuItem* item_2)
 void
 RouteTimeAxisView::set_track_mode (TrackMode mode)
 {
+       if (_ignore_track_mode_change) {
+               return;
+       }
+       
        RadioMenuItem* item;
        RadioMenuItem* other_item;
        RadioMenuItem* other_item_2;
@@ -606,7 +621,7 @@ RouteTimeAxisView::set_track_mode (TrackMode mode)
                return;
        }
 
-       if (item && other_item && other_item_2 && item->get_active() && track()->mode() != mode) {
+       if (item && other_item && other_item_2 && track()->mode() != mode) {
                _set_track_mode (track().get(), mode, other_item, other_item_2);
        }
 }
@@ -624,6 +639,7 @@ RouteTimeAxisView::_set_track_mode (Track* track, TrackMode mode, RadioMenuItem*
                        return;
                } else {
                        cerr << "would bounce this one\n";
+                       /* XXX: radio menu item becomes inconsistent with track state in this case */
                        return;
                }
        }
@@ -781,7 +797,7 @@ RouteTimeAxisView::set_height (uint32_t h)
 
                gm.get_gain_slider().show();
                mute_button->show();
-               if (!_route || _route->is_control()) {
+               if (!_route || _route->is_monitor()) {
                        solo_button->hide();
                } else {
                        solo_button->show();
@@ -807,7 +823,7 @@ RouteTimeAxisView::set_height (uint32_t h)
 
                gm.get_gain_slider().hide();
                mute_button->show();
-               if (!_route || _route->is_control()) {
+               if (!_route || _route->is_monitor()) {
                        solo_button->hide();
                } else {
                        solo_button->show();
@@ -885,8 +901,8 @@ RouteTimeAxisView::set_samples_per_unit (double spu)
 {
        double speed = 1.0;
 
-       if (get_diskstream() != 0) {
-               speed = get_diskstream()->speed();
+       if (track()) {
+               speed = track()->speed();
        }
 
        if (_view) {
@@ -899,7 +915,7 @@ RouteTimeAxisView::set_samples_per_unit (double spu)
 void
 RouteTimeAxisView::align_style_changed ()
 {
-       switch (get_diskstream()->alignment_style()) {
+       switch (track()->alignment_style()) {
        case ExistingMaterial:
                if (!align_existing_item->get_active()) {
                        align_existing_item->set_active();
@@ -932,7 +948,7 @@ RouteTimeAxisView::set_align_style (AlignStyle style)
        }
 
        if (item->get_active()) {
-               get_diskstream()->set_align_style (style);
+               track()->set_align_style (style);
        }
 }
 
@@ -942,15 +958,18 @@ RouteTimeAxisView::rename_current_playlist ()
        ArdourPrompter prompter (true);
        string name;
 
-       boost::shared_ptr<Diskstream> ds = get_diskstream();
-       if (!ds || ds->destructive())
+       boost::shared_ptr<Track> tr = track();
+       if (!tr || tr->destructive()) {
                return;
+       }
 
-       boost::shared_ptr<Playlist> pl = ds->playlist();
-       if (!pl)
+       boost::shared_ptr<Playlist> pl = tr->playlist();
+       if (!pl) {
                return;
+       }
 
-       prompter.set_prompt (_("Name for playlist"));
+       prompter.set_title (_("Rename Playlist"));
+       prompter.set_prompt (_("New name for playlist:"));
        prompter.set_initial_text (pl->name());
        prompter.add_button (_("Rename"), Gtk::RESPONSE_ACCEPT);
        prompter.set_response_sensitive (Gtk::RESPONSE_ACCEPT, false);
@@ -1008,13 +1027,15 @@ RouteTimeAxisView::use_copy_playlist (bool prompt, vector<boost::shared_ptr<Play
 {
        string name;
 
-       boost::shared_ptr<Diskstream> ds = get_diskstream();
-       if (!ds || ds->destructive())
+       boost::shared_ptr<Track> tr = track ();
+       if (!tr || tr->destructive()) {
                return;
+       }
 
-       boost::shared_ptr<const Playlist> pl = ds->playlist();
-       if (!pl)
+       boost::shared_ptr<const Playlist> pl = tr->playlist();
+       if (!pl) {
                return;
+       }
 
        name = pl->name();
 
@@ -1033,7 +1054,8 @@ RouteTimeAxisView::use_copy_playlist (bool prompt, vector<boost::shared_ptr<Play
 
                ArdourPrompter prompter (true);
 
-               prompter.set_prompt (_("Name for Playlist"));
+               prompter.set_title (_("New Copy Playlist"));
+               prompter.set_prompt (_("Name for new playlist:"));
                prompter.set_initial_text (name);
                prompter.add_button (Gtk::Stock::NEW, Gtk::RESPONSE_ACCEPT);
                prompter.set_response_sensitive (Gtk::RESPONSE_ACCEPT, true);
@@ -1050,8 +1072,8 @@ RouteTimeAxisView::use_copy_playlist (bool prompt, vector<boost::shared_ptr<Play
        }
 
        if (name.length()) {
-               ds->use_copy_playlist ();
-               ds->playlist()->set_name (name);
+               tr->use_copy_playlist ();
+               tr->playlist()->set_name (name);
        }
 }
 
@@ -1060,13 +1082,15 @@ RouteTimeAxisView::use_new_playlist (bool prompt, vector<boost::shared_ptr<Playl
 {
        string name;
 
-       boost::shared_ptr<Diskstream> ds = get_diskstream();
-       if (!ds || ds->destructive())
+       boost::shared_ptr<Track> tr = track ();
+       if (!tr || tr->destructive()) {
                return;
+       }
 
-       boost::shared_ptr<const Playlist> pl = ds->playlist();
-       if (!pl)
+       boost::shared_ptr<const Playlist> pl = tr->playlist();
+       if (!pl) {
                return;
+       }
 
        name = pl->name();
 
@@ -1083,7 +1107,8 @@ RouteTimeAxisView::use_new_playlist (bool prompt, vector<boost::shared_ptr<Playl
 
                ArdourPrompter prompter (true);
 
-               prompter.set_prompt (_("Name for Playlist"));
+               prompter.set_title (_("New Playlist"));
+               prompter.set_prompt (_("Name for new playlist:"));
                prompter.set_initial_text (name);
                prompter.add_button (Gtk::Stock::NEW, Gtk::RESPONSE_ACCEPT);
                prompter.set_response_sensitive (Gtk::RESPONSE_ACCEPT, true);
@@ -1099,21 +1124,23 @@ RouteTimeAxisView::use_new_playlist (bool prompt, vector<boost::shared_ptr<Playl
        }
 
        if (name.length()) {
-               ds->use_new_playlist ();
-               ds->playlist()->set_name (name);
+               tr->use_new_playlist ();
+               tr->playlist()->set_name (name);
        }
 }
 
 void
 RouteTimeAxisView::clear_playlist ()
 {
-       boost::shared_ptr<Diskstream> ds = get_diskstream();
-       if (!ds || ds->destructive())
+       boost::shared_ptr<Track> tr = track ();
+       if (!tr || tr->destructive()) {
                return;
+       }
 
-       boost::shared_ptr<Playlist> pl = ds->playlist();
-       if (!pl)
+       boost::shared_ptr<Playlist> pl = tr->playlist();
+       if (!pl) {
                return;
+       }
 
        _editor.clear_playlist (pl);
 }
@@ -1121,20 +1148,15 @@ RouteTimeAxisView::clear_playlist ()
 void
 RouteTimeAxisView::speed_changed ()
 {
-       Gtkmm2ext::UI::instance()->call_slot (boost::bind (&RouteTimeAxisView::reset_samples_per_unit, this));
-}
-
-void
-RouteTimeAxisView::diskstream_changed ()
-{
-       Gtkmm2ext::UI::instance()->call_slot (boost::bind (&RouteTimeAxisView::update_diskstream_display, this));
+       Gtkmm2ext::UI::instance()->call_slot (invalidator (*this), boost::bind (&RouteTimeAxisView::reset_samples_per_unit, this));
 }
 
 void
 RouteTimeAxisView::update_diskstream_display ()
 {
-       if (!get_diskstream()) // bus
+       if (!track()) {
                return;
+       }
 
        map_frozen ();
 }
@@ -1197,8 +1219,8 @@ RouteTimeAxisView::get_selectables (nframes_t start, nframes_t end, double top,
 {
        double speed = 1.0;
 
-       if (get_diskstream() != 0) {
-               speed = get_diskstream()->speed();
+       if (track() != 0) {
+               speed = track()->speed();
        }
 
        nframes_t start_adjusted = session_frame_to_track_frame(start, speed);
@@ -1276,10 +1298,10 @@ RouteTimeAxisView::name() const
 boost::shared_ptr<Playlist>
 RouteTimeAxisView::playlist () const
 {
-       boost::shared_ptr<Diskstream> ds;
+       boost::shared_ptr<Track> tr;
 
-       if ((ds = get_diskstream()) != 0) {
-               return ds->playlist();
+       if ((tr = track()) != 0) {
+               return tr->playlist();
        } else {
                return boost::shared_ptr<Playlist> ();
        }
@@ -1334,11 +1356,10 @@ RouteTimeAxisView::hide_click ()
 boost::shared_ptr<Region>
 RouteTimeAxisView::find_next_region (nframes_t pos, RegionPoint point, int32_t dir)
 {
-       boost::shared_ptr<Diskstream> stream;
-       boost::shared_ptr<Playlist> playlist;
+       boost::shared_ptr<Playlist> pl = playlist ();
 
-       if ((stream = get_diskstream()) != 0 && (playlist = stream->playlist()) != 0) {
-               return playlist->find_next_region (pos, point, dir);
+       if (pl) {
+               return pl->find_next_region (pos, point, dir);
        }
 
        return boost::shared_ptr<Region> ();
@@ -1347,11 +1368,10 @@ RouteTimeAxisView::find_next_region (nframes_t pos, RegionPoint point, int32_t d
 nframes64_t
 RouteTimeAxisView::find_next_region_boundary (nframes64_t pos, int32_t dir)
 {
-       boost::shared_ptr<Diskstream> stream;
-       boost::shared_ptr<Playlist> playlist;
+       boost::shared_ptr<Playlist> pl = playlist ();
 
-       if ((stream = get_diskstream()) != 0 && (playlist = stream->playlist()) != 0) {
-               return playlist->find_next_region_boundary (pos, dir);
+       if (pl) {
+               return pl->find_next_region_boundary (pos, dir);
        }
 
        return -1;
@@ -1361,19 +1381,19 @@ bool
 RouteTimeAxisView::cut_copy_clear (Selection& selection, CutCopyOp op)
 {
        boost::shared_ptr<Playlist> what_we_got;
-       boost::shared_ptr<Diskstream> ds = get_diskstream();
+       boost::shared_ptr<Track> tr = track ();
        boost::shared_ptr<Playlist> playlist;
        bool ret = false;
 
-       if (ds == 0) {
+       if (tr == 0) {
                /* route is a bus, not a track */
                return false;
        }
 
-       playlist = ds->playlist();
+       playlist = tr->playlist();
 
        TimeSelection time (selection.time);
-       float speed = ds->speed();
+       float const speed = tr->speed();
        if (speed != 1.0f) {
                for (TimeSelection::iterator i = time.begin(); i != time.end(); ++i) {
                        (*i).start = session_frame_to_track_frame((*i).start, speed);
@@ -1381,12 +1401,22 @@ RouteTimeAxisView::cut_copy_clear (Selection& selection, CutCopyOp op)
                }
        }
 
-       XMLNode &before = playlist->get_state();
+        playlist->clear_history ();
+        playlist->clear_owned_history ();
+
        switch (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()));
+
+                        vector<StatefulDiffCommand*> cmds;
+                        
+                        playlist->rdiff (cmds);
+                        
+                        for (vector<StatefulDiffCommand*>::iterator c = cmds.begin(); c != cmds.end(); ++c) {
+                                _session->add_command (*c);
+                        }
+                        _session->add_command (new StatefulDiffCommand (playlist));
                        ret = true;
                }
                break;
@@ -1398,7 +1428,14 @@ 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()));
+                        vector<StatefulDiffCommand*> cmds;
+                        
+                        playlist->rdiff (cmds);
+                        
+                        for (vector<StatefulDiffCommand*>::iterator c = cmds.begin(); c != cmds.end(); ++c) {
+                                _session->add_command (*c);
+                        }
+                        _session->add_command (new StatefulDiffCommand (playlist));
                        what_we_got->release ();
                        ret = true;
                }
@@ -1415,7 +1452,7 @@ RouteTimeAxisView::paste (nframes_t pos, float times, Selection& selection, size
                return false;
        }
 
-       boost::shared_ptr<Playlist> playlist = get_diskstream()->playlist();
+       boost::shared_ptr<Playlist> pl = playlist ();
        PlaylistSelection::iterator p;
 
        for (p = selection.playlists.begin(); p != selection.playlists.end() && nth; ++p, --nth) {}
@@ -1424,13 +1461,13 @@ RouteTimeAxisView::paste (nframes_t pos, float times, Selection& selection, size
                return false;
        }
 
-       if (get_diskstream()->speed() != 1.0f) {
-               pos = session_frame_to_track_frame(pos, get_diskstream()->speed() );
+       if (track()->speed() != 1.0f) {
+               pos = session_frame_to_track_frame (pos, track()->speed());
        }
 
-       XMLNode &before = playlist->get_state();
-       playlist->paste (*p, pos, times);
-       _session->add_command( new MementoCommand<Playlist>(*playlist, &before, &playlist->get_state()));
+        pl->clear_history ();
+       pl->paste (*p, pos, times);
+       _session->add_command (new StatefulDiffCommand (pl));
 
        return true;
 }
@@ -1450,6 +1487,12 @@ RouteTimeAxisView::get_child_list()
 }
 
 
+struct PlaylistSorter {
+    bool operator() (boost::shared_ptr<Playlist> a, boost::shared_ptr<Playlist> b) const {
+            return a->sort_id() < b->sort_id();
+    }
+};
+
 void
 RouteTimeAxisView::build_playlist_menu (Gtk::Menu * menu)
 {
@@ -1465,39 +1508,43 @@ RouteTimeAxisView::build_playlist_menu (Gtk::Menu * menu)
 
        delete playlist_menu;
 
-       playlist_menu = new Menu;
-       playlist_menu->set_name ("ArdourContextMenu");
 
-       vector<boost::shared_ptr<Playlist> > playlists;
-       boost::shared_ptr<Diskstream> ds = get_diskstream();
+        vector<boost::shared_ptr<Playlist> > playlists, playlists_tr;
+       boost::shared_ptr<Track> tr = track();
        RadioMenuItem::Group playlist_group;
 
        _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(), 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(), sigc::bind (sigc::mem_fun (*this, &RouteTimeAxisView::use_playlist),
-                                                                                                    boost::weak_ptr<Playlist>(*i))));
-                       static_cast<RadioMenuItem*>(&playlist_items.back())->set_active();
-
+        /* find the playlists for this diskstream */
+        for (vector<boost::shared_ptr<Playlist> >::iterator i = playlists.begin(); i != playlists.end(); ++i) {
+                if (((*i)->get_orig_diskstream_id() == tr->diskstream_id()) || (tr->playlist()->id() == (*i)->id())) {
+                        playlists_tr.push_back(*i);
+                }
+        }
+
+        /* sort the playlists */
+        PlaylistSorter cmp;
+        sort (playlists_tr.begin(), playlists_tr.end(), cmp);
+        
+        /* add the playlists to the menu */
+        for (vector<boost::shared_ptr<Playlist> >::iterator i = playlists_tr.begin(); i != playlists_tr.end(); ++i) {
+                playlist_items.push_back (RadioMenuElem (playlist_group, (*i)->name()));
+                RadioMenuItem *item = static_cast<RadioMenuItem*>(&playlist_items.back());
+                item->signal_toggled().connect(sigc::bind (sigc::mem_fun (*this, &RouteTimeAxisView::use_playlist), item, boost::weak_ptr<Playlist> (*i)));
+                
+                if (tr->playlist()->id() == (*i)->id()) {
+                        item->set_active();
+                        
                }
        }
-
+        
        playlist_items.push_back (SeparatorElem());
-       playlist_items.push_back (MenuElem (_("Rename"), sigc::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"), 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)));
+               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
@@ -1510,14 +1557,19 @@ RouteTimeAxisView::build_playlist_menu (Gtk::Menu * menu)
        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 ..."), sigc::mem_fun(*this, &RouteTimeAxisView::show_playlist_selector)));
+       playlist_items.push_back (MenuElem(_("Select from all..."), sigc::mem_fun(*this, &RouteTimeAxisView::show_playlist_selector)));
 }
 
 void
-RouteTimeAxisView::use_playlist (boost::weak_ptr<Playlist> wpl)
+RouteTimeAxisView::use_playlist (RadioMenuItem *item, boost::weak_ptr<Playlist> wpl)
 {
        assert (is_track());
 
+        // exit if we were triggered by deactivating the old playlist
+        if (!item->get_active()) {
+                return;
+        }
+
        boost::shared_ptr<Playlist> pl (wpl.lock());
 
        if (!pl) {
@@ -1527,13 +1579,12 @@ RouteTimeAxisView::use_playlist (boost::weak_ptr<Playlist> wpl)
        boost::shared_ptr<AudioPlaylist> apl = boost::dynamic_pointer_cast<AudioPlaylist> (pl);
 
        if (apl) {
-               if (get_diskstream()->playlist() == apl) {
-                       // radio button cotnrols mean this function is called for both the
-                       // old and new playlist
+               if (track()->playlist() == apl) {
+                        // exit when use_playlist is called by the creation of the playlist menu
+                        // or the playlist choice is unchanged
                        return;
                }
-               get_diskstream()->use_playlist (apl);
-
+               track()->use_playlist (apl);
 
                if (route_group() && route_group()->is_active()) {
                        std::string group_string = "."+route_group()->name()+".";
@@ -1564,10 +1615,10 @@ RouteTimeAxisView::use_playlist (boost::weak_ptr<Playlist> wpl)
                                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();
-                                       track->diskstream()->playlist()->set_name(playlist_name);
+                                       track->use_new_playlist();
+                                       track->playlist()->set_name(playlist_name);
                                } else {
-                                       track->diskstream()->use_playlist(ipl);
+                                       track->use_playlist(ipl);
                                }
                        }
                }
@@ -2247,9 +2298,9 @@ RouteTimeAxisView::clear_meter ()
 }
 
 void
-RouteTimeAxisView::meter_changed (void *src)
+RouteTimeAxisView::meter_changed ()
 {
-       ENSURE_GUI_THREAD (*this, &RouteTimeAxisView::meter_changed, src)
+       ENSURE_GUI_THREAD (*this, &RouteTimeAxisView::meter_changed)
        reset_meter();
 }
 
@@ -2379,18 +2430,21 @@ RouteTimeAxisView::set_button_names ()
 {
        rec_enable_button_label.set_text (_("r"));
 
-       if (Config->get_solo_control_is_listen_control()) {
-               switch (Config->get_listen_position()) {
-               case AfterFaderListen:
-                       solo_button_label.set_text (_("A"));
-                       break;
-               case PreFaderListen:
-                       solo_button_label.set_text (_("P"));
-                       break;
-               }
-       } else {
-               solo_button_label.set_text (_("s"));
-       }
-
+        if (_route && _route->solo_safe()) {
+                solo_button_label.set_text (X_("!"));
+        } else {
+                if (Config->get_solo_control_is_listen_control()) {
+                        switch (Config->get_listen_position()) {
+                        case AfterFaderListen:
+                                solo_button_label.set_text (_("A"));
+                                break;
+                        case PreFaderListen:
+                                solo_button_label.set_text (_("P"));
+                                break;
+                        }
+                } else {
+                        solo_button_label.set_text (_("s"));
+                }
+        }
        mute_button_label.set_text (_("m"));
 }