Really fixed conflicts. Ready to merge into trunk.
[ardour.git] / gtk2_ardour / editor_ops.cc
index 192269eb15bc3d6ff909d2b5b177491b918150a2..1e5f336e525bb1ef7e89064d9e82948db0f7bd72 100644 (file)
@@ -28,6 +28,7 @@
 #include <pbd/error.h>
 #include <pbd/basename.h>
 #include <pbd/pthread_utils.h>
+#include <pbd/memento_command.h>
 
 #include <gtkmm2ext/utils.h>
 #include <gtkmm2ext/choice.h>
@@ -36,9 +37,7 @@
 #include <ardour/session.h>
 #include <ardour/audioplaylist.h>
 #include <ardour/audioregion.h>
-#include <ardour/diskstream.h>
-#include <ardour/filesource.h>
-#include <ardour/externalsource.h>
+#include <ardour/audio_diskstream.h>
 #include <ardour/utils.h>
 #include <ardour/location.h>
 #include <ardour/named_selection.h>
@@ -53,7 +52,7 @@
 #include "audio_time_axis.h"
 #include "automation_time_axis.h"
 #include "streamview.h"
-#include "regionview.h"
+#include "audio_region_view.h"
 #include "rgb_macros.h"
 #include "selection_templates.h"
 #include "selection.h"
@@ -66,6 +65,7 @@
 
 using namespace std;
 using namespace ARDOUR;
+using namespace PBD;
 using namespace sigc;
 using namespace Gtk;
 using namespace Editing;
@@ -93,14 +93,78 @@ Editor::redo (uint32_t n)
 void
 Editor::set_meter_hold (int32_t cnt)
 {
+       Config->set_meter_hold_off(false);
+       Config->set_meter_hold_short(false);
+       Config->set_meter_hold_medium(false);
+       Config->set_meter_hold_long(false);
+
+       switch (cnt)
+       {
+               case 0:
+                Config->set_meter_hold_off(true);
+                break;
+               case 40:
+                Config->set_meter_hold_short(true);
+                break;
+               case 100:
+                Config->set_meter_hold_medium(true);
+                break;
+               case 200:
+                Config->set_meter_hold_long(true);
+                break;
+       }
+                
        if (session) {
                session->set_meter_hold (cnt);
        }
 }
 
 void
