fixes for destructive track offsets of various kinds; move from jack_nframes_t -...
[ardour.git] / gtk2_ardour / editor_mouse.cc
index bf8b16c0d74517f10ac556ef40b565bf5e459f32..0ee110bf30bd71ba0342048331edbc72180fea35 100644 (file)
@@ -18,6 +18,7 @@
     $Id$
 */
 
+#include <cassert>
 #include <cstdlib>
 #include <stdint.h>
 #include <cmath>
 
 #include <pbd/error.h>
 #include <gtkmm2ext/utils.h>
+#include <pbd/memento_command.h>
 
 #include "ardour_ui.h"
 #include "editor.h"
 #include "time_axis_view.h"
 #include "audio_time_axis.h"
-#include "regionview.h"
+#include "audio_region_view.h"
 #include "marker.h"
 #include "streamview.h"
 #include "region_gain_line.h"
@@ -47,7 +49,7 @@
 #include <ardour/types.h>
 #include <ardour/route.h>
 #include <ardour/audio_track.h>
-#include <ardour/diskstream.h>
+#include <ardour/audio_diskstream.h>
 #include <ardour/playlist.h>
 #include <ardour/audioplaylist.h>
 #include <ardour/audioregion.h>
 
 using namespace std;
 using namespace ARDOUR;
+using namespace PBD;
 using namespace sigc;
 using namespace Gtk;
 using namespace Editing;
 
-jack_nframes_t
+nframes_t
 Editor::event_frame (GdkEvent* event, double* pcx, double* pcy)
 {
        double cx, cy;
@@ -96,7 +99,6 @@ Editor::event_frame (GdkEvent* event, double* pcx, double* pcy)
                break;
        case GDK_KEY_PRESS:
        case GDK_KEY_RELEASE:
-               cerr << "here\n";
                // track_canvas.w2c(event->key.x, event->key.y, *pcx, *pcy);
                break;
        default:
@@ -168,7 +170,7 @@ Editor::set_mouse_mode (MouseMode m, bool force)
                return;
        }
 
-       if (m == mouse_mode && !force) {
+       if (!force && m == mouse_mode) {
                return;
        }
        
@@ -182,7 +184,7 @@ Editor::set_mouse_mode (MouseMode m, bool force)
                   show the object (region) selection.
                */
 
-               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) {
                        (*i)->set_should_show_selection (true);
                }
                for (TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) {
@@ -196,7 +198,7 @@ Editor::set_mouse_mode (MouseMode m, bool force)
                */
 
                for (TrackSelection::iterator i = selection->tracks.begin(); i != selection->tracks.end(); ++i) {
-                       if ((*i)->selected()) {
+                       if ((*i)->get_selected()) {
                                (*i)->show_selection (selection->time);
                        }
                }
@@ -284,66 +286,68 @@ Editor::step_mouse_mode (bool next)
        }
 }
 
-bool
-Editor::button_press_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item_type)
+void
+Editor::button_selection (ArdourCanvas::Item* item, GdkEvent* event, ItemType item_type)
 {
-       jack_nframes_t where = event_frame (event, 0, 0);
-
-       track_canvas.grab_focus();
-
-       if (session && session->actively_recording()) {
-               return true;
-       }
+       bool commit = false;
+       bool c1; 
+       bool c2;
 
        /* in object/audition/timefx mode, any button press sets
           the selection if the object can be selected. this is a
           bit of hack, because we want to avoid this if the
           mouse operation is a region alignment.
-       */
-
-       if (((mouse_mode == MouseObject) ||
-            (mouse_mode == MouseAudition && item_type == RegionItem) ||
-            (mouse_mode == MouseTimeFX && item_type == RegionItem)) &&
-           event->type == GDK_BUTTON_PRESS && 
-           event->button.button <= 3) {
-
-               AudioRegionView* rv;
-               ControlPoint* cp;
-
-               /* not dbl-click or triple-click */
 
-               switch (item_type) {
-               case RegionItem:
-                       set_selected_regionview_from_click (Keyboard::selection_type (event->button.state), true);
-                       break;
-                       
-               case AudioRegionViewNameHighlight:
-               case AudioRegionViewName:
-                       if ((rv = static_cast<AudioRegionView *> (item->get_data ("regionview"))) != 0) {
-                               set_selected_regionview_from_click (Keyboard::selection_type (event->button.state), true);
-                       }
-                       break;
-                       
-               case GainAutomationControlPointItem:
-               case PanAutomationControlPointItem:
-               case RedirectAutomationControlPointItem:
-                       if ((cp = static_cast<ControlPoint *> (item->get_data ("control_point"))) != 0) {
-                               set_selected_control_point_from_click (Keyboard::selection_type (event->button.state), false);
-                       }
-                       break;
+          note: not dbl-click or triple-click
+       */
 
-               case StreamItem:
-                       set_selected_track_from_click (Keyboard::selection_type (event->button.state), true, true);
-                       break;
+       if (((mouse_mode != MouseObject) &&
+            (mouse_mode != MouseAudition || item_type != RegionItem) &&
+            (mouse_mode != MouseTimeFX || item_type != RegionItem)) ||
+           (event->type != GDK_BUTTON_PRESS && event->type != GDK_BUTTON_RELEASE || event->button.button > 3)) {
+               
+               return;
+       }
+           
+       Selection::Operation op = Keyboard::selection_type (event->button.state);
+       bool press = (event->type == GDK_BUTTON_PRESS);
 
-               case AutomationTrackItem:
-                       break;
+       begin_reversible_command (_("select on click"));
 
-               default:
-                       break;
-               }
+       switch (item_type) {
+       case RegionItem:
+               c1 = set_selected_track_from_click (press, op, true, true);
+               c2 = set_selected_regionview_from_click (press, op, true);
+               commit = (c1 || c2);
+               break;
+               
+       case RegionViewNameHighlight:
+       case RegionViewName:
+               c1 = set_selected_track_from_click (press, op, true, true);
+               c2 = set_selected_regionview_from_click (press, op, true);
+               commit = (c1 || c2);
+               break;
+               
+       case GainAutomationControlPointItem:
+       case PanAutomationControlPointItem:
+       case RedirectAutomationControlPointItem:
+               c1 = set_selected_track_from_click (press, op, true, true);
+               c2 = set_selected_control_point_from_click (press, op, false);
+               commit = (c1 || c2);
+               break;
+               
+       case StreamItem:
+               commit = set_selected_track_from_click (press, op, true, true);
+               break;
+                   
+       case AutomationTrackItem:
+               commit = set_selected_track_from_click (press, op, true, true);
+               break;
+               
+       default:
+               break;
        }
-
+       
 #define SELECT_TRACK_FROM_CANVAS_IN_RANGE_MODE
 #ifdef  SELECT_TRACK_FROM_CANVAS_IN_RANGE_MODE
        /* in range mode, button 1/2/3 press potentially selects a track */
@@ -352,24 +356,36 @@ Editor::button_press_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemTyp
            event->type == GDK_BUTTON_PRESS && 
            event->button.button <= 3) {
                
-               AudioRegionView* rv;
-
                switch (item_type) {
                case StreamItem:
                case RegionItem:
                case AutomationTrackItem:
-                       set_selected_track_from_click (Keyboard::selection_type (event->button.state), true, true);
+                       commit = set_selected_track_from_click (press, op, true, true);
                        break;
 
-               case AudioRegionViewNameHighlight:
-               case AudioRegionViewName:
-                       rv = reinterpret_cast<AudioRegionView *> (item->get_data ("regionview"));
                default:
                        break;
                }
        }
 #endif
+       if (commit) {
+               commit_reversible_command ();
+       }
+}
+
+bool
+Editor::button_press_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item_type)
+{
+       nframes_t where = event_frame (event, 0, 0);
 
+       track_canvas.grab_focus();
+
+       if (session && session->actively_recording()) {
+               return true;
+       }
+
+       button_selection (item, event, item_type);
+       
        if (drag_info.item == 0 &&
            (Keyboard::is_delete_event (&event->button) ||
             Keyboard::is_context_menu_event (&event->button) ||
@@ -435,6 +451,7 @@ Editor::button_press_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemTyp
                                start_range_markerbar_op (item, event, CreateRangeMarker); 
                                return true;
                                break;
+
                        case TransportMarkerBarItem:
                                start_range_markerbar_op (item, event, CreateTransportMarker); 
                                return true;
@@ -506,12 +523,12 @@ Editor::button_press_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemTyp
                                        }
                                        break;
                                        
-                               case AudioRegionViewNameHighlight:
+                               case RegionViewNameHighlight:
                                        start_trim (item, event);
                                        return true;
                                        break;
                                        
-                               case AudioRegionViewName:
+                               case RegionViewName:
                                        /* rename happens on edit clicks */
                                                start_trim (clicked_regionview->get_name_highlight(), event);
                                                return true;
@@ -564,6 +581,10 @@ Editor::button_press_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemTyp
                                        break ;
                                /* </CMT Additions> */
 
+                               case MarkerBarItem:
+                                       
+                                       break;
+
                                default:
                                        break;
                                }
@@ -673,12 +694,12 @@ Editor::button_press_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemTyp
                        
                        
                        switch (item_type) {
-                       case AudioRegionViewNameHighlight:
+                       case RegionViewNameHighlight:
                                start_trim (item, event);
                                return true;
                                break;
                                
-                       case AudioRegionViewName:
+                       case RegionViewName:
                                start_trim (clicked_regionview->get_name_highlight(), event);
                                return true;
                                break;
@@ -799,7 +820,7 @@ Editor::button_press_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemTyp
 bool
 Editor::button_release_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item_type)
 {
-       jack_nframes_t where = event_frame (event, 0, 0);
+       nframes_t where = event_frame (event, 0, 0);
 
        /* no action if we're recording */
                                                
@@ -816,6 +837,8 @@ Editor::button_release_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemT
                }
        }
        
+       button_selection (item, event, item_type);
+
        /* edit events get handled here */
        
        if (drag_info.item == 0 && Keyboard::is_edit_event (&event->button)) {
@@ -832,7 +855,7 @@ Editor::button_release_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemT
                        edit_meter_marker (item);
                        break;
                        
-               case AudioRegionViewName:
+               case RegionViewName:
                        if (clicked_regionview->name_active()) {
                                return mouse_rename_region (item, event);
                        }
@@ -867,8 +890,8 @@ Editor::button_release_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemT
                                break;
                                
                        case RegionItem:
-                       case AudioRegionViewNameHighlight:
-                       case AudioRegionViewName:
+                       case RegionViewNameHighlight:
+                       case RegionViewName:
                                popup_track_context_menu (1, event->button.time, item_type, false, where);
                                break;
                                
@@ -1026,9 +1049,13 @@ Editor::button_release_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemT
                        break;
 
                case MouseGain:
+                       // Gain only makes sense for audio regions
+                       if ( ! dynamic_cast<AudioRegionView*>(clicked_regionview))
+                               break;
+
                        switch (item_type) {
                        case RegionItem:
-                               clicked_regionview->add_gain_point_event (item, event);
+                               dynamic_cast<AudioRegionView*>(clicked_regionview)->add_gain_point_event (item, event);
                                return true;
                                break;
                                
@@ -1104,28 +1131,6 @@ Editor::button_release_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemT
        return false;
 }
 
-void
-Editor::maybe_autoscroll (GdkEvent* event)
-{
-       jack_nframes_t rightmost_frame = leftmost_frame + current_page_frames();
-
-       jack_nframes_t frame = drag_info.current_pointer_frame;
-
-       cerr << "maybe autoscroll @ " << frame << " left = " << leftmost_frame << " right = " << rightmost_frame << endl;
-
-       if (frame > rightmost_frame) {
-               if (rightmost_frame < max_frames) {
-                       autoscroll_direction = 1;
-                       autoscroll_canvas ();
-               }
-       } else if (frame < leftmost_frame) {
-               if (leftmost_frame > 0) {
-                       autoscroll_direction = -1;
-                       autoscroll_canvas ();
-               }
-       } 
-}
-
 bool
 Editor::enter_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item_type)
 {
@@ -1204,7 +1209,7 @@ Editor::enter_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item_
                }
                break;
                