-Editor::set_meter_falloff (float val)
-{
+Editor::set_meter_falloff (int intval)
+{
+       float val = 0.0f; /* off */
+       std::string str;
+
+       Config->set_meter_falloff_off(false);
+       Config->set_meter_falloff_slowest(false);
+       Config->set_meter_falloff_slow(false);
+       Config->set_meter_falloff_medium(false);
+       Config->set_meter_falloff_fast(false);
+       Config->set_meter_falloff_faster(false);
+       Config->set_meter_falloff_fastest(false);
+       
+       switch (intval)
+       {
+               case 0:
+                val = 0.0f;
+                Config->set_meter_falloff_off(true);
+                break;
+               case 1:
+                val = 0.266f;
+                Config->set_meter_falloff_slowest(true);
+                break;
+               case 2:
+                val = 0.342f;
+                Config->set_meter_falloff_slow(true);
+                break;
+               case 3:
+                val = 0.7f;
+                Config->set_meter_falloff_medium(true);
+                break;
+               case 4:
+                val = 1.1f;
+                Config->set_meter_falloff_fast(true);
+                break;
+               case 5:
+                val = 1.5f;
+                Config->set_meter_falloff_faster(true);
+                break;
+               case 6:
+                val = 2.5f;
+                Config->set_meter_falloff_fastest(true);
+                break;
+       }
+       
        if (session) {
                session->set_meter_falloff (val);
        }
@@ -123,30 +187,33 @@ Editor::split_region ()
 void
 Editor::split_region_at (jack_nframes_t where)
 {
-       split_regions_at (where, selection->audio_regions);
+       split_regions_at (where, selection->regions);
 }
 
 void
-Editor::split_regions_at (jack_nframes_t where, AudioRegionSelection& regions)
+Editor::split_regions_at (jack_nframes_t where, RegionSelection& regions)
 {
        begin_reversible_command (_("split"));
 
        snap_to (where);
-       for (AudioRegionSelection::iterator a = regions.begin(); a != regions.end(); ) {
+       for (RegionSelection::iterator a = regions.begin(); a != regions.end(); ) {
 
-               AudioRegionSelection::iterator tmp;
+               RegionSelection::iterator tmp;
                
                tmp = a;
                ++tmp;
 
-               Playlist* pl = (*a)->region.playlist();
+               Playlist* pl = (*a)->region().playlist();
 
-               _new_regionviews_show_envelope = (*a)->envelope_visible();
+               AudioRegionView* const arv = dynamic_cast<AudioRegionView*>(*a);
+               if (arv)
+                       _new_regionviews_show_envelope = arv->envelope_visible();
                
                if (pl) {
-                       session->add_undo (pl->get_memento());
-                       pl->split_region ((*a)->region, where);
-                       session->add_redo_no_execute (pl->get_memento());
+                        XMLNode &before = pl->get_state();
+                       pl->split_region ((*a)->region(), where);
+                        XMLNode &after = pl->get_state();
+                        session->add_command(new MementoCommand<Playlist>(*pl, before, after));
                }
 
                a = tmp;
@@ -166,16 +233,17 @@ Editor::remove_clicked_region ()
        Playlist* playlist = clicked_audio_trackview->playlist();
        
        begin_reversible_command (_("remove region"));
-       session->add_undo (playlist->get_memento());
-       playlist->remove_region (&clicked_regionview->region);
-       session->add_redo_no_execute (playlist->get_memento());
+        XMLNode &before = playlist->get_state();
+       playlist->remove_region (&clicked_regionview->region());
+        XMLNode &after = playlist->get_state();
+       session->add_command(new MementoCommand<Playlist>(*playlist, before, after));
        commit_reversible_command ();
 }
 
 void
 Editor::destroy_clicked_region ()
 {
-       int32_t selected = selection->audio_regions.size();
+       int32_t selected = selection->regions.size();
 
        if (!session || clicked_regionview == 0 && selected == 0) {
                return;
@@ -190,46 +258,46 @@ Do you really want to destroy %1 ?"),
                           (selected > 1 ? 
                            _("these regions") : _("this region")));
 
+       choices.push_back (_("No, do nothing."));
+
        if (selected > 1) {
                choices.push_back (_("Yes, destroy them."));
        } else {
                choices.push_back (_("Yes, destroy it."));
        }
 
-       choices.push_back (_("No, do nothing."));
-
        Gtkmm2ext::Choice prompter (prompt, choices);
        
-       if (prompter.run() != 0) { /* first choice */
+       if (prompter.run() == 0) { /* first choice */
                return;
        }
 
        if (selected > 0) {
                list<Region*> r;
 
-               for (AudioRegionSelection::iterator i = selection->audio_regions.begin(); i != selection->audio_regions.end(); ++i) {
-                       r.push_back (&(*i)->region);
+               for (RegionSelection::iterator i = selection->regions.begin(); i != selection->regions.end(); ++i) {
+                       r.push_back (&(*i)->region());
                }
 
                session->destroy_regions (r);
 
        } else if (clicked_regionview) {
-               session->destroy_region (&clicked_regionview->region);
+               session->destroy_region (&clicked_regionview->region());
        } 
 }
 
-AudioRegion *
+Region *
 Editor::select_region_for_operation (int dir, TimeAxisView **tv)
 {
-       AudioRegionView* rv;
-       AudioRegion *region;
+       RegionView* rv;
+       Region *region;
        jack_nframes_t start = 0;
 
        if (selection->time.start () == selection->time.end_frame ()) {
                
                /* no current selection-> is there a selected regionview? */
 
-               if (selection->audio_regions.empty()) {
+               if (selection->regions.empty()) {
                        return 0;
                }
 
@@ -237,26 +305,26 @@ Editor::select_region_for_operation (int dir, TimeAxisView **tv)
 
        region = 0;
 
-       if (!selection->audio_regions.empty()) {
+       if (!selection->regions.empty()) {
 
-               rv = *(selection->audio_regions.begin());
+               rv = *(selection->regions.begin());
                (*tv) = &rv->get_time_axis_view();
-               region = &rv->region;
+               region = &rv->region();
 
        } else if (!selection->tracks.empty()) {
 
                (*tv) = selection->tracks.front();
 
-               AudioTimeAxisView* atv;
+               RouteTimeAxisView* rtv;
 
-               if ((atv = dynamic_cast<AudioTimeAxisView*> (*tv)) != 0) {
+               if ((rtv = dynamic_cast<RouteTimeAxisView*> (*tv)) != 0) {
                        Playlist *pl;
                        
-                       if ((pl = atv->playlist()) == 0) {
+                       if ((pl = rtv->playlist()) == 0) {
                                return 0;
                        }
                        
-                       region = dynamic_cast<AudioRegion*> (pl->top_region_at (start));
+                       region = pl->top_region_at (start);
                }
        } 
        
@@ -328,12 +396,12 @@ Editor::nudge_forward (bool next)
 
        if (!session) return;
        
-       if (!selection->audio_regions.empty()) {
+       if (!selection->regions.empty()) {
 
                begin_reversible_command (_("nudge forward"));
 
-               for (AudioRegionSelection::iterator i = selection->audio_regions.begin(); i != selection->audio_regions.end(); ++i) {
-                       AudioRegion& r ((*i)->region);
+               for (RegionSelection::iterator i = selection->regions.begin(); i != selection->regions.end(); ++i) {
+                       Region& r ((*i)->region());
                        
                        distance = get_nudge_distance (r.position(), next_distance);
 
@@ -341,9 +409,10 @@ Editor::nudge_forward (bool next)
                                distance = next_distance;
                        }
 
-                       session->add_undo (r.playlist()->get_memento());
+                        XMLNode &before = r.playlist()->get_state();
                        r.set_position (r.position() + distance, this);
-                       session->add_redo_no_execute (r.playlist()->get_memento());
+                        XMLNode &after = r.playlist()->get_state();
+                       session->add_command (new MementoCommand<Playlist>(*(r.playlist()), before, after));
                }
 
                commit_reversible_command ();
@@ -362,12 +431,12 @@ Editor::nudge_backward (bool next)
 
        if (!session) return;
        
-       if (!selection->audio_regions.empty()) {
+       if (!selection->regions.empty()) {
 
                begin_reversible_command (_("nudge forward"));
 
-               for (AudioRegionSelection::iterator i = selection->audio_regions.begin(); i != selection->audio_regions.end(); ++i) {
-                       AudioRegion& r ((*i)->region);
+               for (RegionSelection::iterator i = selection->regions.begin(); i != selection->regions.end(); ++i) {
+                       Region& r ((*i)->region());
 
                        distance = get_nudge_distance (r.position(), next_distance);
                        
@@ -375,14 +444,15 @@ Editor::nudge_backward (bool next)
                                distance = next_distance;
                        }
 
-                       session->add_undo (r.playlist()->get_memento());
+                        XMLNode &before = r.playlist()->get_state();
                        
                        if (r.position() > distance) {
                                r.set_position (r.position() - distance, this);
                        } else {
                                r.set_position (0, this);
                        }
-                       session->add_redo_no_execute (r.playlist()->get_memento());
+                        XMLNode &after = r.playlist()->get_state();
+                       session->add_command(new MementoCommand<Playlist>(*(r.playlist()), before, after));
                }
 
                commit_reversible_command ();
@@ -406,18 +476,19 @@ Editor::nudge_forward_capture_offset ()
 
        if (!session) return;
        
-       if (!selection->audio_regions.empty()) {
+       if (!selection->regions.empty()) {
 
                begin_reversible_command (_("nudge forward"));
 
                distance = session->worst_output_latency();
 
-               for (AudioRegionSelection::iterator i = selection->audio_regions.begin(); i != selection->audio_regions.end(); ++i) {
-                       AudioRegion& r ((*i)->region);
+               for (RegionSelection::iterator i = selection->regions.begin(); i != selection->regions.end(); ++i) {
+                       Region& r ((*i)->region());
                        
-                       session->add_undo (r.playlist()->get_memento());
+                       XMLNode &before = r.playlist()->get_state();
                        r.set_position (r.position() + distance, this);
-                       session->add_redo_no_execute (r.playlist()->get_memento());
+                       XMLNode &after = r.playlist()->get_state();
+                       session->add_command(new MementoCommand<Playlist>(*(r.playlist()), before, after));
                }
 
                commit_reversible_command ();
@@ -432,23 +503,24 @@ Editor::nudge_backward_capture_offset ()
 
        if (!session) return;
        
-       if (!selection->audio_regions.empty()) {
+       if (!selection->regions.empty()) {
 
                begin_reversible_command (_("nudge forward"));
 
                distance = session->worst_output_latency();
 
-               for (AudioRegionSelection::iterator i = selection->audio_regions.begin(); i != selection->audio_regions.end(); ++i) {
-                       AudioRegion& r ((*i)->region);
+               for (RegionSelection::iterator i = selection->regions.begin(); i != selection->regions.end(); ++i) {
+                       Region& r ((*i)->region());
 
-                       session->add_undo (r.playlist()->get_memento());
+                        XMLNode &before = r.playlist()->get_state();
                        
                        if (r.position() > distance) {
                                r.set_position (r.position() - distance, this);
                        } else {
                                r.set_position (0, this);
                        }
-                       session->add_redo_no_execute (r.playlist()->get_memento());
+                        XMLNode &after = r.playlist()->get_state();
+                       session->add_command(new MementoCommand<Playlist>(*(r.playlist()), before, after));
                }
 
                commit_reversible_command ();
@@ -719,8 +791,8 @@ Editor::cursor_to_selection_start (Cursor *cursor)
        jack_nframes_t pos = 0;
        switch (mouse_mode) {
        case MouseObject:
-               if (!selection->audio_regions.empty()) {
-                       pos = selection->audio_regions.start();
+               if (!selection->regions.empty()) {
+                       pos = selection->regions.start();
                }
                break;
 
@@ -748,8 +820,8 @@ Editor::cursor_to_selection_end (Cursor *cursor)
 
        switch (mouse_mode) {
        case MouseObject:
-               if (!selection->audio_regions.empty()) {
-                       pos = selection->audio_regions.end_frame();
+               if (!selection->regions.empty()) {
+                       pos = selection->regions.end_frame();
                }
                break;
 
@@ -1225,9 +1297,10 @@ Editor::add_location_from_selection ()
        Location *location = new Location (start, end, "selection");
 
        session->begin_reversible_command (_("add marker"));
-       session->add_undo (session->locations()->get_memento());
+        XMLNode &before = session->locations()->get_state();
        session->locations()->add (location, true);
-       session->add_redo_no_execute (session->locations()->get_memento());
+        XMLNode &after = session->locations()->get_state();
+       session->add_command(new MementoCommand<Locations>(*(session->locations()), before, after));
        session->commit_reversible_command ();
 }
 
@@ -1238,27 +1311,29 @@ Editor::add_location_from_playhead_cursor ()
        
        Location *location = new Location (where, where, "mark", Location::IsMark);
        session->begin_reversible_command (_("add marker"));
-       session->add_undo (session->locations()->get_memento());
+        XMLNode &before = session->locations()->get_state();
        session->locations()->add (location, true);
-       session->add_redo_no_execute (session->locations()->get_memento());
+        XMLNode &after = session->locations()->get_state();
+       session->add_command(new MementoCommand<Locations>(*(session->locations()), before, after));
        session->commit_reversible_command ();
 }
 
 void
 Editor::add_location_from_audio_region ()
 {
-       if (selection->audio_regions.empty()) {
+       if (selection->regions.empty()) {
                return;
        }
 
-       AudioRegionView* rv = *(selection->audio_regions.begin());
-       Region& region = rv->region;
+       RegionView* rv = *(selection->regions.begin());
+       Region& region = rv->region();
        
        Location *location = new Location (region.position(), region.last_frame(), region.name());
        session->begin_reversible_command (_("add marker"));
-       session->add_undo (session->locations()->get_memento());
+        XMLNode &before = session->locations()->get_state();
        session->locations()->add (location, true);
-       session->add_redo_no_execute (session->locations()->get_memento());
+        XMLNode &after = session->locations()->get_state();
+       session->add_command(new MementoCommand<Locations>(*(session->locations()), before, after));
        session->commit_reversible_command ();
 }
 
@@ -1351,18 +1426,27 @@ Editor::select_all_within (jack_nframes_t start, jack_nframes_t end, double top,
                }
                (*iter)->get_selectables (start, end, top, bot, touched);
        }
+
+       cerr << "select all within found " << touched.size() << endl;
+
        begin_reversible_command (_("select all within"));
        switch (op) {
        case Selection::Toggle:
+               cerr << "toggle\n";
                selection->add (touched);
                break;
        case Selection::Set:
+               cerr << "set\n";
                selection->set (touched);
                break;
        case Selection::Extend:
+               cerr << "extend\n";
                /* not defined yet */
                break;
        }
+
+       cerr << "selection now has " << selection->points.size() << endl;
+
        commit_reversible_command ();
        return !touched.empty();
 }
@@ -1370,12 +1454,12 @@ Editor::select_all_within (jack_nframes_t start, jack_nframes_t end, double top,
 void
 Editor::set_selection_from_audio_region ()
 {
-       if (selection->audio_regions.empty()) {
+       if (selection->regions.empty()) {
                return;
        }
 
-       AudioRegionView* rv = *(selection->audio_regions.begin());
-       Region& region = rv->region;
+       RegionView* rv = *(selection->regions.begin());
+       Region& region = rv->region();
        
        begin_reversible_command (_("set selection from region"));
        selection->set (0, region.position(), region.last_frame());
@@ -1665,9 +1749,10 @@ Editor::clear_markers ()
 {
        if (session) {
                session->begin_reversible_command (_("clear markers"));
-               session->add_undo (session->locations()->get_memento());
+                XMLNode &before = session->locations()->get_state();
                session->locations()->clear_markers ();
-               session->add_redo_no_execute (session->locations()->get_memento());
+                XMLNode &after = session->locations()->get_state();
+               session->add_command(new MementoCommand<Locations>(*(session->locations()), before, after));
                session->commit_reversible_command ();
        }
 }
@@ -1677,7 +1762,7 @@ Editor::clear_ranges ()
 {
        if (session) {
                session->begin_reversible_command (_("clear ranges"));
-               session->add_undo (session->locations()->get_memento());
+                XMLNode &before = session->locations()->get_state();
                
                Location * looploc = session->locations()->auto_loop_location();
                Location * punchloc = session->locations()->auto_punch_location();
@@ -1687,7 +1772,8 @@ Editor::clear_ranges ()
                if (looploc) session->locations()->add (looploc);
                if (punchloc) session->locations()->add (punchloc);
                
-               session->add_redo_no_execute (session->locations()->get_memento());
+                XMLNode &after = session->locations()->get_state();
+               session->add_command(new MementoCommand<Locations>(*(session->locations()), before, after));
                session->commit_reversible_command ();
        }
 }
@@ -1696,9 +1782,10 @@ void
 Editor::clear_locations ()
 {
        session->begin_reversible_command (_("clear locations"));
-       session->add_undo (session->locations()->get_memento());
+        XMLNode &before = session->locations()->get_state();
        session->locations()->clear ();
-       session->add_redo_no_execute (session->locations()->get_memento());
+        XMLNode &after = session->locations()->get_state();
+       session->add_command(new MementoCommand<Locations>(*(session->locations()), before, after));
        session->commit_reversible_command ();
        session->locations()->clear ();
 }
@@ -1746,22 +1833,22 @@ Editor::insert_region_list_drag (AudioRegion& region, int x, int y)
        snap_to (where);
        
        begin_reversible_command (_("insert dragged region"));
-       session->add_undo (playlist->get_memento());
+        XMLNode &before = playlist->get_state();
        playlist->add_region (*(new AudioRegion (region)), where, 1.0);
-       session->add_redo_no_execute (playlist->get_memento());
+       session->add_command(new MementoCommand<Playlist>(*playlist, before, playlist->get_state()));
        commit_reversible_command ();
 }
 
 void
 Editor::insert_region_list_selection (float times)
 {
-       AudioTimeAxisView *tv = 0;
+       RouteTimeAxisView *tv = 0;
        Playlist *playlist;
 
        if (clicked_audio_trackview != 0) {
                tv = clicked_audio_trackview;
        } else if (!selection->tracks.empty()) {
-               if ((tv = dynamic_cast<AudioTimeAxisView*>(selection->tracks.front())) == 0) {
+               if ((tv = dynamic_cast<RouteTimeAxisView*>(selection->tracks.front())) == 0) {
                        return;
                }
        } else {
@@ -1782,9 +1869,9 @@ Editor::insert_region_list_selection (float times)
        Region* region = (*i)[region_list_columns.region];
 
        begin_reversible_command (_("insert region"));
-       session->add_undo (playlist->get_memento());
+        XMLNode &before = playlist->get_state();
        playlist->add_region (*(createRegion (*region)), edit_cursor->current_frame, times);
-       session->add_redo_no_execute (playlist->get_memento());
+       session->add_command(new MementoCommand<Playlist>(*playlist, before, playlist->get_state()));
        commit_reversible_command ();
 }
 
@@ -1856,23 +1943,23 @@ Editor::play_selection ()
 void
 Editor::play_selected_region ()
 {
-       if (!selection->audio_regions.empty()) {
-               AudioRegionView *rv = *(selection->audio_regions.begin());
+       if (!selection->regions.empty()) {
+               RegionView *rv = *(selection->regions.begin());
 
-               session->request_bounded_roll (rv->region.position(), rv->region.last_frame()); 
+               session->request_bounded_roll (rv->region().position(), rv->region().last_frame());     
        }
 }
 
 void
 Editor::loop_selected_region ()
 {
-       if (!selection->audio_regions.empty()) {
-               AudioRegionView *rv = *(selection->audio_regions.begin());
+       if (!selection->regions.empty()) {
+               RegionView *rv = *(selection->regions.begin());
                Location* tll;
 
                if ((tll = transport_loop_location()) != 0)  {
 
-                       tll->set (rv->region.position(), rv->region.last_frame());
+                       tll->set (rv->region().position(), rv->region().last_frame());
                        
                        // enable looping, reposition and start rolling
 
@@ -1915,10 +2002,10 @@ void
 Editor::toggle_region_mute ()
 {
        if (clicked_regionview) {
-               clicked_regionview->region.set_muted (!clicked_regionview->region.muted());
-       } else if (!selection->audio_regions.empty()) {
-               bool yn = ! (*selection->audio_regions.begin())->region.muted();
-               selection->foreach_audio_region (&AudioRegion::set_muted, yn);
+               clicked_regionview->region().set_muted (!clicked_regionview->region().muted());
+       } else if (!selection->regions.empty()) {
+               bool yn = ! (*selection->regions.begin())->region().muted();
+               selection->foreach_region (&Region::set_muted, yn);
        }
 }
 
@@ -1926,35 +2013,35 @@ void
 Editor::toggle_region_opaque ()
 {
        if (clicked_regionview) {
-               clicked_regionview->region.set_opaque (!clicked_regionview->region.opaque());
-       } else if (!selection->audio_regions.empty()) {
-               bool yn = ! (*selection->audio_regions.begin())->region.opaque();
-               selection->foreach_audio_region (&Region::set_opaque, yn);
+               clicked_regionview->region().set_opaque (!clicked_regionview->region().opaque());
+       } else if (!selection->regions.empty()) {
+               bool yn = ! (*selection->regions.begin())->region().opaque();
+               selection->foreach_region (&Region::set_opaque, yn);
        }
 }
 
 void
 Editor::raise_region ()
 {
-       selection->foreach_audio_region (&Region::raise);
+       selection->foreach_region (&Region::raise);
 }
 
 void
 Editor::raise_region_to_top ()
 {
-       selection->foreach_audio_region (&Region::raise_to_top);
+       selection->foreach_region (&Region::raise_to_top);
 }
 
 void
 Editor::lower_region ()
 {
-       selection->foreach_audio_region (&Region::lower);
+       selection->foreach_region (&Region::lower);
 }
 
 void
 Editor::lower_region_to_bottom ()
 {
-       selection->foreach_audio_region (&Region::lower_to_bottom);
+       selection->foreach_region (&Region::lower_to_bottom);
 }
 
 void
@@ -1975,7 +2062,7 @@ Editor::rename_region ()
        Button ok_button (_("OK"));
        Button cancel_button (_("Cancel"));
 
-       if (selection->audio_regions.empty()) {
+       if (selection->regions.empty()) {
                return;
        }
 
@@ -2006,7 +2093,7 @@ Editor::rename_region ()
        Main::run ();
 
        if (region_renamed) {
-               (*selection->audio_regions.begin())->region.set_name (entry.get_text());
+               (*selection->regions.begin())->region().set_name (entry.get_text());
                redisplay_regions ();
        }
 }
@@ -2020,7 +2107,7 @@ Editor::rename_region_finished (bool status)
 }
 
 void
-Editor::audition_playlist_region_via_route (AudioRegion& region, Route& route)
+Editor::audition_playlist_region_via_route (Region& region, Route& route)
 {
        if (session->is_auditioning()) {
                session->cancel_audition ();
@@ -2041,14 +2128,14 @@ Editor::audition_playlist_region_via_route (AudioRegion& region, Route& route)
 void
 Editor::audition_selected_region ()
 {
-       if (!selection->audio_regions.empty()) {
-               AudioRegionView* rv = *(selection->audio_regions.begin());
-               session->audition_region (rv->region);
+       if (!selection->regions.empty()) {
+               RegionView* rv = *(selection->regions.begin());
+               session->audition_region (rv->region());
        }
 }
 
 void
-Editor::audition_playlist_region_standalone (AudioRegion& region)
+Editor::audition_playlist_region_standalone (Region& region)
 {
        session->audition_region (region);
 }
@@ -2098,7 +2185,6 @@ Editor::region_from_selection ()
        jack_nframes_t selection_cnt = end - start + 1;
        
        for (TrackSelection::iterator i = selection->tracks.begin(); i != selection->tracks.end(); ++i) {
-
                AudioRegion *region;
                AudioRegion *current;
                Region* current_r;
@@ -2115,7 +2201,9 @@ Editor::region_from_selection ()
                        continue;
                }
 
-               if ((current = dynamic_cast<AudioRegion*> (current_r)) != 0) {
+               current = dynamic_cast<AudioRegion*> (current_r);
+               // FIXME: audio only
+               if (current != 0) {
                        internal_start = start - current->position();
                        session->region_name (new_name, current->name(), true);
                        region = new AudioRegion (*current, internal_start, selection_cnt, new_name);
@@ -2165,11 +2253,13 @@ Editor::split_multichannel_region ()
 {
        vector<AudioRegion*> v;
 
-       if (!clicked_regionview || clicked_regionview->region.n_channels() < 2) {
+       AudioRegionView* clicked_arv = dynamic_cast<AudioRegionView*>(clicked_regionview);
+       
+       if (!clicked_arv || clicked_arv->audio_region().n_channels() < 2) {
                return;
        }
 
-       clicked_regionview->region.separate_by_channel (*session, v);
+       clicked_arv->audio_region().separate_by_channel (*session, v);
 
        /* nothing else to do, really */
 }
@@ -2205,7 +2295,9 @@ Editor::separate_region_from_selection ()
                                                begin_reversible_command (_("separate"));
                                                doing_undo = true;
                                        }
-                                       if (doing_undo) session->add_undo ((playlist)->get_memento());
+                                        XMLNode *before;
+                                       if (doing_undo) 
+                                            before = &(playlist->get_state());
                        
                                        /* XXX need to consider musical time selections here at some point */
 
@@ -2215,7 +2307,8 @@ Editor::separate_region_from_selection ()
                                                playlist->partition ((jack_nframes_t)((*t).start * speed), (jack_nframes_t)((*t).end * speed), true);
                                        }
 
-                                       if (doing_undo) session->add_redo_no_execute (playlist->get_memento());
+                                       if (doing_undo) 
+                                            session->add_command(new MementoCommand<Playlist>(*playlist, *before, playlist->get_state()));
                                }
                        }
                }
@@ -2250,11 +2343,14 @@ Editor::separate_regions_using_location (Location& loc)
                        if (atv->is_audio_track()) {
                                        
                                if ((playlist = atv->playlist()) != 0) {
+                                        XMLNode *before;
                                        if (!doing_undo) {
                                                begin_reversible_command (_("separate"));
                                                doing_undo = true;
                                        }
-                                       if (doing_undo) session->add_undo ((playlist)->get_memento());
+                                       if (doing_undo) 
+                                            before = &(playlist->get_state());
+                                            
                        
                                        /* XXX need to consider musical time selections here at some point */
 
@@ -2262,7 +2358,8 @@ Editor::separate_regions_using_location (Location& loc)
 
 
                                        playlist->partition ((jack_nframes_t)(loc.start() * speed), (jack_nframes_t)(loc.end() * speed), true);
-                                       if (doing_undo) session->add_redo_no_execute (playlist->get_memento());
+                                       if (doing_undo) 
+                                            session->add_command(new MementoCommand<Playlist>(*playlist, *before, playlist->get_state()));
                                }
                        }
                }
@@ -2333,9 +2430,10 @@ Editor::crop_region_to_selection ()
                        end = min (selection->time.end_frame(), start + region->length() - 1);
                        cnt = end - start + 1;
 
-                       session->add_undo ((*i)->get_memento());
+                        XMLNode &before = (*i)->get_state();
                        region->trim_to (start, cnt, this);
-                       session->add_redo_no_execute ((*i)->get_memento());
+                        XMLNode &after = (*i)->get_state();
+                       session->add_command (new MementoCommand<Playlist>(*(*i), before, after));
                }
 
                commit_reversible_command ();
@@ -2347,7 +2445,7 @@ Editor::region_fill_track ()
 {
        jack_nframes_t end;
 
-       if (!session || selection->audio_regions.empty()) {
+       if (!session || selection->regions.empty()) {
                return;
        }
 
@@ -2355,9 +2453,15 @@ Editor::region_fill_track ()
 
        begin_reversible_command (_("region fill"));
 
-       for (AudioRegionSelection::iterator i = selection->audio_regions.begin(); i != selection->audio_regions.end(); ++i) {
+       for (RegionSelection::iterator i = selection->regions.begin(); i != selection->regions.end(); ++i) {
+
+               Region& region ((*i)->region());
+               
+               // FIXME
+               AudioRegion* const ar = dynamic_cast<AudioRegion*>(&region);
+               if (!ar)
+                       continue;
 
-               AudioRegion& region ((*i)->region);
                Playlist* pl = region.playlist();
 
                if (end <= region.last_frame()) {
@@ -2370,9 +2474,9 @@ Editor::region_fill_track ()
                        return;
                }
 
-               session->add_undo (pl->get_memento());
-               pl->add_region (*(new AudioRegion (region)), region.last_frame(), times);
-               session->add_redo_no_execute (pl->get_memento());
+                XMLNode &before = pl->get_state();
+               pl->add_region (*(new AudioRegion (*ar)), ar->last_frame(), times);
+               session->add_command (new MementoCommand<Playlist>(*pl, before, pl->get_state()));
        }
 
        commit_reversible_command ();
@@ -2420,9 +2524,9 @@ Editor::region_fill_selection ()
                        continue;
                }               
                
-               session->add_undo (playlist->get_memento());
+                XMLNode &before = playlist->get_state();
                playlist->add_region (*(createRegion (*region)), start, times);
-               session->add_redo_no_execute (playlist->get_memento());
+               session->add_command (new MementoCommand<Playlist>(*playlist, before, playlist->get_state()));
        }
        
        commit_reversible_command ();                   
@@ -2437,9 +2541,10 @@ Editor::set_a_regions_sync_position (Region& region, jack_nframes_t position)
                return;
        }
        begin_reversible_command (_("set region sync position"));
-       session->add_undo (region.playlist()->get_memento());
+        XMLNode &before = region.playlist()->get_state();
        region.set_sync_position (position);
-       session->add_redo_no_execute (region.playlist()->get_memento());
+        XMLNode &after = region.playlist()->get_state();
+       session->add_command(new MementoCommand<Playlist>(*(region.playlist()), before, after));
        commit_reversible_command ();
 }
 
@@ -2450,16 +2555,17 @@ Editor::set_region_sync_from_edit_cursor ()
                return;
        }
 
-       if (!clicked_regionview->region.covers (edit_cursor->current_frame)) {
+       if (!clicked_regionview->region().covers (edit_cursor->current_frame)) {
                error << _("Place the edit cursor at the desired sync point") << endmsg;
                return;
        }
 
-       Region& region (clicked_regionview->region);
+       Region& region (clicked_regionview->region());
        begin_reversible_command (_("set sync from edit cursor"));
-       session->add_undo (region.playlist()->get_memento());
+        XMLNode &before = region.playlist()->get_state();
        region.set_sync_position (edit_cursor->current_frame);
-       session->add_redo_no_execute (region.playlist()->get_memento());
+        XMLNode &after = region.playlist()->get_state();
+       session->add_command(new MementoCommand<Playlist>(*(region.playlist()), before, after));
        commit_reversible_command ();
 }
 
@@ -2467,11 +2573,12 @@ void
 Editor::remove_region_sync ()
 {
        if (clicked_regionview) {
-               Region& region (clicked_regionview->region);
+               Region& region (clicked_regionview->region());
                begin_reversible_command (_("remove sync"));
-               session->add_undo (region.playlist()->get_memento());
+                XMLNode &before = region.playlist()->get_state();
                region.clear_sync_position ();
-               session->add_redo_no_execute (region.playlist()->get_memento());
+                XMLNode &after = region.playlist()->get_state();
+               session->add_command(new MementoCommand<Playlist>(*(region.playlist()), before, after));
                commit_reversible_command ();
        }
 }
@@ -2479,14 +2586,15 @@ Editor::remove_region_sync ()
 void
 Editor::naturalize ()
 {
-       if (selection->audio_regions.empty()) {
+       if (selection->regions.empty()) {
                return;
        }
        begin_reversible_command (_("naturalize"));
-       for (AudioRegionSelection::iterator i = selection->audio_regions.begin(); i != selection->audio_regions.end(); ++i) {
-               session->add_undo ((*i)->region.get_memento());
-               (*i)->region.move_to_natural_position (this);
-               session->add_redo_no_execute ((*i)->region.get_memento());
+       for (RegionSelection::iterator i = selection->regions.begin(); i != selection->regions.end(); ++i) {
+                XMLNode &before = (*i)->region().get_state();
+               (*i)->region().move_to_natural_position (this);
+                XMLNode &after = (*i)->region().get_state();
+               session->add_command (new MementoCommand<Region>((*i)->region(), before, after));
        }
        commit_reversible_command ();
 }
@@ -2505,14 +2613,14 @@ Editor::align_relative (RegionPoint what)
 
 struct RegionSortByTime {
     bool operator() (const AudioRegionView* a, const AudioRegionView* b) {
-           return a->region.position() < b->region.position();
+           return a->region().position() < b->region().position();
     }
 };
 
 void
 Editor::align_selection_relative (RegionPoint point, jack_nframes_t position)
 {
-       if (selection->audio_regions.empty()) {
+       if (selection->regions.empty()) {
                return;
        }
 
@@ -2520,9 +2628,9 @@ Editor::align_selection_relative (RegionPoint point, jack_nframes_t position)
        jack_nframes_t pos = 0;
        int dir;
 
-       list<AudioRegionView*> sorted;
-       selection->audio_regions.by_position (sorted);
-       Region& r ((*sorted.begin())->region);
+       list<RegionView*> sorted;
+       selection->regions.by_position (sorted);
+       Region& r ((*sorted.begin())->region());
 
        switch (point) {
        case Start:
@@ -2548,11 +2656,11 @@ Editor::align_selection_relative (RegionPoint point, jack_nframes_t position)
 
        begin_reversible_command (_("align selection (relative)"));
 
-       for (AudioRegionSelection::iterator i = selection->audio_regions.begin(); i != selection->audio_regions.end(); ++i) {
+       for (RegionSelection::iterator i = selection->regions.begin(); i != selection->regions.end(); ++i) {
 
-               Region& region ((*i)->region);
+               Region& region ((*i)->region());
 
-               session->add_undo (region.playlist()->get_memento());
+                XMLNode &before = region.playlist()->get_state();
                
                if (dir > 0) {
                        region.set_position (region.position() + distance, this);
@@ -2560,7 +2668,8 @@ Editor::align_selection_relative (RegionPoint point, jack_nframes_t position)
                        region.set_position (region.position() - distance, this);
                }
 
-               session->add_redo_no_execute (region.playlist()->get_memento());
+                XMLNode &after = region.playlist()->get_state();
+               session->add_command(new MementoCommand<Playlist>(*(region.playlist()), before, after));
 
        }
 
@@ -2570,14 +2679,14 @@ Editor::align_selection_relative (RegionPoint point, jack_nframes_t position)
 void
 Editor::align_selection (RegionPoint point, jack_nframes_t position)
 {
-       if (selection->audio_regions.empty()) {
+       if (selection->regions.empty()) {
                return;
        }
 
        begin_reversible_command (_("align selection"));
 
-       for (AudioRegionSelection::iterator i = selection->audio_regions.begin(); i != selection->audio_regions.end(); ++i) {
-               align_region_internal ((*i)->region, point, position);
+       for (RegionSelection::iterator i = selection->regions.begin(); i != selection->regions.end(); ++i) {
+               align_region_internal ((*i)->region(), point, position);
        }
 
        commit_reversible_command ();
@@ -2594,7 +2703,7 @@ Editor::align_region (Region& region, RegionPoint point, jack_nframes_t position
 void
 Editor::align_region_internal (Region& region, RegionPoint point, jack_nframes_t position)
 {
-       session->add_undo (region.playlist()->get_memento());
+       XMLNode &before = region.playlist()->get_state();
 
        switch (point) {
        case SyncPoint:
@@ -2612,7 +2721,8 @@ Editor::align_region_internal (Region& region, RegionPoint point, jack_nframes_t
                break;
        }
 
-       session->add_redo_no_execute (region.playlist()->get_memento());
+       XMLNode &after = region.playlist()->get_state();
+       session->add_command(new MementoCommand<Playlist>(*(region.playlist()), before, after));
 }      
 
 void
@@ -2622,7 +2732,7 @@ Editor::trim_region_to_edit_cursor ()
                return;
        }
 
-       Region& region (clicked_regionview->region);
+       Region& region (clicked_regionview->region());
 
        float speed = 1.0f;
        AudioTimeAxisView *atav;
@@ -2634,9 +2744,10 @@ Editor::trim_region_to_edit_cursor ()
        }
 
        begin_reversible_command (_("trim to edit"));
-       session->add_undo (region.playlist()->get_memento());
+        XMLNode &before = region.playlist()->get_state();
        region.trim_end( session_frame_to_track_frame(edit_cursor->current_frame, speed), this);
-       session->add_redo_no_execute (region.playlist()->get_memento());
+        XMLNode &after = region.playlist()->get_state();
+       session->add_command(new MementoCommand<Playlist>(*(region.playlist()), before, after));
        commit_reversible_command ();
 }
 
@@ -2647,7 +2758,7 @@ Editor::trim_region_from_edit_cursor ()
                return;
        }
 
-       Region& region (clicked_regionview->region);
+       Region& region (clicked_regionview->region());
 
        float speed = 1.0f;
        AudioTimeAxisView *atav;
@@ -2659,9 +2770,10 @@ Editor::trim_region_from_edit_cursor ()
        }
 
        begin_reversible_command (_("trim to edit"));
-       session->add_undo (region.playlist()->get_memento());
+        XMLNode &before = region.playlist()->get_state();
        region.trim_front ( session_frame_to_track_frame(edit_cursor->current_frame, speed), this);
-       session->add_redo_no_execute (region.playlist()->get_memento());
+        XMLNode &after = region.playlist()->get_state();
+       session->add_command(new MementoCommand<Playlist>(*(region.playlist()), before, after));
        commit_reversible_command ();
 }
 
@@ -2773,9 +2885,10 @@ Editor::bounce_range_selection ()
                itt.cancel = false;
                itt.progress = false;
                
-               session->add_undo (playlist->get_memento());
+                XMLNode &before = playlist->get_state();
                atv->audio_track()->bounce_range (start, cnt, itt);
-               session->add_redo_no_execute (playlist->get_memento());
+                XMLNode &after = playlist->get_state();
+               session->add_command (new MementoCommand<Playlist> (*playlist, before, after));
        }
        
        commit_reversible_command ();
@@ -2818,16 +2931,16 @@ Editor::cut_copy (CutCopyOp op)
 
        switch (current_mouse_mode()) {
        case MouseObject: 
-               if (!selection->audio_regions.empty() || !selection->points.empty()) {
+               if (!selection->regions.empty() || !selection->points.empty()) {
 
                        begin_reversible_command (opname + _(" objects"));
 
-                       if (!selection->audio_regions.empty()) {
+                       if (!selection->regions.empty()) {
                                
                                cut_copy_regions (op);
                                
                                if (op == Cut) {
-                                       selection->clear_audio_regions ();
+                                       selection->clear_regions ();
                                }
                        }
 
@@ -2884,26 +2997,26 @@ Editor::cut_copy_regions (CutCopyOp op)
        set<Playlist*> freezelist;
        pair<set<Playlist*>::iterator,bool> insert_result;
 
-       for (AudioRegionSelection::iterator x = selection->audio_regions.begin(); x != selection->audio_regions.end(); ++x) {
-               first_position = min ((*x)->region.position(), first_position);
+       for (RegionSelection::iterator x = selection->regions.begin(); x != selection->regions.end(); ++x) {
+               first_position = min ((*x)->region().position(), first_position);
 
                if (op == Cut || op == Clear) {
-                       AudioPlaylist *pl = dynamic_cast<AudioPlaylist*>((*x)->region.playlist());
+                       AudioPlaylist *pl = dynamic_cast<AudioPlaylist*>((*x)->region().playlist());
                        if (pl) {
                                insert_result = freezelist.insert (pl);
                                if (insert_result.second) {
                                        pl->freeze ();
-                                       session->add_undo (pl->get_memento());
+                                        session->add_command (new MementoUndoCommand<Playlist>(*pl, pl->get_state()));
                                }
                        }
                }
        }
 
-       for (AudioRegionSelection::iterator x = selection->audio_regions.begin(); x != selection->audio_regions.end(); ) {
+       for (RegionSelection::iterator x = selection->regions.begin(); x != selection->regions.end(); ) {
 
-               AudioPlaylist *pl = dynamic_cast<AudioPlaylist*>((*x)->region.playlist());
+               AudioPlaylist *pl = dynamic_cast<AudioPlaylist*>((*x)->region().playlist());
                AudioPlaylist* npl;
-               AudioRegionSelection::iterator tmp;
+               RegionSelection::iterator tmp;
                
                tmp = x;
                ++tmp;
@@ -2920,18 +3033,24 @@ Editor::cut_copy_regions (CutCopyOp op)
                                npl = pi->second;
                        }
 
+                       // FIXME
+                       AudioRegion* const ar = dynamic_cast<AudioRegion*>(&(*x)->region());
                        switch (op) {
                        case Cut:
-                               npl->add_region (*(new AudioRegion ((*x)->region)), (*x)->region.position() - first_position);
-                               pl->remove_region (&((*x)->region));
+                               if (!ar) break;
+
+                               npl->add_region (*(new AudioRegion (*ar)), (*x)->region().position() - first_position);
+                               pl->remove_region (&((*x)->region()));
                                break;
 
                        case Copy:
-                               npl->add_region (*(new AudioRegion ((*x)->region)), (*x)->region.position() - first_position);
+                               if (!ar) break;
+
+                               npl->add_region (*(new AudioRegion (*ar)), (*x)->region().position() - first_position);
                                break;
 
                        case Clear:
-                               pl->remove_region (&((*x)->region));
+                               pl->remove_region (&((*x)->region()));
                                break;
                        }
                }
@@ -2951,7 +3070,7 @@ Editor::cut_copy_regions (CutCopyOp op)
        
        for (set<Playlist*>::iterator pl = freezelist.begin(); pl != freezelist.end(); ++pl) {
                (*pl)->thaw ();
-               session->add_redo_no_execute ((*pl)->get_memento());
+               session->add_command (new MementoRedoCommand<Playlist>(*(*pl), (*pl)->get_state()));
        }
 }
 
@@ -3064,9 +3183,9 @@ Editor::paste_named_selection (float times)
                tmp = chunk;
                ++tmp;
 
-               session->add_undo (apl->get_memento());
+                XMLNode &before = apl->get_state();
                apl->paste (**chunk, edit_cursor->current_frame, times);
-               session->add_redo_no_execute (apl->get_memento());
+               session->add_command(new MementoCommand<AudioPlaylist>(*apl, before, apl->get_state()));
 
                if (tmp != ns->playlists.end()) {
                        chunk = tmp;
@@ -3077,27 +3196,27 @@ Editor::paste_named_selection (float times)
 }
 
 void
-Editor::duplicate_some_regions (AudioRegionSelection& regions, float times)
+Editor::duplicate_some_regions (RegionSelection& regions, float times)
 {
        Playlist *playlist; 
-       AudioRegionSelection sel = regions; // clear (below) will clear the argument list
+       RegionSelection sel = regions; // clear (below) will clear the argument list
                
        begin_reversible_command (_("duplicate region"));
 
-       selection->clear_audio_regions ();
+       selection->clear_regions ();
 
-       for (AudioRegionSelection::iterator i = sel.begin(); i != sel.end(); ++i) {
+       for (RegionSelection::iterator i = sel.begin(); i != sel.end(); ++i) {
 
-               Region& r ((*i)->region);
+               Region& r ((*i)->region());
 
                TimeAxisView& tv = (*i)->get_time_axis_view();
                AudioTimeAxisView* atv = dynamic_cast<AudioTimeAxisView*> (&tv);
-               sigc::connection c = atv->view->AudioRegionViewAdded.connect (mem_fun(*this, &Editor::collect_new_region_view));
+               sigc::connection c = atv->view()->RegionViewAdded.connect (mem_fun(*this, &Editor::collect_new_region_view));
                
-               playlist = (*i)->region.playlist();
-               session->add_undo (playlist->get_memento());
+               playlist = (*i)->region().playlist();
+                XMLNode &before = playlist->get_state();
                playlist->duplicate (r, r.last_frame(), times);
-               session->add_redo_no_execute (playlist->get_memento());
+               session->add_command(new MementoCommand<Playlist>(*playlist, before, playlist->get_state()));
 
                c.disconnect ();
 
@@ -3135,9 +3254,10 @@ Editor::duplicate_selection (float times)
                if ((playlist = (*i)->playlist()) == 0) {
                        continue;
                }
-               session->add_undo (playlist->get_memento());
+                XMLNode &before = playlist->get_state();
                playlist->duplicate (**ri, selection->time[clicked_selection].end, times);
-               session->add_redo_no_execute (playlist->get_memento());
+                XMLNode &after = playlist->get_state();
+               session->add_command (new MementoCommand<Playlist>(*playlist, before, after));
 
                ++ri;
                if (ri == new_regions.end()) {
@@ -3148,6 +3268,23 @@ Editor::duplicate_selection (float times)
        commit_reversible_command ();
 }
 
+void
+Editor::reset_point_selection ()
+{
+       /* reset all selected points to the relevant default value */
+
+       cerr << "point selection has " << selection->points.size() << " entries\n";
+       
+       for (PointSelection::iterator i = selection->points.begin(); i != selection->points.end(); ++i) {
+               
+               AutomationTimeAxisView* atv = dynamic_cast<AutomationTimeAxisView*>(&(*i).track);
+               
+               if (atv) {
+                       atv->reset_objects (selection->points);
+               } 
+       }
+}
+
 void
 Editor::center_playhead ()
 {
@@ -3168,9 +3305,10 @@ void
 Editor::clear_playlist (Playlist& playlist)
 {
        begin_reversible_command (_("clear playlist"));
-       session->add_undo (playlist.get_memento());
+        XMLNode &before = playlist.get_state();
        playlist.clear ();
-       session->add_redo_no_execute (playlist.get_memento());
+        XMLNode &after = playlist.get_state();
+       session->add_command (new MementoCommand<Playlist>(playlist, before, after));
        commit_reversible_command ();
 }
 
@@ -3204,9 +3342,10 @@ Editor::nudge_track (bool use_edit_cursor, bool forwards)
                        continue;
                }               
                
-               session->add_undo (playlist->get_memento());
+                XMLNode &before = playlist->get_state();
                playlist->nudge_after (start, distance, forwards);
-               session->add_redo_no_execute (playlist->get_memento());
+                XMLNode &after = playlist->get_state();
+               session->add_command (new MementoCommand<Playlist>(*playlist, before, after));
        }
        
        commit_reversible_command ();                   
@@ -3226,12 +3365,12 @@ Editor::remove_last_capture ()
                prompt  = _("Do you really want to destroy the last capture?"
                            "\n(This is destructive and cannot be undone)");
 
-               choices.push_back (_("Yes, destroy it."));
                choices.push_back (_("No, do nothing."));
+               choices.push_back (_("Yes, destroy it."));
                
                Gtkmm2ext::Choice prompter (prompt, choices);
                
-               if (prompter.run () == 0) {
+               if (prompter.run () == 1) {
                        session->remove_last_capture ();
                }
 
@@ -3247,7 +3386,7 @@ Editor::normalize_region ()
                return;
        }
 
-       if (selection->audio_regions.empty()) {
+       if (selection->regions.empty()) {
                return;
        }
 
@@ -3256,10 +3395,13 @@ Editor::normalize_region ()
        track_canvas.get_window()->set_cursor (*wait_cursor);
        gdk_flush ();
 
-       for (AudioRegionSelection::iterator r = selection->audio_regions.begin(); r != selection->audio_regions.end(); ++r) {
-               session->add_undo ((*r)->region.get_memento());
-               (*r)->region.normalize_to (0.0f);
-               session->add_redo_no_execute ((*r)->region.get_memento());
+       for (RegionSelection::iterator r = selection->regions.begin(); r != selection->regions.end(); ++r) {
+               AudioRegionView* const arv = dynamic_cast<AudioRegionView*>(*r);
+               if (!arv)
+                       continue;
+               XMLNode &before = arv->region().get_state();
+               arv->audio_region().normalize_to (0.0f);
+               session->add_command (new MementoCommand<Region>(arv->region(), before, arv->region().get_state()));
        }
 
        commit_reversible_command ();
@@ -3274,16 +3416,19 @@ Editor::denormalize_region ()
                return;
        }
 
-       if (selection->audio_regions.empty()) {
+       if (selection->regions.empty()) {
                return;
        }
 
        begin_reversible_command ("denormalize");
 
-       for (AudioRegionSelection::iterator r = selection->audio_regions.begin(); r != selection->audio_regions.end(); ++r) {
-               session->add_undo ((*r)->region.get_memento());
-               (*r)->region.set_scale_amplitude (1.0f);
-               session->add_redo_no_execute ((*r)->region.get_memento());
+       for (RegionSelection::iterator r = selection->regions.begin(); r != selection->regions.end(); ++r) {
+               AudioRegionView* const arv = dynamic_cast<AudioRegionView*>(*r);
+               if (!arv)
+                       continue;
+               XMLNode &before = arv->region().get_state();
+               arv->audio_region().set_scale_amplitude (1.0f);
+               session->add_command (new MementoCommand<Region>(arv->region(), before, arv->region().get_state()));
        }
 
        commit_reversible_command ();
@@ -3304,7 +3449,7 @@ Editor::reverse_region ()
 void
 Editor::apply_filter (AudioFilter& filter, string command)
 {
-       if (selection->audio_regions.empty()) {
+       if (selection->regions.empty()) {
                return;
        }
 
@@ -3313,21 +3458,24 @@ Editor::apply_filter (AudioFilter& filter, string command)
        track_canvas.get_window()->set_cursor (*wait_cursor);
        gdk_flush ();
 
-       for (AudioRegionSelection::iterator r = selection->audio_regions.begin(); r != selection->audio_regions.end(); ) {
+       for (RegionSelection::iterator r = selection->regions.begin(); r != selection->regions.end(); ) {
+               AudioRegionView* const arv = dynamic_cast<AudioRegionView*>(*r);
+               if (!arv)
+                       continue;
 
-               AudioRegion& region ((*r)->region);
-               Playlist* playlist = region.playlist();
+               Playlist* playlist = arv->region().playlist();
 
-               AudioRegionSelection::iterator tmp;
+               RegionSelection::iterator tmp;
                
                tmp = r;
                ++tmp;
 
-               if (region.apply (filter) == 0) {
+               if (arv->audio_region().apply (filter) == 0) {
 
-                       session->add_undo (playlist->get_memento());
-                       playlist->replace_region (region, *(filter.results.front()), region.position());
-                       session->add_redo_no_execute (playlist->get_memento());
+                        XMLNode &before = playlist->get_state();
+                       playlist->replace_region (arv->region(), *(filter.results.front()), arv->region().position());
+                        XMLNode &after = playlist->get_state();
+                       session->add_command(new MementoCommand<Playlist>(*playlist, before, after));
                } else {
                        goto out;
                }
@@ -3336,7 +3484,7 @@ Editor::apply_filter (AudioFilter& filter, string command)
        }
 
        commit_reversible_command ();
-       selection->audio_regions.clear ();
+       selection->regions.clear ();
 
   out:
        track_canvas.get_window()->set_cursor (*current_canvas_cursor);
@@ -3345,8 +3493,8 @@ Editor::apply_filter (AudioFilter& filter, string command)
 void
 Editor::region_selection_op (void (Region::*pmf)(void))
 {
-       for (AudioRegionSelection::iterator i = selection->audio_regions.begin(); i != selection->audio_regions.end(); ++i) {
-               ((*i)->region.*pmf)();
+       for (RegionSelection::iterator i = selection->regions.begin(); i != selection->regions.end(); ++i) {
+               ((*i)->region().*pmf)();
        }
 }
 
@@ -3354,16 +3502,16 @@ Editor::region_selection_op (void (Region::*pmf)(void))
 void
 Editor::region_selection_op (void (Region::*pmf)(void*), void *arg)
 {
-       for (AudioRegionSelection::iterator i = selection->audio_regions.begin(); i != selection->audio_regions.end(); ++i) {
-               ((*i)->region.*pmf)(arg);
+       for (RegionSelection::iterator i = selection->regions.begin(); i != selection->regions.end(); ++i) {
+               ((*i)->region().*pmf)(arg);
        }
 }
 
 void
 Editor::region_selection_op (void (Region::*pmf)(bool), bool yn)
 {
-       for (AudioRegionSelection::iterator i = selection->audio_regions.begin(); i != selection->audio_regions.end(); ++i) {
-               ((*i)->region.*pmf)(yn);
+       for (RegionSelection::iterator i = selection->regions.begin(); i != selection->regions.end(); ++i) {
+               ((*i)->region().*pmf)(yn);
        }
 }
 
@@ -3380,20 +3528,20 @@ Editor::external_edit_region ()
 void
 Editor::brush (jack_nframes_t pos)
 {
-       AudioRegionSelection sel;
+       RegionSelection sel;
        snap_to (pos);
 
-       if (selection->audio_regions.empty()) {
+       if (selection->regions.empty()) {
                /* XXX get selection from region list */
        } else { 
-               sel = selection->audio_regions;
+               sel = selection->regions;
        }
 
        if (sel.empty()) {
                return;
        }
 
-       for (AudioRegionSelection::iterator i = selection->audio_regions.begin(); i != selection->audio_regions.end(); ++i) {
+       for (RegionSelection::iterator i = selection->regions.begin(); i != selection->regions.end(); ++i) {
                mouse_brush_insert_region ((*i), pos);
        }
 }
@@ -3401,18 +3549,19 @@ Editor::brush (jack_nframes_t pos)
 void
 Editor::toggle_gain_envelope_visibility ()
 {
-       for (AudioRegionSelection::iterator i = selection->audio_regions.begin(); i != selection->audio_regions.end(); ++i) {
-               (*i)->set_envelope_visible (!(*i)->envelope_visible());
+       for (RegionSelection::iterator i = selection->regions.begin(); i != selection->regions.end(); ++i) {
+               AudioRegionView* const arv = dynamic_cast<AudioRegionView*>(*i);
+               if (arv)
+                       arv->set_envelope_visible (!arv->envelope_visible());
        }
 }
 
 void
 Editor::toggle_gain_envelope_active ()
 {
-       for (AudioRegionSelection::iterator i = selection->audio_regions.begin(); i != selection->audio_regions.end(); ++i) {
-               AudioRegion* ar = dynamic_cast<AudioRegion*>(&(*i)->region);
-               if (ar) {
-                       ar->set_envelope_active (true);
-               }
+       for (RegionSelection::iterator i = selection->regions.begin(); i != selection->regions.end(); ++i) {
+               AudioRegionView* const arv = dynamic_cast<AudioRegionView*>(*i);
+               if (arv)
+                       arv->audio_region().set_envelope_active (true);
        }
 }