-       case AudioRegionViewNameHighlight:
+       case RegionViewNameHighlight:
                if (is_drawable() && mouse_mode == MouseObject) {
                        track_canvas.get_window()->set_cursor (*trimmer_cursor);
                }
@@ -1231,11 +1236,11 @@ Editor::enter_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item_
                }
                break;
 
-       case AudioRegionViewName:
+       case RegionViewName:
                
                /* when the name is not an active item, the entire name highlight is for trimming */
 
-               if (!reinterpret_cast<AudioRegionView *> (item->get_data ("regionview"))->name_active()) {
+               if (!reinterpret_cast<RegionView *> (item->get_data ("regionview"))->name_active()) {
                        if (mouse_mode == MouseObject && is_drawable()) {
                                track_canvas.get_window()->set_cursor (*trimmer_cursor);
                        }
@@ -1340,7 +1345,7 @@ Editor::leave_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item_
        ControlPoint* cp;
        Marker *marker;
        Location *loc;
-       AudioRegionView* rv;
+       RegionView* rv;
        bool is_start;
 
        switch (item_type) {
@@ -1362,7 +1367,7 @@ Editor::leave_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item_
                hide_verbose_canvas_cursor ();
                break;
                
-       case AudioRegionViewNameHighlight:
+       case RegionViewNameHighlight:
        case StartSelectionTrimItem:
        case EndSelectionTrimItem:
        case EditCursorItem:
@@ -1393,9 +1398,9 @@ Editor::leave_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item_
                }
                break;
 
-       case AudioRegionViewName:
+       case RegionViewName:
                /* see enter_handler() for notes */
-               if (!reinterpret_cast<AudioRegionView *> (item->get_data ("regionview"))->name_active()) {
+               if (!reinterpret_cast<RegionView *> (item->get_data ("regionview"))->name_active()) {
                        if (is_drawable() && mouse_mode == MouseObject) {
                                track_canvas.get_window()->set_cursor (*current_canvas_cursor);
                        }
@@ -1430,7 +1435,7 @@ Editor::leave_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item_
 
        case FadeInHandleItem:
        case FadeOutHandleItem:
-               rv = static_cast<AudioRegionView*>(item->get_data ("regionview"));
+               rv = static_cast<RegionView*>(item->get_data ("regionview"));
                {
                        ArdourCanvas::SimpleRect *rect = dynamic_cast<ArdourCanvas::SimpleRect *> (item);
                        if (rect) {
@@ -1497,14 +1502,15 @@ Editor::motion_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item
                                                       &drag_info.current_pointer_y);
 
        if (!from_autoscroll && drag_info.item) {
-               /* item != 0 is the best test i can think of for 
-                  dragging.
+               /* item != 0 is the best test i can think of for dragging.
                */
-               if (!drag_info.move_threshold_passsed) {
-                       drag_info.move_threshold_passsed = (abs ((int) (drag_info.current_pointer_x - drag_info.grab_x)) > 4);
+               if (!drag_info.move_threshold_passed) {
+
+                       drag_info.move_threshold_passed = (abs ((int) (drag_info.current_pointer_x - drag_info.grab_x)) > 4);
                        
                        // and change the initial grab loc/frame if this drag info wants us to
-                       if (drag_info.want_move_threshold && drag_info.move_threshold_passsed) {
+
+                       if (drag_info.want_move_threshold && drag_info.move_threshold_passed) {
                                drag_info.grab_frame = drag_info.current_pointer_frame;
                                drag_info.grab_x = drag_info.current_pointer_x;
                                drag_info.grab_y = drag_info.current_pointer_y;
@@ -1524,7 +1530,7 @@ Editor::motion_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item
        case PanAutomationControlPointItem:
        case TempoMarkerItem:
        case MeterMarkerItem:
-       case AudioRegionViewNameHighlight:
+       case RegionViewNameHighlight:
        case StartSelectionTrimItem:
        case EndSelectionTrimItem:
        case SelectionItem:
@@ -1576,6 +1582,7 @@ Editor::motion_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemType item
 
   handled:
        track_canvas_motion (event);
+       // drag_info.last_pointer_frame = drag_info.current_pointer_frame;
        return true;
        
   not_handled:
@@ -1618,7 +1625,7 @@ Editor::start_grab (GdkEvent* event, Gdk::Cursor *cursor)
        drag_info.cumulative_x_drag = 0;
        drag_info.cumulative_y_drag = 0;
        drag_info.first_move = true;
-       drag_info.move_threshold_passsed = false;
+       drag_info.move_threshold_passed = false;
        drag_info.want_move_threshold = false;
        drag_info.pointer_frame_offset = 0;
        drag_info.brushing = false;
@@ -1646,6 +1653,19 @@ Editor::start_grab (GdkEvent* event, Gdk::Cursor *cursor)
        }
 }
 
+void
+Editor::swap_grab (ArdourCanvas::Item* new_item, Gdk::Cursor* cursor, uint32_t time)
+{
+       drag_info.item->ungrab (0);
+       drag_info.item = new_item;
+
+       if (cursor == 0) {
+               cursor = grabber_cursor;
+       }
+
+       drag_info.item->grab (Gdk::POINTER_MOTION_MASK|Gdk::BUTTON_PRESS_MASK|Gdk::BUTTON_RELEASE_MASK, *cursor, time);
+}
+
 bool
 Editor::end_grab (ArdourCanvas::Item* item, GdkEvent* event)
 {
@@ -1689,7 +1709,7 @@ Editor::end_grab (ArdourCanvas::Item* item, GdkEvent* event)
 void
 Editor::set_edit_cursor (GdkEvent* event)
 {
-       jack_nframes_t pointer_frame = event_frame (event);
+       nframes_t pointer_frame = event_frame (event);
 
        if (!Keyboard::modifier_state_contains (event->button.state, Keyboard::snap_modifier())) {
                if (snap_type != SnapToEditCursor) {
@@ -1704,7 +1724,7 @@ Editor::set_edit_cursor (GdkEvent* event)
 void
 Editor::set_playhead_cursor (GdkEvent* event)
 {
-       jack_nframes_t pointer_frame = event_frame (event);
+       nframes_t pointer_frame = event_frame (event);
 
        if (!Keyboard::modifier_state_contains (event->button.state, Keyboard::snap_modifier())) {
                snap_to (pointer_frame);
@@ -1731,15 +1751,15 @@ Editor::start_fade_in_grab (ArdourCanvas::Item* item, GdkEvent* event)
 
        AudioRegionView* arv = static_cast<AudioRegionView*>(drag_info.data);
 
-       drag_info.pointer_frame_offset = drag_info.grab_frame - ((jack_nframes_t) arv->region.fade_in().back()->when + arv->region.position()); 
+       drag_info.pointer_frame_offset = drag_info.grab_frame - ((nframes_t) arv->audio_region()->fade_in().back()->when + arv->region()->position());  
 }
 
 void
 Editor::fade_in_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
        AudioRegionView* arv = static_cast<AudioRegionView*>(drag_info.data);
-       jack_nframes_t pos;
-       jack_nframes_t fade_length;
+       nframes_t pos;
+       nframes_t fade_length;
 
        if ((long)drag_info.current_pointer_frame > drag_info.pointer_frame_offset) {
                pos = drag_info.current_pointer_frame - drag_info.pointer_frame_offset;
@@ -1752,17 +1772,17 @@ Editor::fade_in_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
                snap_to (pos);
        }
        
-       if (pos < (arv->region.position() + 64)) {
+       if (pos < (arv->region()->position() + 64)) {
                fade_length = 64; // this should be a minimum defined somewhere
-       } else if (pos > arv->region.last_frame()) {
-               fade_length = arv->region.length();
+       } else if (pos > arv->region()->last_frame()) {
+               fade_length = arv->region()->length();
        } else {
-               fade_length = pos - arv->region.position();
+               fade_length = pos - arv->region()->position();
        }
        
        arv->reset_fade_in_shape_width (fade_length);
 
-       show_verbose_duration_cursor (arv->region.position(),  arv->region.position() + fade_length, 10);
+       show_verbose_duration_cursor (arv->region()->position(),  arv->region()->position() + fade_length, 10);
 
        drag_info.first_move = false;
 }
@@ -1773,8 +1793,8 @@ Editor::fade_in_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent* even
        if (drag_info.first_move) return;
 
        AudioRegionView* arv = static_cast<AudioRegionView*>(drag_info.data);
-       jack_nframes_t pos;
-       jack_nframes_t fade_length;
+       nframes_t pos;
+       nframes_t fade_length;
 
        if ((long)drag_info.current_pointer_frame > drag_info.pointer_frame_offset) {
                pos = drag_info.current_pointer_frame - drag_info.pointer_frame_offset;
@@ -1787,20 +1807,23 @@ Editor::fade_in_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent* even
                snap_to (pos);
        }
 
-       if (pos < (arv->region.position() + 64)) {
+       if (pos < (arv->region()->position() + 64)) {
                fade_length = 64; // this should be a minimum defined somewhere
        }
-       else if (pos > arv->region.last_frame()) {
-               fade_length = arv->region.length();
+       else if (pos > arv->region()->last_frame()) {
+               fade_length = arv->region()->length();
        }
        else {
-               fade_length = pos - arv->region.position();
+               fade_length = pos - arv->region()->position();
        }
 
        begin_reversible_command (_("change fade in length"));
-       session->add_undo (arv->region.get_memento());
-       arv->region.set_fade_in_length (fade_length);
-       session->add_redo_no_execute (arv->region.get_memento());
+        XMLNode &before = arv->audio_region()->get_state();
+
+       arv->audio_region()->set_fade_in_length (fade_length);
+
+        XMLNode &after = arv->audio_region()->get_state();
+        session->add_command(new MementoCommand<ARDOUR::AudioRegion>(*arv->audio_region().get(), &before, &after));
        commit_reversible_command ();
        fade_in_drag_motion_callback (item, event);
 }
@@ -1821,15 +1844,15 @@ Editor::start_fade_out_grab (ArdourCanvas::Item* item, GdkEvent* event)
 
        AudioRegionView* arv = static_cast<AudioRegionView*>(drag_info.data);
 
-       drag_info.pointer_frame_offset = drag_info.grab_frame - (arv->region.length() - (jack_nframes_t) arv->region.fade_out().back()->when + arv->region.position()); 
+       drag_info.pointer_frame_offset = drag_info.grab_frame - (arv->region()->length() - (nframes_t) arv->audio_region()->fade_out().back()->when + arv->region()->position());       
 }
 
 void
 Editor::fade_out_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
        AudioRegionView* arv = static_cast<AudioRegionView*>(drag_info.data);
-       jack_nframes_t pos;
-       jack_nframes_t fade_length;
+       nframes_t pos;
+       nframes_t fade_length;
 
        if ((long)drag_info.current_pointer_frame > drag_info.pointer_frame_offset) {
                pos = drag_info.current_pointer_frame - drag_info.pointer_frame_offset;
@@ -1842,19 +1865,19 @@ Editor::fade_out_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event
                snap_to (pos);
        }
 
-       if (pos > (arv->region.last_frame() - 64)) {
+       if (pos > (arv->region()->last_frame() - 64)) {
                fade_length = 64; // this should really be a minimum fade defined somewhere
        }
-       else if (pos < arv->region.position()) {
-               fade_length = arv->region.length();
+       else if (pos < arv->region()->position()) {
+               fade_length = arv->region()->length();
        }
        else {
-               fade_length = arv->region.last_frame() - pos;
+               fade_length = arv->region()->last_frame() - pos;
        }
        
        arv->reset_fade_out_shape_width (fade_length);
 
-       show_verbose_duration_cursor (arv->region.last_frame() - fade_length, arv->region.last_frame(), 10);
+       show_verbose_duration_cursor (arv->region()->last_frame() - fade_length, arv->region()->last_frame(), 10);
 
        drag_info.first_move = false;
 }
@@ -1865,8 +1888,8 @@ Editor::fade_out_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent* eve
        if (drag_info.first_move) return;
 
        AudioRegionView* arv = static_cast<AudioRegionView*>(drag_info.data);
-       jack_nframes_t pos;
-       jack_nframes_t fade_length;
+       nframes_t pos;
+       nframes_t fade_length;
 
        if ((long)drag_info.current_pointer_frame > drag_info.pointer_frame_offset) {
                pos = drag_info.current_pointer_frame - drag_info.pointer_frame_offset;
@@ -1879,20 +1902,23 @@ Editor::fade_out_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent* eve
                snap_to (pos);
        }
 
-       if (pos > (arv->region.last_frame() - 64)) {
+       if (pos > (arv->region()->last_frame() - 64)) {
                fade_length = 64; // this should really be a minimum fade defined somewhere
        }
-       else if (pos < arv->region.position()) {
-               fade_length = arv->region.length();
+       else if (pos < arv->region()->position()) {
+               fade_length = arv->region()->length();
        }
        else {
-               fade_length = arv->region.last_frame() - pos;
+               fade_length = arv->region()->last_frame() - pos;
        }
 
        begin_reversible_command (_("change fade out length"));
-       session->add_undo (arv->region.get_memento());
-       arv->region.set_fade_out_length (fade_length);
-       session->add_redo_no_execute (arv->region.get_memento());
+        XMLNode &before = arv->region()->get_state();
+
+       arv->audio_region()->set_fade_out_length (fade_length);
+
+        XMLNode &after = arv->region()->get_state();
+        session->add_command(new MementoCommand<ARDOUR::Region>(*arv->region().get(), &before, &after));
        commit_reversible_command ();
 
        fade_out_drag_motion_callback (item, event);
@@ -1929,7 +1955,7 @@ void
 Editor::cursor_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
        Cursor* cursor = (Cursor *) drag_info.data;
-       jack_nframes_t adjusted_frame;
+       nframes_t adjusted_frame;
        
        if ((long)drag_info.current_pointer_frame > drag_info.pointer_frame_offset) {
                adjusted_frame = drag_info.current_pointer_frame - drag_info.pointer_frame_offset;
@@ -2015,7 +2041,7 @@ Editor::start_marker_grab (ArdourCanvas::Item* item, GdkEvent* event)
 
        drag_info.copied_location = new Location (*location);
        drag_info.pointer_frame_offset = drag_info.grab_frame - (is_start ? location->start() : location->end());       
-       
+
        update_marker_drag_item (location);
 
        if (location->is_mark()) {
@@ -2034,28 +2060,31 @@ Editor::start_marker_grab (ArdourCanvas::Item* item, GdkEvent* event)
 void
 Editor::marker_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
-       jack_nframes_t f_delta; 
+       nframes_t f_delta;      
        Marker* marker = (Marker *) drag_info.data;
        Location  *real_location;
        Location  *copy_location;
        bool is_start;
        bool move_both = false;
 
-       jack_nframes_t newframe;
+
+       nframes_t newframe;
        if (drag_info.pointer_frame_offset <= (long) drag_info.current_pointer_frame) {
                newframe = drag_info.current_pointer_frame - drag_info.pointer_frame_offset;
        }
        else {
                newframe = 0;
        }
-               
-       jack_nframes_t next = newframe;
+
+       nframes_t next = newframe;
 
        if (!Keyboard::modifier_state_contains (event->button.state, Keyboard::snap_modifier())) {
                snap_to (newframe, 0, true);
        }
        
-       if (drag_info.current_pointer_frame == drag_info.last_pointer_frame) return;
+       if (drag_info.current_pointer_frame == drag_info.last_pointer_frame) { 
+               return;
+       }
 
        /* call this to find out if its the start or end */
        
@@ -2071,31 +2100,39 @@ Editor::marker_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
                move_both = true;
        }
 
-       if (is_start) { // start marker
+       if (copy_location->is_mark()) {
+               /* just move it */
 
-               if (move_both) {
-                       copy_location->set_start (newframe);
-                       copy_location->set_end (newframe + f_delta);
-               } else  if (newframe < copy_location->end()) {
-                       copy_location->set_start (newframe);
-               } else { 
-                       snap_to (next, 1, true);
-                       copy_location->set_end (next);
-                       copy_location->set_start (newframe);
-               }
+               copy_location->set_start (newframe);
 
-       } else { // end marker
+       } else {
 
-               if (move_both) {
-                       copy_location->set_end (newframe);
-                       copy_location->set_start (newframe - f_delta);
-               } else if (newframe > copy_location->start()) {
-                       copy_location->set_end (newframe);
+               if (is_start) { // start-of-range marker
+                       
+                       if (move_both) {
+                               copy_location->set_start (newframe);
+                               copy_location->set_end (newframe + f_delta);
+                       } else  if (newframe < copy_location->end()) {
+                               copy_location->set_start (newframe);
+                       } else { 
+                               snap_to (next, 1, true);
+                               copy_location->set_end (next);
+                               copy_location->set_start (newframe);
+                       }
                        
-               } else if (newframe > 0) {
-                       snap_to (next, -1, true);
-                       copy_location->set_start (next);
-                       copy_location->set_end (newframe);
+               } else { // end marker
+                       
+                       if (move_both) {
+                               copy_location->set_end (newframe);
+                               copy_location->set_start (newframe - f_delta);
+                       } else if (newframe > copy_location->start()) {
+                               copy_location->set_end (newframe);
+                               
+                       } else if (newframe > 0) {
+                               snap_to (next, -1, true);
+                               copy_location->set_start (next);
+                               copy_location->set_end (newframe);
+                       }
                }
        }
 
@@ -2122,17 +2159,21 @@ Editor::marker_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent* event
        bool is_start;
 
 
-
        begin_reversible_command ( _("move marker") );
-       session->add_undo( session->locations()->get_memento() );
+       XMLNode &before = session->locations()->get_state();
        
        Location * location = find_location_from_marker (marker, is_start);
        
        if (location) {
-               location->set (drag_info.copied_location->start(), drag_info.copied_location->end());
+               if (location->is_mark()) {
+                       location->set_start (drag_info.copied_location->start());
+               } else {
+                       location->set (drag_info.copied_location->start(), drag_info.copied_location->end());
+               }
        }
 
-       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));
        commit_reversible_command ();
        
        marker_drag_line->hide();
@@ -2207,7 +2248,7 @@ void
 Editor::meter_marker_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
        MeterMarker* marker = (MeterMarker *) drag_info.data;
-       jack_nframes_t adjusted_frame;
+       nframes_t adjusted_frame;
 
        if ((long)drag_info.current_pointer_frame > drag_info.pointer_frame_offset) {
                adjusted_frame = drag_info.current_pointer_frame - drag_info.pointer_frame_offset;
@@ -2246,9 +2287,10 @@ Editor::meter_marker_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent*
        
        if (drag_info.copy == true) {
                begin_reversible_command (_("copy meter mark"));
-               session->add_undo (map.get_memento());
+                XMLNode &before = map.get_state();
                map.add_meter (marker->meter(), when);
-               session->add_redo_no_execute (map.get_memento());
+               XMLNode &after = map.get_state();
+                session->add_command(new MementoCommand<TempoMap>(map, &before, &after));
                commit_reversible_command ();
                
                // delete the dummy marker we used for visual representation of copying.
@@ -2256,9 +2298,10 @@ Editor::meter_marker_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent*
                delete marker;
        } else {
                begin_reversible_command (_("move meter mark"));
-               session->add_undo (map.get_memento());
+               XMLNode &before = map.get_state();
                map.move_meter (marker->meter(), when);
-               session->add_redo_no_execute (map.get_memento());
+               XMLNode &after = map.get_state();
+                session->add_command(new MementoCommand<TempoMap>(map, &before, &after));
                commit_reversible_command ();
        }
 }
@@ -2336,7 +2379,7 @@ void
 Editor::tempo_marker_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
        TempoMarker* marker = (TempoMarker *) drag_info.data;
-       jack_nframes_t adjusted_frame;
+       nframes_t adjusted_frame;
        
        if ((long)drag_info.current_pointer_frame > drag_info.pointer_frame_offset) {
                adjusted_frame = drag_info.current_pointer_frame - drag_info.pointer_frame_offset;
@@ -2373,12 +2416,13 @@ Editor::tempo_marker_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent*
        
        TempoMap& map (session->tempo_map());
        map.bbt_time (drag_info.last_pointer_frame, when);
-       
+
        if (drag_info.copy == true) {
                begin_reversible_command (_("copy tempo mark"));
-               session->add_undo (map.get_memento());
+               XMLNode &before = map.get_state();
                map.add_tempo (marker->tempo(), when);
-               session->add_redo_no_execute (map.get_memento());
+               XMLNode &after = map.get_state();
+               session->add_command (new MementoCommand<TempoMap>(map, &before, &after));
                commit_reversible_command ();
                
                // delete the dummy marker we used for visual representation of copying.
@@ -2386,9 +2430,10 @@ Editor::tempo_marker_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent*
                delete marker;
        } else {
                begin_reversible_command (_("move tempo mark"));
-               session->add_undo (map.get_memento());
+                XMLNode &before = map.get_state();
                map.move_tempo (marker->tempo(), when);
-               session->add_redo_no_execute (map.get_memento());
+                XMLNode &after = map.get_state();
+                session->add_command (new MementoCommand<TempoMap>(map, &before, &after));
                commit_reversible_command ();
        }
 }
@@ -2476,7 +2521,7 @@ Editor::control_point_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent*
        cy = min ((double) cp->line.height(), cy);
 
        //translate cx to frames
-       jack_nframes_t cx_frames = unit_to_frame (cx);
+       nframes_t cx_frames = unit_to_frame (cx);
 
        if (!Keyboard::modifier_state_contains (event->button.state, Keyboard::snap_modifier()) && !drag_info.x_constrained) {
                snap_to (cx_frames);
@@ -2523,7 +2568,8 @@ Editor::start_line_grab_from_regionview (ArdourCanvas::Item* item, GdkEvent* eve
 {
        switch (mouse_mode) {
        case MouseGain:
-               start_line_grab (clicked_regionview->get_gain_line(), event);
+               assert(dynamic_cast<AudioRegionView*>(clicked_regionview));
+               start_line_grab (dynamic_cast<AudioRegionView*>(clicked_regionview)->get_gain_line(), event);
                break;
        default:
                break;
@@ -2548,7 +2594,7 @@ Editor::start_line_grab (AutomationLine* line, GdkEvent* event)
 {
        double cx;
        double cy;
-       jack_nframes_t frame_within_region;
+       nframes_t frame_within_region;
 
        /* need to get x coordinate in terms of parent (TimeAxisItemView)
           origin.
@@ -2557,7 +2603,7 @@ Editor::start_line_grab (AutomationLine* line, GdkEvent* event)
        cx = event->button.x;
        cy = event->button.y;
        line->parent_group().w2i (cx, cy);
-       frame_within_region = (jack_nframes_t) floor (cx * frames_per_unit);
+       frame_within_region = (nframes_t) floor (cx * frames_per_unit);
 
        if (!line->control_points_adjacent (frame_within_region, current_line_drag_info.before, 
                                            current_line_drag_info.after)) {
@@ -2617,7 +2663,7 @@ Editor::line_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent* event)
 void
 Editor::start_region_grab (ArdourCanvas::Item* item, GdkEvent* event)
 {
-       if (selection->audio_regions.empty() || clicked_regionview == 0) {
+       if (selection->regions.empty() || clicked_regionview == 0) {
                return;
        }
 
@@ -2631,13 +2677,13 @@ Editor::start_region_grab (ArdourCanvas::Item* item, GdkEvent* event)
 
        double speed = 1.0;
        TimeAxisView* tvp = clicked_trackview;
-       AudioTimeAxisView* tv = dynamic_cast<AudioTimeAxisView*>(tvp);
+       RouteTimeAxisView* tv = dynamic_cast<RouteTimeAxisView*>(tvp);
 
        if (tv && tv->is_audio_track()) {
                speed = tv->get_diskstream()->speed();
        }
        
-       drag_info.last_frame_position = (jack_nframes_t) (clicked_regionview->region.position() / speed);
+       drag_info.last_frame_position = (nframes_t) (clicked_regionview->region()->position() / speed);
        drag_info.pointer_frame_offset = drag_info.grab_frame - drag_info.last_frame_position;
        drag_info.last_trackview = &clicked_regionview->get_time_axis_view();
        // we want a move threshold
@@ -2651,77 +2697,18 @@ Editor::start_region_grab (ArdourCanvas::Item* item, GdkEvent* event)
 void
 Editor::start_region_copy_grab (ArdourCanvas::Item* item, GdkEvent* event)
 {
-       if (selection->audio_regions.empty() || clicked_regionview == 0) {
+       if (selection->regions.empty() || clicked_regionview == 0) {
                return;
        }
 
-       /* this is committed in the grab finished callback. */
-
-       begin_reversible_command (_("Drag region copy"));
-
-       /* duplicate the region(s) */
-       
-       vector<AudioRegionView*> new_regionviews;
-       
-       set<Playlist*> affected_playlists;
-       pair<set<Playlist*>::iterator,bool> insert_result;
-       
-       for (list<AudioRegionView*>::const_iterator i = selection->audio_regions.by_layer().begin(); i != selection->audio_regions.by_layer().end(); ++i) {
-               AudioRegionView* rv;
-               
-               rv = (*i);
-               
-               Playlist* to_playlist = rv->region.playlist();
-               AudioTimeAxisView* atv = dynamic_cast<AudioTimeAxisView*>(&rv->get_time_axis_view());
-               
-               insert_result = affected_playlists.insert (to_playlist);
-               if (insert_result.second) {
-                       session->add_undo (to_playlist->get_memento ());
-               }
-               
-               latest_regionview = 0;
-               
-               sigc::connection c = atv->view->AudioRegionViewAdded.connect (mem_fun(*this, &Editor::collect_new_region_view));
-               
-               /* create a new region with the same name.
-                */
-               
-               AudioRegion* newregion = new AudioRegion (rv->region);
-               
-               /* if the original region was locked, we don't care */
-               
-               newregion->set_locked (false);
-               
-               to_playlist->add_region (*newregion, (jack_nframes_t) (rv->region.position() * atv->get_diskstream()->speed()));
-               
-               c.disconnect ();
-               
-               if (latest_regionview) {
-                       new_regionviews.push_back (latest_regionview);
-               }
-       }
-       
-       if (new_regionviews.empty()) {
-               return;
-       }
-       
-       /* reset selection to new regionviews */
-       
-       selection->set (new_regionviews);
-       
-       /* reset drag_info data to reflect the fact that we are dragging the copies */
-       
-       drag_info.data = new_regionviews.front();
-       drag_info.item = new_regionviews.front()->get_canvas_group ();
-       
        drag_info.copy = true;
-       drag_info.motion_callback = &Editor::region_drag_motion_callback;
-       drag_info.finished_callback = &Editor::region_drag_finished_callback;
+       drag_info.item = item;
+       drag_info.data = clicked_regionview;    
 
        start_grab(event);
 
        TimeAxisView* tv = &clicked_regionview->get_time_axis_view();
-       AudioTimeAxisView* atv = dynamic_cast<AudioTimeAxisView*>(tv);
+       RouteTimeAxisView* atv = dynamic_cast<RouteTimeAxisView*>(tv);
        double speed = 1.0;
 
        if (atv && atv->is_audio_track()) {
@@ -2729,20 +2716,18 @@ Editor::start_region_copy_grab (ArdourCanvas::Item* item, GdkEvent* event)
        }
        
        drag_info.last_trackview = &clicked_regionview->get_time_axis_view();
-       drag_info.last_frame_position = (jack_nframes_t) (clicked_regionview->region.position() / speed);
+       drag_info.last_frame_position = (nframes_t) (clicked_regionview->region()->position() / speed);
        drag_info.pointer_frame_offset = drag_info.grab_frame - drag_info.last_frame_position;
        // we want a move threshold
        drag_info.want_move_threshold = true;
-
-       show_verbose_time_cursor (drag_info.last_frame_position, 10);
-
-       //begin_reversible_command (_("copy region(s)"));
+       drag_info.motion_callback = &Editor::region_drag_motion_callback;
+       drag_info.finished_callback = &Editor::region_drag_finished_callback;
 }
 
 void
 Editor::start_region_brush_grab (ArdourCanvas::Item* item, GdkEvent* event)
 {
-       if (selection->audio_regions.empty() || clicked_regionview == 0) {
+       if (selection->regions.empty() || clicked_regionview == 0) {
                return;
        }
 
@@ -2756,13 +2741,13 @@ Editor::start_region_brush_grab (ArdourCanvas::Item* item, GdkEvent* event)
 
        double speed = 1.0;
        TimeAxisView* tvp = clicked_trackview;
-       AudioTimeAxisView* tv = dynamic_cast<AudioTimeAxisView*>(tvp);
+       RouteTimeAxisView* tv = dynamic_cast<RouteTimeAxisView*>(tvp);
 
        if (tv && tv->is_audio_track()) {
                speed = tv->get_diskstream()->speed();
        }
        
-       drag_info.last_frame_position = (jack_nframes_t) (clicked_regionview->region.position() / speed);
+       drag_info.last_frame_position = (nframes_t) (clicked_regionview->region()->position() / speed);
        drag_info.pointer_frame_offset = drag_info.grab_frame - drag_info.last_frame_position;
        drag_info.last_trackview = &clicked_regionview->get_time_axis_view();
        // we want a move threshold
@@ -2777,15 +2762,86 @@ Editor::region_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
        double x_delta;
        double y_delta = 0;
-       AudioRegionView *rv = reinterpret_cast<AudioRegionView*> (drag_info.data); 
-       jack_nframes_t pending_region_position = 0;
+       RegionView* rv = reinterpret_cast<RegionView*> (drag_info.data); 
+       nframes_t pending_region_position = 0;
        int32_t pointer_y_span = 0, canvas_pointer_y_span = 0, original_pointer_order;
        int32_t visible_y_high = 0, visible_y_low = 512;  //high meaning higher numbered.. not the height on the screen
        bool clamp_y_axis = false;
        vector<int32_t>  height_list(512) ;
        vector<int32_t>::iterator j;
 
-       cerr << "region motion to " << drag_info.current_pointer_frame << endl;
+       show_verbose_time_cursor (drag_info.last_frame_position, 10);
+
+       if (drag_info.copy && drag_info.move_threshold_passed && drag_info.want_move_threshold) {
+
+               drag_info.want_move_threshold = false; // don't copy again
+
+               /* this is committed in the grab finished callback. */
+               
+               begin_reversible_command (_("Drag region copy"));
+               
+               /* duplicate the region(s) */
+               
+               vector<RegionView*> new_regionviews;
+               
+               set<Playlist*> affected_playlists;
+               pair<set<Playlist*>::iterator,bool> insert_result;
+               
+               for (list<RegionView*>::const_iterator i = selection->regions.by_layer().begin(); i != selection->regions.by_layer().end(); ++i) {
+                       RegionView* rv;
+                       
+                       rv = (*i);
+                       
+                       Playlist* to_playlist = rv->region()->playlist();
+                       RouteTimeAxisView* atv = dynamic_cast<RouteTimeAxisView*>(&rv->get_time_axis_view());
+                       
+                       insert_result = affected_playlists.insert (to_playlist);
+                       if (insert_result.second) {
+                               session->add_command (new MementoCommand<Playlist>(*to_playlist, &to_playlist->get_state(), 0));
+                       }
+                       
+                       latest_regionview = 0;
+                       
+                       sigc::connection c = atv->view()->RegionViewAdded.connect (mem_fun(*this, &Editor::collect_new_region_view));
+                       
+                       /* create a new region with the same name. */
+                       
+                       // FIXME: ew.  need a (virtual) Region::duplicate() or something?
+
+                       boost::shared_ptr<Region> newregion;
+                       boost::shared_ptr<Region> ar;
+
+                       if ((ar = boost::dynamic_pointer_cast<AudioRegion>(rv->region())) != 0) {
+                               newregion = RegionFactory::create (ar);
+                       }
+                       assert(newregion != 0);
+
+                       /* if the original region was locked, we don't care */
+                       
+                       newregion->set_locked (false);
+                       
+                       to_playlist->add_region (newregion, (nframes_t) (rv->region()->position() * atv->get_diskstream()->speed()));
+                       
+                       c.disconnect ();
+                       
+                       if (latest_regionview) {
+                               new_regionviews.push_back (latest_regionview);
+                       }
+               }
+               
+               if (new_regionviews.empty()) {
+                       return;
+               }
+               
+               /* reset selection to new regionviews */
+               
+               selection->set (new_regionviews);
+               
+               /* reset drag_info data to reflect the fact that we are dragging the copies */
+               
+               drag_info.data = new_regionviews.front();
+               swap_grab (new_regionviews.front()->get_canvas_group (), 0, event->motion.time);
+       }
 
        /* Which trackview is this ? */
 
@@ -2876,16 +2932,15 @@ Editor::region_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
                        }
                }
 
-               for (list<AudioRegionView*>::const_iterator i = selection->audio_regions.by_layer().begin(); i != selection->audio_regions.by_layer().end(); ++i) {
-                       AudioRegionView* rv2;
-                       rv2 = (*i);
+               for (list<RegionView*>::const_iterator i = selection->regions.by_layer().begin(); i != selection->regions.by_layer().end(); ++i) {
+                       RegionView* rv2 = (*i);
                        double ix1, ix2, iy1, iy2;
                        int32_t n = 0;
 
                        rv2->get_canvas_frame()->get_bounds (ix1, iy1, ix2, iy2);
                        rv2->get_canvas_group()->i2w (ix1, iy1);
                        TimeAxisView* tvp2 = trackview_by_y_position (iy1);
-                       AudioTimeAxisView* atv2 = dynamic_cast<AudioTimeAxisView*>(tvp2);
+                       RouteTimeAxisView* atv2 = dynamic_cast<RouteTimeAxisView*>(tvp2);
 
                        if (atv2->order != original_pointer_order) {    
                                /* this isn't the pointer track */      
@@ -2970,18 +3025,18 @@ Editor::region_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
           the region would be if we moved it by that much.
        */
 
-       if (drag_info.move_threshold_passsed) {
+       if (drag_info.move_threshold_passed) {
 
                if ((int32_t)drag_info.current_pointer_frame > drag_info.pointer_frame_offset) {
 
-                       jack_nframes_t sync_frame;
-                       jack_nframes_t sync_offset;
+                       nframes_t sync_frame;
+                       nframes_t sync_offset;
                        int32_t sync_dir;
            
                        pending_region_position = drag_info.current_pointer_frame - drag_info.pointer_frame_offset;
            
-                       sync_offset = rv->region.sync_offset (sync_dir);
-                       sync_frame = rv->region.adjust_to_sync (pending_region_position);
+                       sync_offset = rv->region()->sync_offset (sync_dir);
+                       sync_frame = rv->region()->adjust_to_sync (pending_region_position);
 
                        /* we snap if the snap modifier is not enabled.
                         */
@@ -3000,7 +3055,7 @@ Editor::region_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
                        pending_region_position = 0;
                }
          
-               if (pending_region_position > max_frames - rv->region.length()) {
+               if (pending_region_position > max_frames - rv->region()->length()) {
                        pending_region_position = drag_info.last_frame_position;
                }
          
@@ -3042,14 +3097,11 @@ Editor::region_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
        } 
 
        if (x_delta < 0) {
-               for (list<AudioRegionView*>::const_iterator i = selection->audio_regions.by_layer().begin(); i != selection->audio_regions.by_layer().end(); ++i) {
+               for (list<RegionView*>::const_iterator i = selection->regions.by_layer().begin(); i != selection->regions.by_layer().end(); ++i) {
 
-                       AudioRegionView* rv2;
-                       rv2 = (*i);
+                       RegionView* rv2 = (*i);
 
-                       /* if any regionview is at zero, we need to know so we can 
-                          stop further leftward motion.
-                       */
+                       // If any regionview is at zero, we need to know so we can stop further leftward motion.
                        
                        double ix1, ix2, iy1, iy2;
                        rv2->get_canvas_frame()->get_bounds (ix1, iy1, ix2, iy2);
@@ -3067,12 +3119,11 @@ Editor::region_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
        ************************************************************/
 
        pair<set<Playlist*>::iterator,bool> insert_result;
-       const list<AudioRegionView*>& layered_regions = selection->audio_regions.by_layer();
+       const list<RegionView*>& layered_regions = selection->regions.by_layer();
 
-       for (list<AudioRegionView*>::const_iterator i = layered_regions.begin(); i != layered_regions.end(); ++i) {
+       for (list<RegionView*>::const_iterator i = layered_regions.begin(); i != layered_regions.end(); ++i) {
            
-               AudioRegionView* rv;
-               rv = (*i);
+               RegionView* rv = (*i);
                double ix1, ix2, iy1, iy2;
                int32_t temp_pointer_y_span = pointer_y_span;
 
@@ -3153,8 +3204,8 @@ Editor::region_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
                        if (-x_delta > ix1) {
                                x_delta = -ix1;
                        }
-               } else if ((x_delta > 0) &&(rv->region.last_frame() > max_frames - x_delta)) {
-                       x_delta = max_frames - rv->region.last_frame();
+               } else if ((x_delta > 0) &&(rv->region()->last_frame() > max_frames - x_delta)) {
+                       x_delta = max_frames - rv->region()->last_frame();
                }
                        
                if (drag_info.first_move) {
@@ -3179,14 +3230,14 @@ Editor::region_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 
                        AudioTimeAxisView* atv = dynamic_cast<AudioTimeAxisView*> (&rv->get_time_axis_view());
                        if (atv && atv->is_audio_track()) {
-                               AudioPlaylist* pl = atv->get_diskstream()->playlist();
+                               AudioPlaylist* pl = dynamic_cast<AudioPlaylist*>(atv->get_diskstream()->playlist());
                                if (pl) {
                                        /* only freeze and capture state once */
 
                                        insert_result = motion_frozen_playlists.insert (pl);
                                        if (insert_result.second) {
                                                pl->freeze();
-                                               session->add_undo(pl->get_memento());
+                                               session->add_command(new MementoCommand<Playlist>(*pl, &pl->get_state(), 0));
                                        }
                                }
                        }
@@ -3214,12 +3265,12 @@ Editor::region_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 void
 Editor::region_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
-       jack_nframes_t where;
-       AudioRegionView* rv = reinterpret_cast<AudioRegionView *> (drag_info.data);
+       nframes_t where;
+       RegionView* rv = reinterpret_cast<RegionView *> (drag_info.data);
        pair<set<Playlist*>::iterator,bool> insert_result;
        bool nocommit = true;
        double speed;
-       AudioTimeAxisView* atv;
+       RouteTimeAxisView* atv;
        bool regionview_y_movement;
        bool regionview_x_movement;
 
@@ -3253,7 +3304,7 @@ Editor::region_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent* event
                speed = atv->get_diskstream()->speed();
        }
        
-       regionview_x_movement = (drag_info.last_frame_position != (jack_nframes_t) (rv->region.position()/speed));
+       regionview_x_movement = (drag_info.last_frame_position != (nframes_t) (rv->region()->position()/speed));
        regionview_y_movement = (drag_info.last_trackview != &rv->get_time_axis_view());
 
        //printf ("last_frame: %s position is %lu  %g\n", rv->get_time_axis_view().name().c_str(), drag_info.last_frame_position, speed); 
@@ -3263,13 +3314,13 @@ Editor::region_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent* event
 
                /* motion between tracks */
 
-               list<AudioRegionView*> new_selection;
+               list<RegionView*> new_selection;
        
                /* moved to a different audio track. */
 
-               for (list<AudioRegionView*>::const_iterator i = selection->audio_regions.by_layer().begin(); i != selection->audio_regions.by_layer().end(); ) {
+               for (list<RegionView*>::const_iterator i = selection->regions.by_layer().begin(); i != selection->regions.by_layer().end(); ) {
            
-                       AudioRegionView* rv2 = (*i);                
+                       RegionView* rv2 = (*i);             
            
                        /* the region that used to be in the old playlist is not
                           moved to the new one - we make a copy of it. as a result,
@@ -3286,7 +3337,7 @@ Editor::region_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent* event
 
                /* first, freeze the target tracks */
 
-               for (list<AudioRegionView*>::const_iterator i = new_selection.begin(); i != new_selection.end();i++ ) {
+               for (list<RegionView*>::const_iterator i = new_selection.begin(); i != new_selection.end();i++ ) {
 
                        Playlist* from_playlist;
                        Playlist* to_playlist;
@@ -3298,7 +3349,7 @@ Editor::region_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent* event
                        TimeAxisView* tvp2 = trackview_by_y_position (iy1);
                        AudioTimeAxisView* atv2 = dynamic_cast<AudioTimeAxisView*>(tvp2);
            
-                       from_playlist = (*i)->region.playlist();
+                       from_playlist = (*i)->region()->playlist();
                        to_playlist = atv2->playlist();
 
                        /* the from_playlist was frozen in the "first_move" case 
@@ -3314,14 +3365,14 @@ Editor::region_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent* event
                        insert_result = motion_frozen_playlists.insert(to_playlist);
                        if (insert_result.second) {
                                to_playlist->freeze();
-                               session->add_undo(to_playlist->get_memento());
+                                session->add_command(new MementoCommand<Playlist>(*to_playlist, &to_playlist->get_state(), 0));
                        }
 
                }
 
                /* now do it again with the actual operations */
 
-               for (list<AudioRegionView*>::const_iterator i = new_selection.begin(); i != new_selection.end();i++ ) {
+               for (list<RegionView*>::const_iterator i = new_selection.begin(); i != new_selection.end();i++ ) {
 
                        Playlist* from_playlist;
                        Playlist* to_playlist;
@@ -3333,20 +3384,20 @@ Editor::region_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent* event
                        TimeAxisView* tvp2 = trackview_by_y_position (iy1);
                        AudioTimeAxisView* atv2 = dynamic_cast<AudioTimeAxisView*>(tvp2);
            
-                       from_playlist = (*i)->region.playlist();
+                       from_playlist = (*i)->region()->playlist();
                        to_playlist = atv2->playlist();
 
                        latest_regionview = 0;
            
-                       where = (jack_nframes_t) (unit_to_frame (ix1) * speed);
-                       Region* new_region = createRegion ((*i)->region);
+                       where = (nframes_t) (unit_to_frame (ix1) * speed);
+                       boost::shared_ptr<Region> new_region (RegionFactory::create ((*i)->region()));
 
-                       from_playlist->remove_region (&((*i)->region));
-         
-                       sigc::connection c = atv2->view->AudioRegionViewAdded.connect (mem_fun(*this, &Editor::collect_new_region_view));
-                       to_playlist->add_region (*new_region, where);
-                       c.disconnect ();
+                       from_playlist->remove_region (((*i)->region()));
                        
+                       sigc::connection c = atv2->view()->RegionViewAdded.connect (mem_fun(*this, &Editor::collect_new_region_view));
+                       to_playlist->add_region (new_region, where);
+                       c.disconnect ();
+                                                             
                        if (latest_regionview) {
                                selection->add (latest_regionview);
                        }
@@ -3356,11 +3407,11 @@ Editor::region_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent* event
 
                /* motion within a single track */
                
-               for (list<AudioRegionView*>::const_iterator i = selection->audio_regions.by_layer().begin(); i != selection->audio_regions.by_layer().end(); ++i) {
+               for (list<RegionView*>::const_iterator i = selection->regions.by_layer().begin(); i != selection->regions.by_layer().end(); ++i) {
 
                        rv = (*i);
 
-                       if (rv->region.locked()) {
+                       if (rv->region()->locked()) {
                                continue;
                        }
                        
@@ -3378,25 +3429,25 @@ Editor::region_drag_finished_callback (ArdourCanvas::Item* item, GdkEvent* event
                                
                                rv->get_canvas_frame()->get_bounds (ix1, iy1, ix2, iy2);
                                rv->get_canvas_group()->i2w (ix1, iy1);
-                               where = (jack_nframes_t) (unit_to_frame (ix1) * ownspeed);
+                               where = (nframes_t) (unit_to_frame (ix1) * ownspeed);
                                
                        } else {
                                
-                               where = rv->region.position();
+                               where = rv->region()->position();
                        }
 
                        rv->get_time_axis_view().reveal_dependent_views (*rv);
 
                        /* no need to add an undo here, we did that when we added this playlist to motion_frozen playlists */
                        
-                       rv->region.set_position (where, (void *) this);
+                       rv->region()->set_position (where, (void *) this);
                }
        }
 
   out:
        for (set<Playlist*>::iterator p = motion_frozen_playlists.begin(); p != motion_frozen_playlists.end(); ++p) {
                (*p)->thaw ();
-               session->add_redo_no_execute ((*p)->get_memento());
+               session->add_command (new MementoCommand<Playlist>(*(*p), 0, & (*p)->get_state()));
        }
 
        motion_frozen_playlists.clear ();
@@ -3423,24 +3474,24 @@ Editor::region_view_item_click (AudioRegionView& rv, GdkEventButton* event)
 
                if (Keyboard::modifier_state_equals (event->state, Keyboard::ModifierMask (Keyboard::Control|Keyboard::Alt))) {
 
-                       align_region (rv.region, SyncPoint, (jack_nframes_t) (edit_cursor->current_frame * speed));
+                       align_region (rv.region(), SyncPoint, (nframes_t) (edit_cursor->current_frame * speed));
 
                } else if (Keyboard::modifier_state_equals (event->state, Keyboard::ModifierMask (Keyboard::Control|Keyboard::Shift))) {
 
-                       align_region (rv.region, End, (jack_nframes_t) (edit_cursor->current_frame * speed));
+                       align_region (rv.region(), End, (nframes_t) (edit_cursor->current_frame * speed));
 
                } else {
 
-                       align_region (rv.region, Start, (jack_nframes_t) (edit_cursor->current_frame * speed));
+                       align_region (rv.region(), Start, (nframes_t) (edit_cursor->current_frame * speed));
                }
        }
 }
 
 void
-Editor::show_verbose_time_cursor (jack_nframes_t frame, double offset, double xpos, double ypos) 
+Editor::show_verbose_time_cursor (nframes_t frame, double offset, double xpos, double ypos) 
 {
        char buf[128];
-       SMPTE_Time smpte;
+       SMPTE::Time smpte;
        BBT_Time bbt;
        float secs;
 
@@ -3462,7 +3513,7 @@ Editor::show_verbose_time_cursor (jack_nframes_t frame, double offset, double xp
        case AudioClock::MinSec:
                /* XXX fix this to compute min/sec properly */
                session->smpte_time (frame, smpte);
-               secs = smpte.seconds + ((float) smpte.frames / session->smpte_frames_per_second);
+               secs = smpte.seconds + ((float) smpte.frames / Config->get_smpte_frames_per_second());
                snprintf (buf, sizeof (buf), "%02" PRId32 ":%02" PRId32 ":%.4f", smpte.hours, smpte.minutes, secs);
                break;
 
@@ -3481,10 +3532,10 @@ Editor::show_verbose_time_cursor (jack_nframes_t frame, double offset, double xp
 }
 
 void
-Editor::show_verbose_duration_cursor (jack_nframes_t start, jack_nframes_t end, double offset, double xpos, double ypos) 
+Editor::show_verbose_duration_cursor (nframes_t start, nframes_t end, double offset, double xpos, double ypos) 
 {
        char buf[128];
-       SMPTE_Time smpte;
+       SMPTE::Time smpte;
        BBT_Time sbbt;
        BBT_Time ebbt;
        float secs;
@@ -3529,8 +3580,8 @@ Editor::show_verbose_duration_cursor (jack_nframes_t start, jack_nframes_t end,
        case AudioClock::MinSec:
                /* XXX fix this to compute min/sec properly */
                session->smpte_duration (end - start, smpte);
-               secs = smpte.seconds + ((float) smpte.frames / session->smpte_frames_per_second);
-               snprintf (buf, sizeof (buf), "%02ld:%02ld:%.4f", smpte.hours, smpte.minutes, secs);
+               secs = smpte.seconds + ((float) smpte.frames / Config->get_smpte_frames_per_second());
+               snprintf (buf, sizeof (buf), "%02" PRId32 ":%02" PRId32 ":%.4f", smpte.hours, smpte.minutes, secs);
                break;
 
        default:
@@ -3548,7 +3599,7 @@ Editor::show_verbose_duration_cursor (jack_nframes_t start, jack_nframes_t end,
 }
 
 void
-Editor::collect_new_region_view (AudioRegionView* rv)
+Editor::collect_new_region_view (RegionView* rv)
 {
        latest_regionview = rv;
 }
@@ -3562,7 +3613,7 @@ Editor::start_selection_grab (ArdourCanvas::Item* item, GdkEvent* event)
 
        /* lets try to create new Region for the selection */
 
-       vector<AudioRegion*> new_regions;
+       vector<boost::shared_ptr<AudioRegion> > new_regions;
        create_region_from_selection (new_regions);
 
        if (new_regions.empty()) {
@@ -3571,7 +3622,7 @@ Editor::start_selection_grab (ArdourCanvas::Item* item, GdkEvent* event)
 
        /* XXX fix me one day to use all new regions */
        
-       Region* region = new_regions.front();
+       boost::shared_ptr<Region> region (new_regions.front());
 
        /* add it to the current stream/playlist.
 
@@ -3581,7 +3632,7 @@ Editor::start_selection_grab (ArdourCanvas::Item* item, GdkEvent* event)
        */
        
        latest_regionview = 0;
-       sigc::connection c = clicked_audio_trackview->view->AudioRegionViewAdded.connect (mem_fun(*this, &Editor::collect_new_region_view));
+       sigc::connection c = clicked_audio_trackview->view()->RegionViewAdded.connect (mem_fun(*this, &Editor::collect_new_region_view));
        
        /* A selection grab currently creates two undo/redo operations, one for 
           creating the new region and another for moving it.
@@ -3591,9 +3642,10 @@ Editor::start_selection_grab (ArdourCanvas::Item* item, GdkEvent* event)
 
        Playlist* playlist = clicked_trackview->playlist();
 
-       session->add_undo (playlist->get_memento ());
-       clicked_trackview->playlist()->add_region (*region, selection->time[clicked_selection].start);
-       session->add_redo_no_execute (playlist->get_memento ());
+        XMLNode *before = &(playlist->get_state());
+       clicked_trackview->playlist()->add_region (region, selection->time[clicked_selection].start);
+        XMLNode *after = &(playlist->get_state());
+       session->add_command(new MementoCommand<Playlist>(*playlist, before, after));
 
        commit_reversible_command ();
        
@@ -3616,7 +3668,7 @@ Editor::start_selection_grab (ArdourCanvas::Item* item, GdkEvent* event)
        start_grab (event);
        
        drag_info.last_trackview = clicked_trackview;
-       drag_info.last_frame_position = latest_regionview->region.position();
+       drag_info.last_frame_position = latest_regionview->region()->position();
        drag_info.pointer_frame_offset = drag_info.grab_frame - drag_info.last_frame_position;
        
        show_verbose_time_cursor (drag_info.last_frame_position, 10);
@@ -3637,8 +3689,8 @@ Editor::cancel_selection ()
 void
 Editor::start_selection_op (ArdourCanvas::Item* item, GdkEvent* event, SelectionOp op)
 {
-       jack_nframes_t start = 0;
-       jack_nframes_t end = 0;
+       nframes_t start = 0;
+       nframes_t end = 0;
 
        if (session == 0) {
                return;
@@ -3652,7 +3704,6 @@ Editor::start_selection_op (ArdourCanvas::Item* item, GdkEvent* event, Selection
 
        switch (op) {
        case CreateSelection:
-               
                if (Keyboard::modifier_state_equals (event->button.state, Keyboard::Shift)) {
                        drag_info.copy = true;
                } else {
@@ -3662,14 +3713,18 @@ Editor::start_selection_op (ArdourCanvas::Item* item, GdkEvent* event, Selection
                break;
 
        case SelectionStartTrim:
-               clicked_trackview->order_selection_trims (item, true);
+               if (clicked_trackview) {
+                       clicked_trackview->order_selection_trims (item, true);
+               } 
                start_grab (event, trimmer_cursor);
                start = selection->time[clicked_selection].start;
                drag_info.pointer_frame_offset = drag_info.grab_frame - start;  
                break;
                
        case SelectionEndTrim:
-               clicked_trackview->order_selection_trims (item, false);
+               if (clicked_trackview) {
+                       clicked_trackview->order_selection_trims (item, false);
+               }
                start_grab (event, trimmer_cursor);
                end = selection->time[clicked_selection].end;
                drag_info.pointer_frame_offset = drag_info.grab_frame - end;    
@@ -3692,10 +3747,10 @@ Editor::start_selection_op (ArdourCanvas::Item* item, GdkEvent* event, Selection
 void
 Editor::drag_selection (ArdourCanvas::Item* item, GdkEvent* event)
 {
-       jack_nframes_t start = 0;
-       jack_nframes_t end = 0;
-       jack_nframes_t length;
-       jack_nframes_t pending_position;
+       nframes_t start = 0;
+       nframes_t end = 0;
+       nframes_t length;
+       nframes_t pending_position;
 
        if ((int32_t) drag_info.current_pointer_frame > drag_info.pointer_frame_offset) {
                pending_position = drag_info.current_pointer_frame - drag_info.pointer_frame_offset;
@@ -3854,9 +3909,9 @@ Editor::start_trim (ArdourCanvas::Item* item, GdkEvent* event)
                speed = tv->get_diskstream()->speed();
        }
        
-       jack_nframes_t region_start = (jack_nframes_t) (clicked_regionview->region.position() / speed);
-       jack_nframes_t region_end = (jack_nframes_t) (clicked_regionview->region.last_frame() / speed);
-       jack_nframes_t region_length = (jack_nframes_t) (clicked_regionview->region.length() / speed);
+       nframes_t region_start = (nframes_t) (clicked_regionview->region()->position() / speed);
+       nframes_t region_end = (nframes_t) (clicked_regionview->region()->last_frame() / speed);
+       nframes_t region_length = (nframes_t) (clicked_regionview->region()->length() / speed);
 
        motion_frozen_playlists.clear();
        
@@ -3896,8 +3951,8 @@ Editor::start_trim (ArdourCanvas::Item* item, GdkEvent* event)
 void
 Editor::trim_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 {
-       AudioRegionView* rv = clicked_regionview;
-       jack_nframes_t frame_delta = 0;
+       RegionView* rv = clicked_regionview;
+       nframes_t frame_delta = 0;
        bool left_direction;
        bool obey_snap = !Keyboard::modifier_state_contains (event->button.state, Keyboard::snap_modifier());
 
@@ -3908,7 +3963,7 @@ Editor::trim_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 
        double speed = 1.0;
        TimeAxisView* tvp = clicked_trackview;
-       AudioTimeAxisView* tv = dynamic_cast<AudioTimeAxisView*>(tvp);
+       RouteTimeAxisView* tv = dynamic_cast<RouteTimeAxisView*>(tvp);
        pair<set<Playlist*>::iterator,bool> insert_result;
 
        if (tv && tv->is_audio_track()) {
@@ -3947,14 +4002,17 @@ Editor::trim_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 
                begin_reversible_command (trim_type);
 
-               for (list<AudioRegionView*>::const_iterator i = selection->audio_regions.by_layer().begin(); i != selection->audio_regions.by_layer().end(); ++i) {
-                       (*i)->region.freeze ();
-                       (*i)->temporarily_hide_envelope ();
+               for (list<RegionView*>::const_iterator i = selection->regions.by_layer().begin(); i != selection->regions.by_layer().end(); ++i) {
+                       (*i)->region()->freeze ();
+               
+                       AudioRegionView* const arv = dynamic_cast<AudioRegionView*>(*i);
+                       if (arv)
+                               arv->temporarily_hide_envelope ();
 
-                       Playlist * pl = (*i)->region.playlist();
+                       Playlist * pl = (*i)->region()->playlist();
                        insert_result = motion_frozen_playlists.insert (pl);
                        if (insert_result.second) {
-                               session->add_undo (pl->get_memento());
+                                session->add_command(new MementoCommand<Playlist>(*pl, &pl->get_state(), 0));
                        }
                }
        }
@@ -3967,20 +4025,20 @@ Editor::trim_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 
        switch (trim_op) {              
        case StartTrim:
-               if ((left_direction == false) && (drag_info.current_pointer_frame <= rv->region.first_frame()/speed)) {
+               if ((left_direction == false) && (drag_info.current_pointer_frame <= rv->region()->first_frame()/speed)) {
                        break;
                 } else {
-                       for (list<AudioRegionView*>::const_iterator i = selection->audio_regions.by_layer().begin(); i != selection->audio_regions.by_layer().end(); ++i) {
+                       for (list<RegionView*>::const_iterator i = selection->regions.by_layer().begin(); i != selection->regions.by_layer().end(); ++i) {
                                single_start_trim (**i, frame_delta, left_direction, obey_snap);
                        }
                        break;
                }
                
        case EndTrim:
-               if ((left_direction == true) && (drag_info.current_pointer_frame > (jack_nframes_t) (rv->region.last_frame()/speed))) {
+               if ((left_direction == true) && (drag_info.current_pointer_frame > (nframes_t) (rv->region()->last_frame()/speed))) {
                        break;
                } else {
-                       for (list<AudioRegionView*>::const_iterator i = selection->audio_regions.by_layer().begin(); i != selection->audio_regions.by_layer().end(); ++i) {
+                       for (list<RegionView*>::const_iterator i = selection->regions.by_layer().begin(); i != selection->regions.by_layer().end(); ++i) {
                                single_end_trim (**i, frame_delta, left_direction, obey_snap);
                        }
                        break;
@@ -3994,8 +4052,8 @@ Editor::trim_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
                                swap_direction = true;
                        }
                        
-                       for (list<AudioRegionView*>::const_iterator i = selection->audio_regions.by_layer().begin();
-                            i != selection->audio_regions.by_layer().end(); ++i)
+                       for (list<RegionView*>::const_iterator i = selection->regions.by_layer().begin();
+                            i != selection->regions.by_layer().end(); ++i)
                        {
                                single_contents_trim (**i, frame_delta, left_direction, swap_direction, obey_snap);
                        }
@@ -4005,10 +4063,10 @@ Editor::trim_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 
        switch (trim_op) {
        case StartTrim:
-               show_verbose_time_cursor((jack_nframes_t) (rv->region.position()/speed), 10);   
+               show_verbose_time_cursor((nframes_t) (rv->region()->position()/speed), 10);     
                break;
        case EndTrim:
-               show_verbose_time_cursor((jack_nframes_t) (rv->region.last_frame()/speed), 10); 
+               show_verbose_time_cursor((nframes_t) (rv->region()->last_frame()/speed), 10);   
                break;
        case ContentsTrim:
                show_verbose_time_cursor(drag_info.current_pointer_frame, 10);  
@@ -4020,19 +4078,19 @@ Editor::trim_motion_callback (ArdourCanvas::Item* item, GdkEvent* event)
 }
 
 void
-Editor::single_contents_trim (AudioRegionView& rv, jack_nframes_t frame_delta, bool left_direction, bool swap_direction, bool obey_snap)
+Editor::single_contents_trim (RegionView& rv, nframes_t frame_delta, bool left_direction, bool swap_direction, bool obey_snap)
 {
-       Region& region (rv.region);
+       boost::shared_ptr<Region> region (rv.region());
 
-       if (region.locked()) {
+       if (region->locked()) {
                return;
        }
 
-       jack_nframes_t new_bound;
+       nframes_t new_bound;
 
        double speed = 1.0;
        TimeAxisView* tvp = clicked_trackview;
-       AudioTimeAxisView* tv = dynamic_cast<AudioTimeAxisView*>(tvp);
+       RouteTimeAxisView* tv = dynamic_cast<RouteTimeAxisView*>(tvp);
 
        if (tv && tv->is_audio_track()) {
                speed = tv->get_diskstream()->speed();
@@ -4040,35 +4098,35 @@ Editor::single_contents_trim (AudioRegionView& rv, jack_nframes_t frame_delta, b
        
        if (left_direction) {
                if (swap_direction) {
-                       new_bound = (jack_nframes_t) (region.position()/speed) + frame_delta;
+                       new_bound = (nframes_t) (region->position()/speed) + frame_delta;
                } else {
-                       new_bound = (jack_nframes_t) (region.position()/speed) - frame_delta;
+                       new_bound = (nframes_t) (region->position()/speed) - frame_delta;
                }
        } else {
                if (swap_direction) {
-                       new_bound = (jack_nframes_t) (region.position()/speed) - frame_delta;
+                       new_bound = (nframes_t) (region->position()/speed) - frame_delta;
                } else {
-                       new_bound = (jack_nframes_t) (region.position()/speed) + frame_delta;
+                       new_bound = (nframes_t) (region->position()/speed) + frame_delta;
                }
        }
 
        if (obey_snap) {
                snap_to (new_bound);
        }
-       region.trim_start ((jack_nframes_t) (new_bound * speed), this); 
+       region->trim_start ((nframes_t) (new_bound * speed), this);     
        rv.region_changed (StartChanged);
 }
 
 void
-Editor::single_start_trim (AudioRegionView& rv, jack_nframes_t frame_delta, bool left_direction, bool obey_snap)
+Editor::single_start_trim (RegionView& rv, nframes_t frame_delta, bool left_direction, bool obey_snap)
 {
-       Region& region (rv.region);     
+       boost::shared_ptr<Region> region (rv.region()); 
 
-       if (region.locked()) {
+       if (region->locked()) {
                return;
        }
 
-       jack_nframes_t new_bound;
+       nframes_t new_bound;
 
        double speed = 1.0;
        TimeAxisView* tvp = clicked_trackview;
@@ -4079,30 +4137,30 @@ Editor::single_start_trim (AudioRegionView& rv, jack_nframes_t frame_delta, bool
        }
        
        if (left_direction) {
-               new_bound = (jack_nframes_t) (region.position()/speed) - frame_delta;
+               new_bound = (nframes_t) (region->position()/speed) - frame_delta;
        } else {
-               new_bound = (jack_nframes_t) (region.position()/speed) + frame_delta;
+               new_bound = (nframes_t) (region->position()/speed) + frame_delta;
        }
 
        if (obey_snap) {
                snap_to (new_bound, (left_direction ? 0 : 1));  
        }
 
-       region.trim_front ((jack_nframes_t) (new_bound * speed), this);
+       region->trim_front ((nframes_t) (new_bound * speed), this);
 
        rv.region_changed (Change (LengthChanged|PositionChanged|StartChanged));
 }
 
 void
-Editor::single_end_trim (AudioRegionView& rv, jack_nframes_t frame_delta, bool left_direction, bool obey_snap)
+Editor::single_end_trim (RegionView& rv, nframes_t frame_delta, bool left_direction, bool obey_snap)
 {
-       Region& region (rv.region);
+       boost::shared_ptr<Region> region (rv.region());
 
-       if (region.locked()) {
+       if (region->locked()) {
                return;
        }
 
-       jack_nframes_t new_bound;
+       nframes_t new_bound;
 
        double speed = 1.0;
        TimeAxisView* tvp = clicked_trackview;
@@ -4113,15 +4171,15 @@ Editor::single_end_trim (AudioRegionView& rv, jack_nframes_t frame_delta, bool l
        }
        
        if (left_direction) {
-               new_bound = (jack_nframes_t) ((region.last_frame() + 1)/speed) - frame_delta;
+               new_bound = (nframes_t) ((region->last_frame() + 1)/speed) - frame_delta;
        } else {
-               new_bound = (jack_nframes_t) ((region.last_frame() + 1)/speed) + frame_delta;
+               new_bound = (nframes_t) ((region->last_frame() + 1)/speed) + frame_delta;
        }
 
        if (obey_snap) {
                snap_to (new_bound);
        }
-       region.trim_end ((jack_nframes_t) (new_bound * speed), this);
+       region->trim_end ((nframes_t) (new_bound * speed), this);
        rv.region_changed (LengthChanged);
 }
        
@@ -4135,8 +4193,8 @@ Editor::trim_finished_callback (ArdourCanvas::Item* item, GdkEvent* event)
                        thaw_region_after_trim (*clicked_regionview);           
                } else {
                        
-                       for (list<AudioRegionView*>::const_iterator i = selection->audio_regions.by_layer().begin();
-                            i != selection->audio_regions.by_layer().end(); ++i)
+                       for (list<RegionView*>::const_iterator i = selection->regions.by_layer().begin();
+                            i != selection->regions.by_layer().end(); ++i)
                        {
                                thaw_region_after_trim (**i);
                        }
@@ -4144,8 +4202,8 @@ Editor::trim_finished_callback (ArdourCanvas::Item* item, GdkEvent* event)
                
                for (set<Playlist*>::iterator p = motion_frozen_playlists.begin(); p != motion_frozen_playlists.end(); ++p) {
                        //(*p)->thaw ();
-                       session->add_redo_no_execute ((*p)->get_memento());
-               }
+                        session->add_command (new MementoCommand<Playlist>(*(*p), 0, &(*p)->get_state()));
+                }
                
                motion_frozen_playlists.clear ();
 
@@ -4159,8 +4217,8 @@ Editor::trim_finished_callback (ArdourCanvas::Item* item, GdkEvent* event)
 void
 Editor::point_trim (GdkEvent* event)
 {
-       AudioRegionView* rv = clicked_regionview;
-       jack_nframes_t new_bound = drag_info.current_pointer_frame;
+       RegionView* rv = clicked_regionview;
+       nframes_t new_bound = drag_info.current_pointer_frame;
 
        if (!Keyboard::modifier_state_contains (event->button.state, Keyboard::snap_modifier())) {
                snap_to (new_bound);
@@ -4174,22 +4232,26 @@ Editor::point_trim (GdkEvent* event)
 
                if (rv->get_selected()) {
 
-                       for (list<AudioRegionView*>::const_iterator i = selection->audio_regions.by_layer().begin();
-                            i != selection->audio_regions.by_layer().end(); ++i)
+                       for (list<RegionView*>::const_iterator i = selection->regions.by_layer().begin();
+                            i != selection->regions.by_layer().end(); ++i)
                        {
-                               if (!(*i)->region.locked()) {
-                                       session->add_undo ((*i)->region.playlist()->get_memento());
-                                       (*i)->region.trim_front (new_bound, this);      
-                                       session->add_redo_no_execute ((*i)->region.playlist()->get_memento());
+                               if (!(*i)->region()->locked()) {
+                                        Playlist *pl = (*i)->region()->playlist();
+                                        XMLNode &before = pl->get_state();
+                                       (*i)->region()->trim_front (new_bound, this);   
+                                        XMLNode &after = pl->get_state();
+                                        session->add_command(new MementoCommand<Playlist>(*pl, &before, &after));
                                }
                        }
 
                } else {
 
-                       if (!rv->region.locked()) {
-                               session->add_undo (rv->region.playlist()->get_memento());
-                               rv->region.trim_front (new_bound, this);        
-                               session->add_redo_no_execute (rv->region.playlist()->get_memento());
+                       if (!rv->region()->locked()) {
+                                Playlist *pl = rv->region()->playlist();
+                               XMLNode &before = pl->get_state();
+                               rv->region()->trim_front (new_bound, this);     
+                                XMLNode &after = pl->get_state();
+                               session->add_command(new MementoCommand<Playlist>(*pl, &before, &after));
                        }
                }
 
@@ -4202,21 +4264,25 @@ Editor::point_trim (GdkEvent* event)
 
                if (rv->get_selected()) {
                        
-                       for (list<AudioRegionView*>::const_iterator i = selection->audio_regions.by_layer().begin(); i != selection->audio_regions.by_layer().end(); ++i)
+                       for (list<RegionView*>::const_iterator i = selection->regions.by_layer().begin(); i != selection->regions.by_layer().end(); ++i)
                        {
-                               if (!(*i)->region.locked()) {
-                                       session->add_undo ((*i)->region.playlist()->get_memento());
-                                       (*i)->region.trim_end (new_bound, this);
-                                       session->add_redo_no_execute ((*i)->region.playlist()->get_memento());  
+                               if (!(*i)->region()->locked()) {
+                                        Playlist *pl = (*i)->region()->playlist();
+                                       XMLNode &before = pl->get_state();
+                                       (*i)->region()->trim_end (new_bound, this);
+                                       XMLNode &after = pl->get_state();
+                                       session->add_command(new MementoCommand<Playlist>(*pl, &before, &after));
                                }
                        }
 
                } else {
 
-                       if (!rv->region.locked()) {
-                               session->add_undo (rv->region.playlist()->get_memento());
-                               rv->region.trim_end (new_bound, this);
-                               session->add_redo_no_execute (rv->region.playlist()->get_memento());    
+                       if (!rv->region()->locked()) {
+                                Playlist *pl = rv->region()->playlist();
+                               XMLNode &before = pl->get_state();
+                               rv->region()->trim_end (new_bound, this);
+                                XMLNode &after = pl->get_state();
+                               session->add_command (new MementoCommand<Playlist>(*pl, &before, &after));
                        }
                }
 
@@ -4229,18 +4295,21 @@ Editor::point_trim (GdkEvent* event)
 }
 
 void
-Editor::thaw_region_after_trim (AudioRegionView& rv)
+Editor::thaw_region_after_trim (RegionView& rv)
 {
-       Region& region (rv.region);
+       boost::shared_ptr<Region> region (rv.region());
 
-       if (region.locked()) {
+       if (region->locked()) {
                return;
        }
 
-       region.thaw (_("trimmed region"));
-       session->add_redo_no_execute (region.playlist()->get_memento());
+       region->thaw (_("trimmed region"));
+        XMLNode &after = region->playlist()->get_state();
+       session->add_command (new MementoCommand<Playlist>(*(region->playlist()), 0, &after));
 
-       rv.unhide_envelope ();
+       AudioRegionView* arv = dynamic_cast<AudioRegionView*>(&rv);
+       if (arv)
+               arv->unhide_envelope ();
 }
 
 void
@@ -4262,7 +4331,6 @@ Editor::hide_marker (ArdourCanvas::Item* item, GdkEvent* event)
 void
 Editor::start_range_markerbar_op (ArdourCanvas::Item* item, GdkEvent* event, RangeMarkerOp op)
 {
-
        if (session == 0) {
                return;
        }
@@ -4297,8 +4365,8 @@ Editor::start_range_markerbar_op (ArdourCanvas::Item* item, GdkEvent* event, Ran
 void
 Editor::drag_range_markerbar_op (ArdourCanvas::Item* item, GdkEvent* event)
 {
-       jack_nframes_t start = 0;
-       jack_nframes_t end = 0;
+       nframes_t start = 0;
+       nframes_t end = 0;
        ArdourCanvas::SimpleRect *crect = (range_marker_op == CreateRangeMarker) ? range_bar_drag_rect: transport_bar_drag_rect;
        
        if (!Keyboard::modifier_state_contains (event->button.state, Keyboard::snap_modifier())) {
@@ -4327,7 +4395,7 @@ Editor::drag_range_markerbar_op (ArdourCanvas::Item* item, GdkEvent* event)
                }
                
                /* first drag: Either add to the selection
-                  or create a new selection->
+                  or create a new selection.
                */
                
                if (drag_info.first_move) {
@@ -4376,16 +4444,19 @@ Editor::end_range_markerbar_op (ArdourCanvas::Item* item, GdkEvent* event)
 
                switch (range_marker_op) {
                case CreateRangeMarker:
+                   {
                        begin_reversible_command (_("new range marker"));
-                       session->add_undo (session->locations()->get_memento());
+                        XMLNode &before = session->locations()->get_state();
                        newloc = new Location(temp_location->start(), temp_location->end(), "unnamed", Location::IsRangeMarker);
                        session->locations()->add (newloc, 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));
                        commit_reversible_command ();
                        
                        range_bar_drag_rect->hide();
                        range_marker_drag_rect->hide();
                        break;
+                   }
 
                case CreateTransportMarker:
                        // popup menu to pick loop or punch
@@ -4394,12 +4465,39 @@ Editor::end_range_markerbar_op (ArdourCanvas::Item* item, GdkEvent* event)
                        break;
                }
        } else {
-               /* just a click, no pointer movement.*/
+               /* just a click, no pointer movement. remember that context menu stuff was handled elsewhere */
 
                if (Keyboard::no_modifier_keys_pressed (&event->button)) {
 
-                       // nothing yet
+                       nframes_t start;
+                       nframes_t end;
 
+                       start = session->locations()->first_mark_before (drag_info.grab_frame);
+                       end = session->locations()->first_mark_after (drag_info.grab_frame);
+                       
+                       if (end == max_frames) {
+                               end = session->current_end_frame ();
+                       }
+
+                       if (start == 0) {
+                               start = session->current_start_frame ();
+                       }
+
+                       switch (mouse_mode) {
+                       case MouseObject:
+                               /* find the two markers on either side and then make the selection from it */
+                               cerr << "select between " << start << " .. " << end << endl;
+                               select_all_within (start, end, 0.0f, FLT_MAX, Selection::Set);
+                               break;
+
+                       case MouseRange:
+                               /* find the two markers on either side of the click and make the range out of it */
+                               selection->set (0, start, end);
+                               break;
+
+                       default:
+                               break;
+                       }
                } 
        }
 
@@ -4423,8 +4521,8 @@ Editor::start_mouse_zoom (ArdourCanvas::Item* item, GdkEvent* event)
 void
 Editor::drag_mouse_zoom (ArdourCanvas::Item* item, GdkEvent* event)
 {
-       jack_nframes_t start;
-       jack_nframes_t end;
+       nframes_t start;
+       nframes_t end;
 
        if (!Keyboard::modifier_state_contains (event->button.state, Keyboard::snap_modifier())) {
                snap_to (drag_info.current_pointer_frame);
@@ -4484,7 +4582,7 @@ Editor::end_mouse_zoom (ArdourCanvas::Item* item, GdkEvent* event)
 }
 
 void
-Editor::reposition_zoom_rect (jack_nframes_t start, jack_nframes_t end)
+Editor::reposition_zoom_rect (nframes_t start, nframes_t end)
 {
        double x1 = frame_to_pixel (start);
        double x2 = frame_to_pixel (end);
@@ -4511,8 +4609,8 @@ Editor::start_rubberband_select (ArdourCanvas::Item* item, GdkEvent* event)
 void
 Editor::drag_rubberband_select (ArdourCanvas::Item* item, GdkEvent* event)
 {
-       jack_nframes_t start;
-       jack_nframes_t end;
+       nframes_t start;
+       nframes_t end;
        double y1;
        double y2;
 
@@ -4604,7 +4702,7 @@ Editor::end_rubberband_select (ArdourCanvas::Item* item, GdkEvent* event)
                }
                
        } else {
-               selection->clear_audio_regions();
+               selection->clear_regions();
                selection->clear_points ();
                selection->clear_lines ();
        }
@@ -4621,7 +4719,7 @@ Editor::mouse_rename_region (ArdourCanvas::Item* item, GdkEvent* event)
        ArdourPrompter prompter (false);
 
        prompter.set_prompt (_("Name for region:"));
-       prompter.set_initial_text (clicked_regionview->region.name());
+       prompter.set_initial_text (clicked_regionview->region()->name());
        prompter.add_button (_("Rename"), Gtk::RESPONSE_ACCEPT);
        prompter.set_response_sensitive (Gtk::RESPONSE_ACCEPT, false);
        prompter.show_all ();
@@ -4630,7 +4728,7 @@ Editor::mouse_rename_region (ArdourCanvas::Item* item, GdkEvent* event)
         string str;
                prompter.get_result(str);
                if (str.length()) {
-               clicked_regionview->region.set_name (str);
+               clicked_regionview->region()->set_name (str);
                }
                break;
        }
@@ -4652,7 +4750,7 @@ Editor::start_time_fx (ArdourCanvas::Item* item, GdkEvent* event)
 void
 Editor::time_fx_motion (ArdourCanvas::Item *item, GdkEvent* event)
 {
-       AudioRegionView* rv = clicked_regionview;
+       RegionView* rv = clicked_regionview;
 
        if (!Keyboard::modifier_state_contains (event->button.state, Keyboard::snap_modifier())) {
                snap_to (drag_info.current_pointer_frame);
@@ -4662,8 +4760,8 @@ Editor::time_fx_motion (ArdourCanvas::Item *item, GdkEvent* event)
                return;
        }
 
-       if (drag_info.current_pointer_frame > rv->region.position()) {
-               rv->get_time_axis_view().show_timestretch (rv->region.position(), drag_info.current_pointer_frame);
+       if (drag_info.current_pointer_frame > rv->region()->position()) {
+               rv->get_time_axis_view().show_timestretch (rv->region()->position(), drag_info.current_pointer_frame);
        }
 
        drag_info.last_pointer_frame = drag_info.current_pointer_frame;
@@ -4681,21 +4779,25 @@ Editor::end_time_fx (ArdourCanvas::Item* item, GdkEvent* event)
                return;
        }
        
-       jack_nframes_t newlen = drag_info.last_pointer_frame - clicked_regionview->region.position();
-       float percentage = (float) ((double) newlen - (double) clicked_regionview->region.length()) / ((double) newlen) * 100.0f;
+       nframes_t newlen = drag_info.last_pointer_frame - clicked_regionview->region()->position();
+       float percentage = (float) ((double) newlen - (double) clicked_regionview->region()->length()) / ((double) newlen) * 100.0f;
        
        begin_reversible_command (_("timestretch"));
 
-       if (run_timestretch (selection->audio_regions, percentage) == 0) {
+       if (run_timestretch (selection->regions, percentage) == 0) {
                session->commit_reversible_command ();
        }
 }
 
 void
-Editor::mouse_brush_insert_region (AudioRegionView* rv, jack_nframes_t pos)
+Editor::mouse_brush_insert_region (RegionView* rv, nframes_t pos)
 {
        /* no brushing without a useful snap setting */
 
+       // FIXME
+       AudioRegionView* arv = dynamic_cast<AudioRegionView*>(rv);
+       assert(arv);
+
        switch (snap_mode) {
        case SnapMagnetic:
                return; /* can't work because it allows region to be placed anywhere */
@@ -4715,11 +4817,11 @@ Editor::mouse_brush_insert_region (AudioRegionView* rv, jack_nframes_t pos)
 
        /* don't brush a copy over the original */
        
-       if (pos == rv->region.position()) {
+       if (pos == rv->region()->position()) {
                return;
        }
 
-       AudioTimeAxisView* atv = dynamic_cast<AudioTimeAxisView*>(&rv->get_time_axis_view());
+       RouteTimeAxisView* atv = dynamic_cast<RouteTimeAxisView*>(&arv->get_time_axis_view());
 
        if (atv == 0 || !atv->is_audio_track()) {
                return;
@@ -4728,9 +4830,10 @@ Editor::mouse_brush_insert_region (AudioRegionView* rv, jack_nframes_t pos)
        Playlist* playlist = atv->playlist();
        double speed = atv->get_diskstream()->speed();
        
-       session->add_undo (playlist->get_memento());
-       playlist->add_region (*(new AudioRegion (rv->region)), (jack_nframes_t) (pos * speed));
-       session->add_redo_no_execute (playlist->get_memento());
+        XMLNode &before = playlist->get_state();
+       playlist->add_region (boost::dynamic_pointer_cast<AudioRegion> (RegionFactory::create (arv->audio_region())), (nframes_t) (pos * speed));
+        XMLNode &after = playlist->get_state();
+       session->add_command(new MementoCommand<Playlist>(*playlist, &before, &after));
        
        // playlist is frozen, so we have to update manually