Merge with 2.0-ongoing R2943.
[ardour.git] / gtk2_ardour / editor_ops.cc
index d8889c1af7d58c2b3d7244b873f6a8638bdac9ae..33a0e6920c1334bed219f436850083174774abd0 100644 (file)
     along with this program; if not, write to the Free Software
     Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
 
-    $Id$
 */
 
+/* Note: public Editor methods are documented in public_editor.h */
+
 #include <unistd.h>
 
 #include <cstdlib>
 #include <pbd/basename.h>
 #include <pbd/pthread_utils.h>
 #include <pbd/memento_command.h>
+#include <pbd/whitespace.h>
 
 #include <gtkmm2ext/utils.h>
 #include <gtkmm2ext/choice.h>
+#include <gtkmm2ext/window_title.h>
 
 #include <ardour/audioengine.h>
 #include <ardour/session.h>
@@ -46,6 +49,8 @@
 #include <ardour/region_factory.h>
 #include <ardour/playlist_factory.h>
 #include <ardour/reverse.h>
+#include <ardour/dB.h>
+#include <ardour/quantize.h>
 
 #include "ardour_ui.h"
 #include "editor.h"
 #include "automation_time_axis.h"
 #include "streamview.h"
 #include "audio_region_view.h"
+#include "midi_region_view.h"
 #include "rgb_macros.h"
 #include "selection_templates.h"
 #include "selection.h"
 #include "editing.h"
 #include "gtk-custom-hruler.h"
 #include "gui_thread.h"
+#include "keyboard.h"
+#include "utils.h"
 
 #include "i18n.h"
 
@@ -68,6 +76,7 @@ using namespace ARDOUR;
 using namespace PBD;
 using namespace sigc;
 using namespace Gtk;
+using namespace Gtkmm2ext;
 using namespace Editing;
 
 /***********************************************************************
@@ -90,17 +99,10 @@ Editor::redo (uint32_t n)
        }
 }
 
-int
-Editor::ensure_cursor (nframes_t *pos)
-{
-       *pos = edit_cursor->current_frame;
-       return 0;
-}
-
 void
 Editor::split_region ()
 {
-       split_region_at (edit_cursor->current_frame);
+       split_region_at (get_preferred_edit_position());
 }
 
 void
@@ -112,21 +114,61 @@ Editor::split_region_at (nframes_t where)
 void
 Editor::split_regions_at (nframes_t where, RegionSelection& regions)
 {
+       list <boost::shared_ptr<Playlist > > used_playlists;
+
+       if (regions.empty()) {
+               return;
+       }
+
        begin_reversible_command (_("split"));
 
-       snap_to (where);
+       // if splitting a single region, and snap-to is using
+       // region boundaries, don't pay attention to them
+
+       if (regions.size() == 1) {
+               switch (snap_type) {
+               case SnapToRegionStart:
+               case SnapToRegionSync:
+               case SnapToRegionEnd:
+                       break;
+               default:
+                       snap_to (where);
+               }
+       } else {
+               snap_to (where);
+       }
+
        for (RegionSelection::iterator a = regions.begin(); a != regions.end(); ) {
 
                RegionSelection::iterator tmp;
+
+               /* XXX this test needs to be more complicated, to make sure we really
+                  have something to split.
+               */
                
+               if (!(*a)->region()->covers (where)) {
+                       ++a;
+                       continue;
+               }
+
                tmp = a;
                ++tmp;
 
                boost::shared_ptr<Playlist> pl = (*a)->region()->playlist();
 
+               if (! pl->frozen()) {
+                       /* we haven't seen this playlist before */
+
+                       /* remember used playlists so we can thaw them later */ 
+                       used_playlists.push_back(pl);
+                       pl->freeze();
+               }
+
                AudioRegionView* const arv = dynamic_cast<AudioRegionView*>(*a);
-               if (arv)
+
+               if (arv) {
                        _new_regionviews_show_envelope = arv->envelope_visible();
+               }
                
                if (pl) {
                         XMLNode &before = pl->get_state();
@@ -136,12 +178,19 @@ Editor::split_regions_at (nframes_t where, RegionSelection& regions)
                }
 
                a = tmp;
-    }
+       }
+       while (used_playlists.size() > 0) {
 
+               list <boost::shared_ptr<Playlist > >::iterator i = used_playlists.begin();
+               (*i)->thaw();
+               used_playlists.pop_front();
+       }
        commit_reversible_command ();
        _new_regionviews_show_envelope = false;
 }
 
+
+/** Remove `clicked_regionview' */
 void
 Editor::remove_clicked_region ()
 {
@@ -159,47 +208,30 @@ Editor::remove_clicked_region ()
        commit_reversible_command ();
 }
 
+
+/** Remove the selected regions */
 void
-Editor::destroy_clicked_region ()
+Editor::remove_selected_regions ()
 {
-       uint32_t selected = selection->regions.size();
-
-       if (!session || !selected) {
+       if (selection->regions.empty()) {
                return;
        }
 
-       vector<string> choices;
-       string prompt;
+       /* XXX: should be called remove regions if we're removing more than one */
+       begin_reversible_command (_("remove region"));
        
-       prompt  = string_compose (_(" This is destructive, will possibly delete audio files\n\
-It cannot be undone\n\
-Do you really want to destroy %1 ?"),
-                          (selected > 1 ? 
-                           _("these regions") : _("this region")));
-
-       choices.push_back (_("No, do nothing."));
 
-       if (selected > 1) {
-               choices.push_back (_("Yes, destroy them."));
-       } else {
-               choices.push_back (_("Yes, destroy it."));
-       }
+       while (!selection->regions.empty()) {
+               boost::shared_ptr<Region> region = selection->regions.front()->region ();
+               boost::shared_ptr<Playlist> playlist = region->playlist ();
 
-       Gtkmm2ext::Choice prompter (prompt, choices);
-       
-       if (prompter.run() == 0) { /* first choice */
-               return;
+               XMLNode &before = playlist->get_state();
+               playlist->remove_region (region);
+               XMLNode &after = playlist->get_state();
+               session->add_command(new MementoCommand<Playlist>(*playlist, &before, &after));
        }
 
-       if (selected) {
-               list<boost::shared_ptr<Region> > r;
-
-               for (RegionSelection::iterator i = selection->regions.begin(); i != selection->regions.end(); ++i) {
-                       r.push_back ((*i)->region());
-               }
-
-               session->destroy_regions (r);
-       } 
+       commit_reversible_command ();
 }
 
 boost::shared_ptr<Region>
@@ -301,18 +333,40 @@ Editor::extend_selection_to_start_of_region (bool previous)
        commit_reversible_command ();
 }
 
+bool
+Editor::nudge_forward_release (GdkEventButton* ev)
+{
+       if (ev->state & Keyboard::PrimaryModifier) {
+               nudge_forward (false, true);
+       } else {
+               nudge_forward (false, false);
+       }
+       return false;
+}
+
+bool
+Editor::nudge_backward_release (GdkEventButton* ev)
+{
+       if (ev->state & Keyboard::PrimaryModifier) {
+               nudge_backward (false, true);
+       } else {
+               nudge_backward (false, false);
+       }
+       return false;
+}
+
 
 void
-Editor::nudge_forward (bool next)
+Editor::nudge_forward (bool next, bool force_playhead)
 {
        nframes_t distance;
        nframes_t next_distance;
 
        if (!session) return;
        
-       if (!selection->regions.empty()) {
+       if (!force_playhead && !selection->regions.empty()) {
 
-               begin_reversible_command (_("nudge forward"));
+               begin_reversible_command (_("nudge regions forward"));
 
                for (RegionSelection::iterator i = selection->regions.begin(); i != selection->regions.end(); ++i) {
                        boost::shared_ptr<Region> r ((*i)->region());
@@ -331,6 +385,44 @@ Editor::nudge_forward (bool next)
 
                commit_reversible_command ();
 
+               
+       } else if (!force_playhead && !selection->markers.empty()) {
+
+               bool is_start;
+               Location* loc = find_location_from_marker (selection->markers.front(), is_start);
+
+               if (loc) {
+
+                       begin_reversible_command (_("nudge location forward"));
+
+                       XMLNode& before (loc->get_state());
+
+                       if (is_start) {
+                               distance = get_nudge_distance (loc->start(), next_distance);
+                               if (next) {
+                                       distance = next_distance;
+                               }
+                               if (max_frames - distance > loc->start() + loc->length()) {
+                                       loc->set_start (loc->start() + distance);
+                               } else {
+                                       loc->set_start (max_frames - loc->length());
+                               }
+                       } else {
+                               distance = get_nudge_distance (loc->end(), next_distance);
+                               if (next) {
+                                       distance = next_distance;
+                               }
+                               if (max_frames - distance > loc->end()) {
+                                       loc->set_end (loc->end() + distance);
+                               } else {
+                                       loc->set_end (max_frames);
+                               }
+                       }
+                       XMLNode& after (loc->get_state());
+                       session->add_command (new MementoCommand<Location>(*loc, &before, &after));
+                       commit_reversible_command ();
+               }
+               
        } else {
                distance = get_nudge_distance (playhead_cursor->current_frame, next_distance);
                session->request_locate (playhead_cursor->current_frame + distance);
@@ -338,16 +430,16 @@ Editor::nudge_forward (bool next)
 }
                
 void
-Editor::nudge_backward (bool next)
+Editor::nudge_backward (bool next, bool force_playhead)
 {
        nframes_t distance;
        nframes_t next_distance;
 
        if (!session) return;
        
-       if (!selection->regions.empty()) {
+       if (!force_playhead && !selection->regions.empty()) {
 
-               begin_reversible_command (_("nudge forward"));
+               begin_reversible_command (_("nudge regions backward"));
 
                for (RegionSelection::iterator i = selection->regions.begin(); i != selection->regions.end(); ++i) {
                        boost::shared_ptr<Region> r ((*i)->region());
@@ -371,6 +463,44 @@ Editor::nudge_backward (bool next)
 
                commit_reversible_command ();
 
+       } else if (!force_playhead && !selection->markers.empty()) {
+
+               bool is_start;
+               Location* loc = find_location_from_marker (selection->markers.front(), is_start);
+
+               if (loc) {
+
+                       begin_reversible_command (_("nudge location forward"));
+                       XMLNode& before (loc->get_state());
+
+                       if (is_start) {
+                               distance = get_nudge_distance (loc->start(), next_distance);
+                               if (next) {
+                                       distance = next_distance;
+                               }
+                               if (distance < loc->start()) {
+                                       loc->set_start (loc->start() - distance);
+                               } else {
+                                       loc->set_start (0);
+                               }
+                       } else {
+                               distance = get_nudge_distance (loc->end(), next_distance);
+
+                               if (next) {
+                                       distance = next_distance;
+                               }
+
+                               if (distance < loc->end() - loc->length()) {
+                                       loc->set_end (loc->end() - distance);
+                               } else {
+                                       loc->set_end (loc->length());
+                               }
+                       }
+
+                       XMLNode& after (loc->get_state());
+                       session->add_command (new MementoCommand<Location>(*loc, &before, &after));
+               }
+               
        } else {
 
                distance = get_nudge_distance (playhead_cursor->current_frame, next_distance);
@@ -460,9 +590,10 @@ void
 Editor::build_region_boundary_cache ()
 {
        nframes_t pos = 0;
-       RegionPoint point;
+       vector<RegionPoint> interesting_points;
        boost::shared_ptr<Region> r;
        TrackViewList tracks;
+       bool at_end = false;
 
        region_boundary_cache.clear ();
 
@@ -472,16 +603,17 @@ Editor::build_region_boundary_cache ()
        
        switch (snap_type) {
        case SnapToRegionStart:
-               point = Start;
+               interesting_points.push_back (Start);
                break;
        case SnapToRegionEnd:
-               point = End;
+               interesting_points.push_back (End);
                break;  
        case SnapToRegionSync:
-               point = SyncPoint;
+               interesting_points.push_back (SyncPoint);
                break;  
        case SnapToRegionBoundary:
-               point = Start;
+               interesting_points.push_back (Start);
+               interesting_points.push_back (End);
                break;  
        default:
                fatal << string_compose (_("build_region_boundary_cache called with snap_type = %1"), snap_type) << endmsg;
@@ -490,71 +622,84 @@ Editor::build_region_boundary_cache ()
        }
        
        TimeAxisView *ontrack = 0;
+       TrackViewList tlist;
 
-       while (pos < session->current_end_frame()) {
+       if (!selection->tracks.empty()) {
+               tlist = selection->tracks;
+       } else {
+               tlist = track_views;
+       }
 
-               if (!selection->tracks.empty()) {
+       while (pos < session->current_end_frame() && !at_end) {
 
-                       if ((r = find_next_region (pos, point, 1, selection->tracks, &ontrack)) == 0) {
-                               break;
-                       }
+               nframes_t rpos;
+               nframes_t lpos = max_frames;
 
-               } else if (clicked_axisview) {
+               for (vector<RegionPoint>::iterator p = interesting_points.begin(); p != interesting_points.end(); ++p) {
 
-                       TrackViewList t;
-                       t.push_back (clicked_axisview);
+                       if ((r = find_next_region (pos, *p, 1, tlist, &ontrack)) == 0) {
+                               if (*p == interesting_points.back()) {
+                                       at_end = true;
+                               }
+                               /* move to next point type */
+                               continue;
+                       }
 
-                       if ((r = find_next_region (pos, point, 1, t, &ontrack)) == 0) {
+                       switch (*p) {
+                       case Start:
+                               rpos = r->first_frame();
                                break;
-                       }
 
-               } else {
+                       case End:
+                               rpos = r->last_frame();
+                               break;  
 
-                       if ((r = find_next_region (pos, point, 1, track_views, &ontrack)) == 0) {
+                       case SyncPoint:
+                               rpos = r->adjust_to_sync (r->first_frame());
+                               break;
+
+                       default:
                                break;
                        }
-               }
+                       
+                       float speed = 1.0f;
+                       RouteTimeAxisView *rtav;
+                       
+                       if (ontrack != 0 && (rtav = dynamic_cast<RouteTimeAxisView*>(ontrack)) != 0 ) {
+                               if (rtav->get_diskstream() != 0) {
+                                       speed = rtav->get_diskstream()->speed();
+                               }
+                       }
+                       
+                       rpos = track_frame_to_session_frame (rpos, speed);
 
-               nframes_t rpos;
-               
-               switch (snap_type) {
-               case SnapToRegionStart:
-                       rpos = r->first_frame();
-                       break;
-               case SnapToRegionEnd:
-                       rpos = r->last_frame();
-                       break;  
-               case SnapToRegionSync:
-                       rpos = r->adjust_to_sync (r->first_frame());
-                       break;
+                       if (rpos < lpos) {
+                               lpos = rpos;
+                       }
 
-               case SnapToRegionBoundary:
-                       rpos = r->last_frame();
-                       break;  
-               default:
-                       break;
-               }
-               
-               float speed = 1.0f;
-               AudioTimeAxisView *atav;
+                       /* prevent duplicates, but we don't use set<> because we want to be able
+                          to sort later.
+                       */
 
-               if ( ontrack != 0 && (atav = dynamic_cast<AudioTimeAxisView*>(ontrack)) != 0 ) {
-                       if (atav->get_diskstream() != 0) {
-                               speed = atav->get_diskstream()->speed();
+                       vector<nframes_t>::iterator ri; 
+                       
+                       for (ri = region_boundary_cache.begin(); ri != region_boundary_cache.end(); ++ri) {
+                               if (*ri == rpos) {
+                                       break;
+                               }
                        }
-               }
 
-               rpos = track_frame_to_session_frame(rpos, speed);
-
-               if (region_boundary_cache.empty() || rpos != region_boundary_cache.back()) {
-                       if (snap_type == SnapToRegionBoundary) {
-                               region_boundary_cache.push_back (r->first_frame());
+                       if (ri == region_boundary_cache.end()) {
+                               region_boundary_cache.push_back (rpos);
                        }
-                       region_boundary_cache.push_back (rpos);
                }
 
-               pos = rpos + 1;
+               pos = lpos + 1;
        }
+
+       /* finally sort to be sure that the order is correct */
+
+       sort (region_boundary_cache.begin(), region_boundary_cache.end());
 }
 
 boost::shared_ptr<Region>
@@ -567,7 +712,7 @@ Editor::find_next_region (nframes_t frame, RegionPoint point, int32_t dir, Track
 
        float track_speed;
        nframes_t track_frame;
-       AudioTimeAxisView *atav;
+       RouteTimeAxisView *rtav;
 
        for (i = tracks.begin(); i != tracks.end(); ++i) {
 
@@ -575,9 +720,9 @@ Editor::find_next_region (nframes_t frame, RegionPoint point, int32_t dir, Track
                boost::shared_ptr<Region> r;
                
                track_speed = 1.0f;
-               if ( (atav = dynamic_cast<AudioTimeAxisView*>(*i)) != 0 ) {
-                       if (atav->get_diskstream()!=0)
-                               track_speed = atav->get_diskstream()->speed();
+               if ( (rtav = dynamic_cast<RouteTimeAxisView*>(*i)) != 0 ) {
+                       if (rtav->get_diskstream()!=0)
+                               track_speed = rtav->get_diskstream()->speed();
                }
 
                track_frame = session_frame_to_track_frame(frame, track_speed);
@@ -599,6 +744,7 @@ Editor::find_next_region (nframes_t frame, RegionPoint point, int32_t dir, Track
                        rpos = r->adjust_to_sync (r->first_frame());
                        break;
                }
+
                // rpos is a "track frame", converting it to "session frame"
                rpos = track_frame_to_session_frame(rpos, track_speed);
 
@@ -619,6 +765,85 @@ Editor::find_next_region (nframes_t frame, RegionPoint point, int32_t dir, Track
        return ret;
 }
 
+nframes64_t
+Editor::find_next_region_boundary (nframes64_t pos, int32_t dir, const TrackViewList& tracks)
+{
+       nframes64_t distance = max_frames;
+       nframes64_t current_nearest = -1;
+
+       for (TrackViewList::const_iterator i = tracks.begin(); i != tracks.end(); ++i) {
+               nframes64_t contender;
+               nframes64_t d;
+
+               RouteTimeAxisView* rtv = dynamic_cast<RouteTimeAxisView*> (*i);
+
+               if (!rtv) {
+                       continue;
+               }
+
+               if ((contender = rtv->find_next_region_boundary (pos, dir)) < 0) {
+                       continue;
+               }
+
+               d = ::llabs (pos - contender);
+
+               if (d < distance) {
+                       current_nearest = contender;
+                       distance = d;
+               }
+       }
+       
+       return current_nearest;
+}
+
+void
+Editor::cursor_to_region_boundary (Cursor* cursor, int32_t dir)
+{
+       nframes64_t pos = cursor->current_frame;
+       nframes64_t target;
+
+       if (!session) {
+               return;
+       }
+
+       // so we don't find the current region again..
+       if (dir > 0 || pos > 0) {
+               pos += dir;
+       }
+
+       if (!selection->tracks.empty()) {
+               
+               target = find_next_region_boundary (pos, dir, selection->tracks);
+               
+       } else {
+               
+               target = find_next_region_boundary (pos, dir, track_views);
+       }
+       
+       if (target < 0) {
+               return;
+       }
+
+
+       if (cursor == playhead_cursor) {
+               session->request_locate (target);
+       } else {
+               cursor->set_position (target);
+       }
+}
+
+void
+Editor::cursor_to_next_region_boundary (Cursor* cursor)
+{
+       cursor_to_region_boundary (cursor, 1);
+}
+
+void
+Editor::cursor_to_previous_region_boundary (Cursor* cursor)
+{
+       cursor_to_region_boundary (cursor, -1);
+}
+
 void
 Editor::cursor_to_region_point (Cursor* cursor, RegionPoint point, int32_t dir)
 {
@@ -670,11 +895,11 @@ Editor::cursor_to_region_point (Cursor* cursor, RegionPoint point, int32_t dir)
        }
        
        float speed = 1.0f;
-       AudioTimeAxisView *atav;
+       RouteTimeAxisView *rtav;
 
-       if ( ontrack != 0 && (atav = dynamic_cast<AudioTimeAxisView*>(ontrack)) != 0 ) {
-               if (atav->get_diskstream() != 0) {
-                       speed = atav->get_diskstream()->speed();
+       if ( ontrack != 0 && (rtav = dynamic_cast<RouteTimeAxisView*>(ontrack)) != 0 ) {
+               if (rtav->get_diskstream() != 0) {
+                       speed = rtav->get_diskstream()->speed();
                }
        }
 
@@ -757,58 +982,299 @@ Editor::cursor_to_selection_end (Cursor *cursor)
 }
 
 void
-Editor::playhead_backward ()
+Editor::selected_marker_to_region_boundary (int32_t dir)
 {
-       nframes_t pos;
-       nframes_t cnt;
-       float prefix;
-       bool was_floating;
+       nframes64_t target;
+       Location* loc;
+       bool ignored;
 
-       if (get_prefix (prefix, was_floating)) {
-               cnt = 1;
-       } else {
-               if (was_floating) {
-                       cnt = (nframes_t) floor (prefix * session->frame_rate ());
-               } else {
-                       cnt = (nframes_t) prefix;
+       if (!session) {
+               return;
+       }
+
+       if (selection->markers.empty()) {
+               nframes64_t mouse;
+               bool ignored;
+
+               if (!mouse_frame (mouse, ignored)) {
+                       return;
                }
+               
+               add_location_mark (mouse);
        }
 
-       pos = playhead_cursor->current_frame;
+       if ((loc = find_location_from_marker (selection->markers.front(), ignored)) == 0) {
+               return;
+       }
 
-       if ((nframes_t) pos < cnt) {
-               pos = 0;
+       nframes64_t pos = loc->start();
+
+       // so we don't find the current region again..
+       if (dir > 0 || pos > 0) {
+               pos += dir;
+       }
+
+       if (!selection->tracks.empty()) {
+               
+               target = find_next_region_boundary (pos, dir, selection->tracks);
+               
        } else {
-               pos -= cnt;
+               
+               target = find_next_region_boundary (pos, dir, track_views);
        }
        
-       /* XXX this is completely insane. with the current buffering
-          design, we'll force a complete track buffer flush and
-          reload, just to move 1 sample !!!
-       */
+       if (target < 0) {
+               return;
+       }
 
-       session->request_locate (pos);
+       loc->move_to (target);
 }
 
 void
-Editor::playhead_forward ()
+Editor::selected_marker_to_next_region_boundary ()
 {
-       nframes_t pos;
-       nframes_t cnt;
-       bool was_floating;
-       float prefix;
+       selected_marker_to_region_boundary (1);
+}
 
-       if (get_prefix (prefix, was_floating)) {
-               cnt = 1;
-       } else {
-               if (was_floating) {
-                       cnt = (nframes_t) floor (prefix * session->frame_rate ());
-               } else {
-                       cnt = (nframes_t) floor (prefix);
-               }
-       }
+void
+Editor::selected_marker_to_previous_region_boundary ()
+{
+       selected_marker_to_region_boundary (-1);
+}
 
-       pos = playhead_cursor->current_frame;
+void
+Editor::selected_marker_to_region_point (RegionPoint point, int32_t dir)
+{
+       boost::shared_ptr<Region> r;
+       nframes_t pos;
+       Location* loc;
+       bool ignored;
+
+       if (!session || selection->markers.empty()) {
+               return;
+       }
+
+       if ((loc = find_location_from_marker (selection->markers.front(), ignored)) == 0) {
+               return;
+       }
+
+       TimeAxisView *ontrack = 0;
+
+       pos = loc->start();
+
+       // so we don't find the current region again..
+       if (dir>0 || pos>0)
+               pos+=dir;
+
+       if (!selection->tracks.empty()) {
+               
+               r = find_next_region (pos, point, dir, selection->tracks, &ontrack);
+               
+       } else {
+               
+               r = find_next_region (pos, point, dir, track_views, &ontrack);
+       }
+
+       if (r == 0) {
+               return;
+       }
+       
+       switch (point){
+       case Start:
+               pos = r->first_frame ();
+               break;
+
+       case End:
+               pos = r->last_frame ();
+               break;
+
+       case SyncPoint:
+               pos = r->adjust_to_sync (r->first_frame());
+               break;  
+       }
+       
+       float speed = 1.0f;
+       AudioTimeAxisView *atav;
+
+       if ( ontrack != 0 && (atav = dynamic_cast<AudioTimeAxisView*>(ontrack)) != 0 ) {
+               if (atav->get_diskstream() != 0) {
+                       speed = atav->get_diskstream()->speed();
+               }
+       }
+
+       pos = track_frame_to_session_frame(pos, speed);
+
+       loc->move_to (pos);
+}
+
+void
+Editor::selected_marker_to_next_region_point (RegionPoint point)
+{
+       selected_marker_to_region_point (point, 1);
+}
+
+void
+Editor::selected_marker_to_previous_region_point (RegionPoint point)
+{
+       selected_marker_to_region_point (point, -1);
+}
+
+void
+Editor::selected_marker_to_selection_start ()
+{
+       nframes_t pos = 0;
+       Location* loc;
+       bool ignored;
+
+       if (!session || selection->markers.empty()) {
+               return;
+       }
+
+       if ((loc = find_location_from_marker (selection->markers.front(), ignored)) == 0) {
+               return;
+       }
+
+       switch (mouse_mode) {
+       case MouseObject:
+               if (!selection->regions.empty()) {
+                       pos = selection->regions.start();
+               }
+               break;
+
+       case MouseRange:
+               if (!selection->time.empty()) {
+                       pos = selection->time.start ();
+               }
+               break;
+
+       default:
+               return;
+       }
+
+       loc->move_to (pos);
+}
+
+void
+Editor::selected_marker_to_selection_end ()
+{
+       nframes_t pos = 0;
+       Location* loc;
+       bool ignored;
+
+       if (!session || selection->markers.empty()) {
+               return;
+       }
+
+       if ((loc = find_location_from_marker (selection->markers.front(), ignored)) == 0) {
+               return;
+       }
+
+       switch (mouse_mode) {
+       case MouseObject:
+               if (!selection->regions.empty()) {
+                       pos = selection->regions.end_frame();
+               }
+               break;
+
+       case MouseRange:
+               if (!selection->time.empty()) {
+                       pos = selection->time.end_frame ();
+               }
+               break;
+
+       default:
+               return;
+       }
+
+       loc->move_to (pos);
+}
+
+void
+Editor::scroll_playhead (bool forward)
+{
+       nframes_t pos = playhead_cursor->current_frame;
+       nframes_t delta = (nframes_t) floor (current_page_frames() / 0.8);
+
+       if (forward) {
+               if (pos == max_frames) {
+                       return;
+               }
+
+               if (pos < max_frames - delta) {
+                       pos += delta ;
+               } else {
+                       pos = max_frames;
+               } 
+
+       } else {
+
+               if (pos == 0) {
+                       return;
+               } 
+
+               if (pos > delta) {
+                       pos -= delta;
+               } else {
+                       pos = 0;
+               }
+       }
+
+       session->request_locate (pos);
+}
+
+void
+Editor::playhead_backward ()
+{
+       nframes_t pos;
+       nframes_t cnt;
+       float prefix;
+       bool was_floating;
+
+       if (get_prefix (prefix, was_floating)) {
+               cnt = 1;
+       } else {
+               if (was_floating) {
+                       cnt = (nframes_t) floor (prefix * session->frame_rate ());
+               } else {
+                       cnt = (nframes_t) prefix;
+               }
+       }
+
+       pos = playhead_cursor->current_frame;
+
+       if ((nframes_t) pos < cnt) {
+               pos = 0;
+       } else {
+               pos -= cnt;
+       }
+       
+       /* XXX this is completely insane. with the current buffering
+          design, we'll force a complete track buffer flush and
+          reload, just to move 1 sample !!!
+       */
+
+       session->request_locate (pos);
+}
+
+void
+Editor::playhead_forward ()
+{
+       nframes_t pos;
+       nframes_t cnt;
+       bool was_floating;
+       float prefix;
+
+       if (get_prefix (prefix, was_floating)) {
+               cnt = 1;
+       } else {
+               if (was_floating) {
+                       cnt = (nframes_t) floor (prefix * session->frame_rate ());
+               } else {
+                       cnt = (nframes_t) floor (prefix);
+               }
+       }
+
+       pos = playhead_cursor->current_frame;
        
        /* XXX this is completely insane. with the current buffering
           design, we'll force a complete track buffer flush and
@@ -821,20 +1287,41 @@ Editor::playhead_forward ()
 void
 Editor::cursor_align (bool playhead_to_edit)
 {
+       if (!session) {
+               return;
+       }
+
        if (playhead_to_edit) {
-               if (session) {
-                       session->request_locate (edit_cursor->current_frame);
+
+               if (selection->markers.empty()) {
+                       return;
                }
+               
+               session->request_locate (selection->markers.front()->position(), session->transport_rolling());
+       
        } else {
-               edit_cursor->set_position (playhead_cursor->current_frame);
+               /* move selected markers to playhead */
+               
+               for (MarkerSelection::iterator i = selection->markers.begin(); i != selection->markers.end(); ++i) {
+                       bool ignored;
+                       
+                       Location* loc = find_location_from_marker (*i, ignored);
+                       
+                       if (loc->is_mark()) {
+                               loc->set_start (playhead_cursor->current_frame);
+                       } else {
+                               loc->set (playhead_cursor->current_frame,
+                                         playhead_cursor->current_frame + loc->length());
+                       }
+               }
        }
 }
 
 void
 Editor::edit_cursor_backward ()
 {
-       nframes_t pos;
-       nframes_t cnt;
+       nframes64_t pos;
+       nframes64_t cnt;
        float prefix;
        bool was_floating;
 
@@ -848,21 +1335,23 @@ Editor::edit_cursor_backward ()
                }
        }
 
-       pos = edit_cursor->current_frame;
+       if ((pos = get_preferred_edit_position()) < 0) {
+               return;
+       }
 
-       if ((nframes_t) pos < cnt) {
+       if (pos < cnt) {
                pos = 0;
        } else {
                pos -= cnt;
        }
        
-       edit_cursor->set_position (pos);
+       // EDIT CURSOR edit_cursor->set_position (pos);
 }
 
 void
 Editor::edit_cursor_forward ()
 {
-       nframes_t pos;
+       //nframes_t pos;
        nframes_t cnt;
        bool was_floating;
        float prefix;
@@ -877,8 +1366,8 @@ Editor::edit_cursor_forward ()
                }
        }
 
-       pos = edit_cursor->current_frame;
-       edit_cursor->set_position (pos+cnt);
+       // pos = edit_cursor->current_frame;
+       // EDIT CURSOR edit_cursor->set_position (pos+cnt);
 }
 
 void
@@ -926,7 +1415,7 @@ Editor::scroll_backward (float pages)
                frame = leftmost_frame - cnt;
        }
 
-       reposition_x_origin (frame);
+       reset_x_origin (frame);
 }
 
 void
@@ -954,7 +1443,7 @@ Editor::scroll_forward (float pages)
                frame = leftmost_frame + cnt;
        }
 
-       reposition_x_origin (frame);
+       reset_x_origin (frame);
 }
 
 void
@@ -1039,17 +1528,28 @@ Editor::temporal_zoom (gdouble fpu)
 {
        if (!session) return;
        
-       nframes_t current_page = current_page_frames();
-       nframes_t current_leftmost = leftmost_frame;
-       nframes_t current_rightmost;
-       nframes_t current_center;
-       nframes_t new_page;
-       nframes_t leftmost_after_zoom = 0;
+       nframes64_t current_page = current_page_frames();
+       nframes64_t current_leftmost = leftmost_frame;
+       nframes64_t current_rightmost;
+       nframes64_t current_center;
+       nframes64_t new_page_size;
+       nframes64_t half_page_size;
+       nframes64_t leftmost_after_zoom = 0;
+       nframes64_t where;
+       bool in_track_canvas;
        double nfpu;
+       double l;
+
+       /* XXX this limit is also in ::set_frames_per_unit() */
+
+       if (frames_per_unit <= 2.0 && fpu <= frames_per_unit) {
+               return;
+       }
 
        nfpu = fpu;
        
-       new_page = (nframes_t) floor (canvas_width * nfpu);
+       new_page_size = (nframes_t) floor (canvas_width * nfpu);
+       half_page_size = new_page_size / 2;
 
        switch (zoom_focus) {
        case ZoomFocusLeft:
@@ -1058,37 +1558,85 @@ Editor::temporal_zoom (gdouble fpu)
                
        case ZoomFocusRight:
                current_rightmost = leftmost_frame + current_page;
-               if (current_rightmost > new_page) {
-                       leftmost_after_zoom = current_rightmost - new_page;
-               } else {
+               if (current_rightmost < new_page_size) {
                        leftmost_after_zoom = 0;
+               } else {
+                       leftmost_after_zoom = current_rightmost - new_page_size;
                }
                break;
                
        case ZoomFocusCenter:
                current_center = current_leftmost + (current_page/2); 
-               if (current_center > (new_page/2)) {
-                       leftmost_after_zoom = current_center - (new_page / 2);
-               } else {
+               if (current_center < half_page_size) {
                        leftmost_after_zoom = 0;
+               } else {
+                       leftmost_after_zoom = current_center - half_page_size;
                }
                break;
                
        case ZoomFocusPlayhead:
-               /* try to keep the playhead in the center */
-               if (playhead_cursor->current_frame > new_page/2) {
-                       leftmost_after_zoom = playhead_cursor->current_frame - (new_page/2);
-               } else {
+               /* try to keep the playhead in the same place */
+
+               where = playhead_cursor->current_frame;
+               
+               l = - ((new_page_size * ((where - current_leftmost)/(double)current_page)) - where);
+
+               if (l < 0) {
                        leftmost_after_zoom = 0;
+               } else if (l > max_frames) { 
+                       leftmost_after_zoom = max_frames - new_page_size;
+               } else {
+                       leftmost_after_zoom = (nframes64_t) l;
+               }
+               break;
+
+       case ZoomFocusMouse:
+               /* try to keep the mouse over the same point in the display */
+
+               if (!mouse_frame (where, in_track_canvas)) {
+                       /* use playhead instead */
+                       where = playhead_cursor->current_frame;
+
+                       if (where < half_page_size) {
+                               leftmost_after_zoom = 0;
+                       } else {
+                               leftmost_after_zoom = where - half_page_size;
+                       }
+
+               } else {
+
+                       l = - ((new_page_size * ((where - current_leftmost)/(double)current_page)) - where);
+
+                       if (l < 0) {
+                               leftmost_after_zoom = 0;
+                       } else if (l > max_frames) { 
+                               leftmost_after_zoom = max_frames - new_page_size;
+                       } else {
+                               leftmost_after_zoom = (nframes64_t) l;
+                       }
                }
+
                break;
 
        case ZoomFocusEdit:
-               /* try to keep the edit cursor in the center */
-               if (edit_cursor->current_frame > new_page/2) {
-                       leftmost_after_zoom = edit_cursor->current_frame - (new_page/2);
+               /* try to keep the edit point in the same place */
+               where = get_preferred_edit_position ();
+
+               if (where > 0) {
+
+                       double l = - ((new_page_size * ((where - current_leftmost)/(double)current_page)) - where);
+
+                       if (l < 0) {
+                               leftmost_after_zoom = 0;
+                       } else if (l > max_frames) { 
+                               leftmost_after_zoom = max_frames - new_page_size;
+                       } else {
+                               leftmost_after_zoom = (nframes64_t) l;
+                       }
+
                } else {
-                       leftmost_after_zoom = 0;
+                       /* edit point not defined */
+                       return;
                }
                break;
                
@@ -1096,14 +1644,70 @@ Editor::temporal_zoom (gdouble fpu)
  
        // leftmost_after_zoom = min (leftmost_after_zoom, session->current_end_frame());
 
-//     begin_reversible_command (_("zoom"));
-//     session->add_undo (bind (mem_fun(*this, &Editor::reposition_and_zoom), current_leftmost, frames_per_unit));
-//     session->add_redo (bind (mem_fun(*this, &Editor::reposition_and_zoom), leftmost_after_zoom, nfpu));
-//     commit_reversible_command ();
-
        reposition_and_zoom (leftmost_after_zoom, nfpu);
 }      
 
+void
+Editor::temporal_zoom_region ()
+{
+
+       nframes64_t start = max_frames;
+       nframes64_t end = 0;
+
+       ensure_entered_region_selected (true);
+
+       if (selection->regions.empty()) {
+               info << _("cannot set loop: no region selected") << endmsg;
+               return;
+       }
+
+       for (RegionSelection::iterator i = selection->regions.begin(); i != selection->regions.end(); ++i) {
+               if ((*i)->region()->position() < start) {
+                       start = (*i)->region()->position();
+               }
+               if ((*i)->region()->last_frame() + 1 > end) {
+                       end = (*i)->region()->last_frame() + 1;
+               }
+       }
+
+       /* now comes an "interesting" hack ... make sure we leave a little space
+          at each end of the editor so that the zoom doesn't fit the region
+          precisely to the screen.
+       */
+
+       GdkScreen* screen = gdk_screen_get_default ();
+       gint pixwidth = gdk_screen_get_width (screen);
+       gint mmwidth = gdk_screen_get_width_mm (screen);
+       double pix_per_mm = (double) pixwidth/ (double) mmwidth;
+       double one_centimeter_in_pixels = pix_per_mm * 10.0;
+       nframes_t extra_samples = unit_to_frame (one_centimeter_in_pixels);
+       
+       if (start > extra_samples) {
+               start -= extra_samples;
+       } else {
+               start = 0;
+       } 
+
+       if (max_frames - extra_samples > end) {
+               end += extra_samples;
+       } else {
+               end = max_frames;
+       }
+
+       temporal_zoom_by_frame (start, end, "zoom to region");
+       zoomed_to_region = true;
+}
+
+void
+Editor::toggle_zoom_region ()
+{
+       if (zoomed_to_region) {
+               swap_visual_state ();
+       } else {
+               temporal_zoom_region ();
+       }
+}
+
 void
 Editor::temporal_zoom_selection ()
 {
@@ -1222,12 +1826,12 @@ Editor::add_location_from_selection ()
 }
 
 void
-Editor::add_location_from_playhead_cursor ()
+Editor::add_location_mark (nframes64_t where)
 {
        string markername;
 
-       nframes_t where = session->audible_frame();
-       
+       select_new_marker = true;
+
        session->locations()->next_available_name(markername,"mark");
        Location *location = new Location (where, where, markername, Location::IsMark);
        session->begin_reversible_command (_("add marker"));
@@ -1238,6 +1842,12 @@ Editor::add_location_from_playhead_cursor ()
        session->commit_reversible_command ();
 }
 
+void
+Editor::add_location_from_playhead_cursor ()
+{
+       add_location_mark (session->audible_frame());
+}
+
 void
 Editor::add_location_from_audio_region ()
 {
@@ -1257,311 +1867,6 @@ Editor::add_location_from_audio_region ()
        session->commit_reversible_command ();
 }
 
-void
-Editor::select_all_in_track (Selection::Operation op)
-{
-       list<Selectable *> touched;
-
-       if (!clicked_axisview) {
-               return;
-       }
-       
-       clicked_axisview->get_selectables (0, max_frames, 0, DBL_MAX, touched);
-
-       switch (op) {
-       case Selection::Toggle:
-               selection->add (touched);
-               break;
-       case Selection::Set:
-               selection->set (touched);
-               break;
-       case Selection::Extend:
-               /* not defined yet */
-               break;
-       case Selection::Add:
-               selection->add (touched);
-               break;
-       }
-}
-
-void
-Editor::select_all (Selection::Operation op)
-{
-       list<Selectable *> touched;
-       
-       for (TrackViewList::iterator iter = track_views.begin(); iter != track_views.end(); ++iter) {
-               if ((*iter)->hidden()) {
-                       continue;
-               }
-               (*iter)->get_selectables (0, max_frames, 0, DBL_MAX, touched);
-       }
-       begin_reversible_command (_("select all"));
-       switch (op) {
-       case Selection::Add:
-       case Selection::Toggle:
-               selection->add (touched);
-               break;
-       case Selection::Set:
-               selection->set (touched);
-               break;
-       case Selection::Extend:
-               /* not defined yet */
-               break;
-       }
-       commit_reversible_command ();
-}
-
-void
-Editor::invert_selection_in_track ()
-{
-       list<Selectable *> touched;
-
-       if (!clicked_axisview) {
-               return;
-       }
-       
-       clicked_axisview->get_inverted_selectables (*selection, touched);
-       selection->set (touched);
-}
-
-void
-Editor::invert_selection ()
-{
-       list<Selectable *> touched;
-       
-       for (TrackViewList::iterator iter = track_views.begin(); iter != track_views.end(); ++iter) {
-               if ((*iter)->hidden()) {
-                       continue;
-               }
-               (*iter)->get_inverted_selectables (*selection, touched);
-       }
-
-       selection->set (touched);
-}
-
-bool
-Editor::select_all_within (nframes_t start, nframes_t end, double top, double bot, Selection::Operation op)
-{
-       list<Selectable *> touched;
-       
-       for (TrackViewList::iterator iter = track_views.begin(); iter != track_views.end(); ++iter) {
-               if ((*iter)->hidden()) {
-                       continue;
-               }
-               (*iter)->get_selectables (start, end, top, bot, touched);
-       }
-
-       cerr << "select all within found " << touched.size() << endl;
-
-       begin_reversible_command (_("select all within"));
-       switch (op) {
-       case Selection::Add:
-       case Selection::Toggle:
-               cerr << "toggle\n";
-               selection->add (touched);
-               break;
-       case Selection::Set:
-               cerr << "set\n";
-               selection->set (touched);
-               break;
-       case Selection::Extend:
-               cerr << "extend\n";
-               /* not defined yet */
-               break;
-       }
-
-       cerr << "selection now has " << selection->points.size() << endl;
-
-       commit_reversible_command ();
-       return !touched.empty();
-}
-
-void
-Editor::set_selection_from_audio_region ()
-{
-       if (selection->regions.empty()) {
-               return;
-       }
-
-       RegionView* rv = *(selection->regions.begin());
-       boost::shared_ptr<Region> region = rv->region();
-       
-       begin_reversible_command (_("set selection from region"));
-       selection->set (0, region->position(), region->last_frame());
-       commit_reversible_command ();
-
-       set_mouse_mode (Editing::MouseRange, false);
-}
-
-void
-Editor::set_selection_from_punch()
-{
-       Location* location;
-
-       if ((location = session->locations()->auto_punch_location()) == 0)  {
-               return;
-       }
-
-       set_selection_from_range (*location);
-}
-
-void
-Editor::set_selection_from_loop()
-{
-       Location* location;
-
-       if ((location = session->locations()->auto_loop_location()) == 0)  {
-               return;
-       }
-       set_selection_from_range (*location);
-}
-
-void
-Editor::set_selection_from_range (Location& loc)
-{
-       begin_reversible_command (_("set selection from range"));
-       selection->set (0, loc.start(), loc.end());
-       commit_reversible_command ();
-
-       set_mouse_mode (Editing::MouseRange, false);
-}
-
-void
-Editor::select_all_selectables_using_time_selection ()
-{
-       list<Selectable *> touched;
-
-       if (selection->time.empty()) {
-               return;
-       }
-
-       nframes_t start = selection->time[clicked_selection].start;
-       nframes_t end = selection->time[clicked_selection].end;
-
-       if (end - start < 1)  {
-               return;
-       }
-
-       for (TrackViewList::iterator iter = selection->tracks.begin(); iter != selection->tracks.end(); ++iter) {
-               if ((*iter)->hidden()) {
-                       continue;
-               }
-               (*iter)->get_selectables (start, end - 1, 0, DBL_MAX, touched);
-       }
-
-       begin_reversible_command (_("select all from range"));
-       selection->set (touched);
-       commit_reversible_command ();
-}
-
-
-void
-Editor::select_all_selectables_using_punch()
-{
-       Location* location = session->locations()->auto_punch_location();
-       list<Selectable *> touched;
-
-       if (location == 0 || (location->end() - location->start() <= 1))  {
-               return;
-       }
-
-       for (TrackViewList::iterator iter = track_views.begin(); iter != track_views.end(); ++iter) {
-               if ((*iter)->hidden()) {
-                       continue;
-               }
-               (*iter)->get_selectables (location->start(), location->end() - 1, 0, DBL_MAX, touched);
-       }
-       begin_reversible_command (_("select all from punch"));
-       selection->set (touched);
-       commit_reversible_command ();
-
-}
-
-void
-Editor::select_all_selectables_using_loop()
-{
-       Location* location = session->locations()->auto_loop_location();
-       list<Selectable *> touched;
-
-       if (location == 0 || (location->end() - location->start() <= 1))  {
-               return;
-       }
-
-       for (TrackViewList::iterator iter = track_views.begin(); iter != track_views.end(); ++iter) {
-               if ((*iter)->hidden()) {
-                       continue;
-               }
-               (*iter)->get_selectables (location->start(), location->end() - 1, 0, DBL_MAX, touched);
-       }
-       begin_reversible_command (_("select all from loop"));
-       selection->set (touched);
-       commit_reversible_command ();
-
-}
-
-void
-Editor::select_all_selectables_using_cursor (Cursor *cursor, bool after)
-{
-        nframes_t start;
-       nframes_t end;
-       list<Selectable *> touched;
-
-       if (after) {
-               begin_reversible_command (_("select all after cursor"));
-               start = cursor->current_frame ;
-               end = session->current_end_frame();
-       } else {
-               if (cursor->current_frame > 0) {
-                       begin_reversible_command (_("select all before cursor"));
-                       start = 0;
-                       end = cursor->current_frame - 1;
-               } else {
-                       return;
-               }
-       }
-
-       for (TrackViewList::iterator iter = track_views.begin(); iter != track_views.end(); ++iter) {
-               if ((*iter)->hidden()) {
-                       continue;
-               }
-               (*iter)->get_selectables (start, end, 0, DBL_MAX, touched);
-       }
-       selection->set (touched);
-       commit_reversible_command ();
-}
-
-void
-Editor::select_all_selectables_between_cursors (Cursor *cursor, Cursor *other_cursor)
-{
-        nframes_t start;
-       nframes_t end;
-       list<Selectable *> touched;
-       bool  other_cursor_is_first = cursor->current_frame > other_cursor->current_frame;
-
-       if (cursor->current_frame == other_cursor->current_frame) {
-               return;
-       }
-
-       begin_reversible_command (_("select all between cursors"));
-       if (other_cursor_is_first) {
-               start = other_cursor->current_frame;
-               end = cursor->current_frame - 1;
-               
-       } else {
-               start = cursor->current_frame;
-               end = other_cursor->current_frame - 1;
-       }
-       
-       for (TrackViewList::iterator iter = track_views.begin(); iter != track_views.end(); ++iter) {
-               if ((*iter)->hidden()) {
-                       continue;
-               }
-               (*iter)->get_selectables (start, end, 0, DBL_MAX, touched);
-       }
-       selection->set (touched);
-       commit_reversible_command ();
-}
-
 void
 Editor::amplitude_zoom_step (bool in)
 {
@@ -1748,7 +2053,7 @@ Editor::insert_region_list_drag (boost::shared_ptr<Region> region, int x, int y)
        double cx, cy;
        TimeAxisView *tv;
        nframes_t where;
-       AudioTimeAxisView *atv = 0;
+       RouteTimeAxisView *rtv = 0;
        boost::shared_ptr<Playlist> playlist;
        
        track_canvas.window_to_world (x, y, wx, wy);
@@ -1771,27 +2076,21 @@ Editor::insert_region_list_drag (boost::shared_ptr<Region> region, int x, int y)
                return;
        }
        
-       if ((atv = dynamic_cast<AudioTimeAxisView*>(tv)) == 0) {
+       if ((rtv = dynamic_cast<RouteTimeAxisView*>(tv)) == 0) {
                return;
        }
 
-       if ((playlist = atv->playlist()) == 0) {
+       if ((playlist = rtv->playlist()) == 0) {
                return;
        }
        
-       cerr << "drop target playlist, UC  = " << playlist.use_count() << endl;
-
        snap_to (where);
        
        begin_reversible_command (_("insert dragged region"));
         XMLNode &before = playlist->get_state();
-       cerr << "pre add target playlist, UC  = " << playlist.use_count() << endl;
        playlist->add_region (RegionFactory::create (region), where, 1.0);
-       cerr << "post add target playlist, UC  = " << playlist.use_count() << endl;
        session->add_command(new MementoCommand<Playlist>(*playlist, &before, &playlist->get_state()));
        commit_reversible_command ();
-
-       cerr << "post drop target playlist, UC  = " << playlist.use_count() << endl;
 }
 
 void
@@ -1806,6 +2105,10 @@ Editor::insert_region_list_selection (float times)
                if ((tv = dynamic_cast<RouteTimeAxisView*>(selection->tracks.front())) == 0) {
                        return;
                }
+       } else if (entered_track != 0) {
+               if ((tv = dynamic_cast<RouteTimeAxisView*>(entered_track)) == 0) {
+                       return;
+               }
        } else {
                return;
        }
@@ -1832,7 +2135,7 @@ Editor::insert_region_list_selection (float times)
                
                begin_reversible_command (_("insert region"));
                XMLNode &before = playlist->get_state();
-               playlist->add_region ((RegionFactory::create (region)), edit_cursor->current_frame, times);
+               playlist->add_region ((RegionFactory::create (region)), get_preferred_edit_position(), times);
                session->add_command(new MementoCommand<Playlist>(*playlist, &before, &playlist->get_state()));
                commit_reversible_command ();
        } 
@@ -1855,6 +2158,30 @@ Editor::edit_envelope ()
 
 /* PLAYBACK */
 
+void
+Editor::transition_to_rolling (bool fwd)
+{
+       if (!session) {
+               return;
+       }
+
+       switch (Config->get_slave_source()) {
+       case None:
+       case JACK:
+               break;
+       default:
+               /* transport controlled by the master */
+               return;
+       }
+
+       if (session->is_auditioning()) {
+               session->cancel_audition ();
+               return;
+       }
+       
+       session->request_transport_speed (fwd ? 1.0f : -1.0f);
+}
+
 void
 Editor::toggle_playback (bool with_abort)
 {
@@ -1893,9 +2220,28 @@ Editor::play_from_start ()
 }
 
 void
-Editor::play_from_edit_cursor ()
+Editor::play_from_edit_point ()
+{
+       session->request_locate (get_preferred_edit_position(), true);
+}
+
+void
+Editor::play_from_edit_point_and_return ()
 {
-       session->request_locate (edit_cursor->current_frame, true);
+       nframes64_t start_frame;
+       nframes64_t return_frame;
+
+       /* don't reset the return frame if its already set */
+
+       if ((return_frame = session->requested_return_frame()) < 0) {
+               return_frame = session->audible_frame();
+       }
+
+       start_frame = get_preferred_edit_position (true);
+
+       if (start_frame >= 0) {
+               session->request_roll_at_and_return (start_frame, return_frame);
+       }
 }
 
 void
@@ -1909,17 +2255,7 @@ Editor::play_selection ()
 }
 
 void
-Editor::play_selected_region ()
-{
-       if (!selection->regions.empty()) {
-               RegionView *rv = *(selection->regions.begin());
-
-               session->request_bounded_roll (rv->region()->position(), rv->region()->last_frame());   
-       }
-}
-
-void
-Editor::loop_selected_region ()
+Editor::loop_selected_region ()
 {
        if (!selection->regions.empty()) {
                RegionView *rv = *(selection->regions.begin());
@@ -1990,66 +2326,62 @@ Editor::lower_region_to_bottom ()
        selection->foreach_region (&Region::lower_to_bottom);
 }
 
+/** Show the region editor for the selected regions */
 void
 Editor::edit_region ()
 {
-       if (clicked_regionview == 0) {
-               return;
-       }
-       
-       clicked_regionview->show_region_editor ();
+       selection->foreach_regionview (&RegionView::show_region_editor);
 }
 
 void
-Editor::rename_region ()
+Editor::rename_region()
 {
-       Dialog dialog;
-       Entry  entry;
-       Button ok_button (_("OK"));
-       Button cancel_button (_("Cancel"));
-
        if (selection->regions.empty()) {
                return;
        }
 
-       dialog.set_title (_("ardour: rename region"));
-       dialog.set_name ("RegionRenameWindow");
-       dialog.set_size_request (300, -1);
-       dialog.set_position (Gtk::WIN_POS_MOUSE);
-       dialog.set_modal (true);
+       WindowTitle title (Glib::get_application_name());
+       title += _("Rename Region");
 
-       dialog.get_vbox()->set_border_width (10);
-       dialog.get_vbox()->pack_start (entry);
-       dialog.get_action_area()->pack_start (ok_button);
-       dialog.get_action_area()->pack_start (cancel_button);
+       ArdourDialog d (*this, title.get_string(), true, false);
+       Entry entry;
+       Label label (_("New name:"));
+       HBox hbox;
 
-       entry.set_name ("RegionNameDisplay");
-       ok_button.set_name ("EditorGTKButton");
-       cancel_button.set_name ("EditorGTKButton");
+       hbox.set_spacing (6);
+       hbox.pack_start (label, false, false);
+       hbox.pack_start (entry, true, true);
 
-       region_renamed = false;
+       d.get_vbox()->set_border_width (12);
+       d.get_vbox()->pack_start (hbox, false, false);
 
-       entry.signal_activate().connect (bind (mem_fun(*this, &Editor::rename_region_finished), true));
-       ok_button.signal_clicked().connect (bind (mem_fun(*this, &Editor::rename_region_finished), true));
-       cancel_button.signal_clicked().connect (bind (mem_fun(*this, &Editor::rename_region_finished), false));
+       d.add_button(Gtk::Stock::OK, Gtk::RESPONSE_OK);
+       d.add_button(Gtk::Stock::CANCEL, Gtk::RESPONSE_CANCEL);
 
-       /* recurse */
+       d.set_size_request (300, -1);
+       d.set_position (Gtk::WIN_POS_MOUSE);
 
-       dialog.show_all ();
-       Main::run ();
+       entry.set_text (selection->regions.front()->region()->name());
+       entry.select_region (0, -1);
 
-       if (region_renamed) {
-               (*selection->regions.begin())->region()->set_name (entry.get_text());
-               redisplay_regions ();
-       }
-}
+       entry.signal_activate().connect (bind (mem_fun (d, &Dialog::response), RESPONSE_OK));
+       
+       d.show_all ();
+       
+       entry.grab_focus();
 
-void
-Editor::rename_region_finished (bool status)
+       int ret = d.run();
 
-{
-       region_renamed = status;
-       Main::quit ();
+       d.hide ();
+
+       if (ret == RESPONSE_OK) {
+               std::string str = entry.get_text();
+               strip_whitespace_edges (str);
+               if (!str.empty()) {
+                       selection->regions.front()->region()->set_name (str);
+                       redisplay_regions ();
+               }
+       }
 }
 
 void
@@ -2071,13 +2403,40 @@ Editor::audition_playlist_region_via_route (boost::shared_ptr<Region> region, Ro
        /* XXX how to unset the solo state ? */
 }
 
+/** Start an audition of the first selected region */
 void
-Editor::audition_selected_region ()
+Editor::play_edit_range ()
 {
-       if (!selection->regions.empty()) {
-               RegionView* rv = *(selection->regions.begin());
-               session->audition_region (rv->region());
+       nframes64_t start, end;
+
+       if (get_edit_op_range (start, end)) {
+               session->request_bounded_roll (start, end);
+       }
+}
+
+void
+Editor::play_selected_region ()
+{
+       nframes64_t start = max_frames;
+       nframes64_t end = 0;
+
+       ExclusiveRegionSelection esr (*this, entered_regionview);
+
+       if (selection->regions.empty()) {
+               return;
+       }
+
+       for (RegionSelection::iterator i = selection->regions.begin(); i != selection->regions.end(); ++i) {
+               if ((*i)->region()->position() < start) {
+                       start = (*i)->region()->position();
+               }
+               if ((*i)->region()->last_frame() + 1 > end) {
+                       end = (*i)->region()->last_frame() + 1;
+               }
        }
+
+       session->request_stop ();
+       session->request_bounded_roll (start, end);
 }
 
 void
@@ -2166,6 +2525,8 @@ Editor::create_region_from_selection (vector<boost::shared_ptr<AudioRegion> >& n
        nframes_t start = selection->time[clicked_selection].start;
        nframes_t end = selection->time[clicked_selection].end;
        
+       sort_track_selection ();
+
        for (TrackSelection::iterator i = selection->tracks.begin(); i != selection->tracks.end(); ++i) {
 
                boost::shared_ptr<AudioRegion> current;
@@ -2196,17 +2557,22 @@ Editor::create_region_from_selection (vector<boost::shared_ptr<AudioRegion> >& n
 void
 Editor::split_multichannel_region ()
 {
-       vector<AudioRegion*> v;
-
-       AudioRegionView* clicked_arv = dynamic_cast<AudioRegionView*>(clicked_regionview);
-       
-       if (!clicked_arv || clicked_arv->audio_region()->n_channels() < 2) {
+       if (selection->regions.empty()) {
                return;
        }
 
-       clicked_arv->audio_region()->separate_by_channel (*session, v);
+       vector<boost::shared_ptr<AudioRegion> > v;
+
+       for (list<RegionView*>::iterator x = selection->regions.begin(); x != selection->regions.end(); ++x) {
+
+               AudioRegionView* arv = dynamic_cast<AudioRegionView*>(*x);
+               
+               if (!arv || arv->audio_region()->n_channels() < 2) {
+                       continue;
+               }
 
-       /* nothing else to do, really */
+               (arv)->audio_region()->separate_by_channel (*session, v);
+       }
 }
 
 void
@@ -2216,133 +2582,212 @@ Editor::new_region_from_selection ()
        cancel_selection ();
 }
 
-void
-Editor::separate_region_from_selection ()
+static void
+add_if_covered (RegionView* rv, const AudioRange* ar, RegionSelection* rs)
 {
-       // FIXME: TYPE
-       
-       bool doing_undo = false;
-
-       if (selection->time.empty()) {
-               return;
+       switch (rv->region()->coverage (ar->start, ar->end - 1)) {
+       case OverlapNone:
+               break;
+       default:
+               rs->push_back (rv);
        }
+}
 
+void
+Editor::separate_regions_between (const TimeSelection& ts)
+{
+       bool in_command = false;
        boost::shared_ptr<Playlist> playlist;
+       RegionSelection new_selection;
+       TrackSelection tmptracks;
+
+       if (selection->tracks.empty()) {
                
-       for (TrackSelection::iterator i = selection->tracks.begin(); i != selection->tracks.end(); ++i) {
+               /* use tracks with selected regions */
 
-               AudioTimeAxisView* atv;
+               for (RegionSelection::iterator i = selection->regions.begin(); i != selection->regions.end(); ++i) {
+                       TimeAxisView* tv = &(*i)->get_time_axis_view();
+
+                       if (find (tmptracks.begin(), tmptracks.end(), tv) == tmptracks.end()) {
+                               tmptracks.push_back (tv);
+                       }
+               }
+
+               if (tmptracks.empty()) {
+                       /* no regions selected: use all tracks */
+                       tmptracks = track_views;
+               }
+
+       } else {
+
+               tmptracks = selection->tracks;
+
+       }
+
+       sort_track_selection (&tmptracks);
+
+
+
+
+       for (TrackSelection::iterator i = tmptracks.begin(); i != tmptracks.end(); ++i) {
 
-               if ((atv = dynamic_cast<AudioTimeAxisView*> ((*i))) != 0) {
+               RouteTimeAxisView* rtv;
+
+               if ((rtv = dynamic_cast<RouteTimeAxisView*> ((*i))) != 0) {
+
+                       if (rtv->is_track()) {
+
+                               /* no edits to destructive tracks */
 
-                       if (atv->is_audio_track()) {
+                               if (rtv->track()->diskstream()->destructive()) {
+                                       continue;
+                               }
                                        
-                               if ((playlist = atv->playlist()) != 0) {
-                                       if (!doing_undo) {
-                                               begin_reversible_command (_("separate"));
-                                               doing_undo = true;
-                                       }
-                                        XMLNode *before;
-                                       if (doing_undo) 
-                                            before = &(playlist->get_state());
-                       
+                               if ((playlist = rtv->playlist()) != 0) {
+
+                                       XMLNode *before;
+                                       bool got_some;
+
+                                       before = &(playlist->get_state());
+                                       got_some = false;
+
                                        /* XXX need to consider musical time selections here at some point */
 
-                                       double speed = atv->get_diskstream()->speed();
+                                       double speed = rtv->get_diskstream()->speed();
+
+
+                                       for (list<AudioRange>::const_iterator t = ts.begin(); t != ts.end(); ++t) {
+
+                                               sigc::connection c = rtv->view()->RegionViewAdded.connect (mem_fun(*this, &Editor::collect_new_region_view));
+                                               latest_regionviews.clear ();
 
-                                       for (list<AudioRange>::iterator t = selection->time.begin(); t != selection->time.end(); ++t) {
                                                playlist->partition ((nframes_t)((*t).start * speed), (nframes_t)((*t).end * speed), true);
+
+                                               c.disconnect ();
+
+                                               if (!latest_regionviews.empty()) {
+                                                       
+                                                       got_some = true;
+
+                                                       rtv->view()->foreach_regionview (bind (sigc::ptr_fun (add_if_covered), &(*t), &new_selection));
+                                                       
+                                                       if (!in_command) {
+                                                               begin_reversible_command (_("separate"));
+                                                               in_command = true;
+                                                       }
+                                                       
+                                                       session->add_command(new MementoCommand<Playlist>(*playlist, before, &playlist->get_state()));
+                                                       
+                                               } 
                                        }
 
-                                       if (doing_undo) 
-                                            session->add_command(new MementoCommand<Playlist>(*playlist, before, &playlist->get_state()));
+                                       if (!got_some) {
+                                               delete before;
+                                       }
                                }
                        }
                }
        }
 
-       if (doing_undo) commit_reversible_command ();
+       if (in_command) {
+               selection->set (new_selection);
+               set_mouse_mode (MouseObject);
+
+               commit_reversible_command ();
+       }
 }
 
 void
-Editor::separate_regions_using_location (Location& loc)
+Editor::separate_region_from_selection ()
 {
-       // FIXME: TYPE
-       
-       bool doing_undo = false;
-
-       if (loc.is_mark()) {
-               return;
-       }
-
-       boost::shared_ptr<Playlist> playlist;
-
-       /* XXX i'm unsure as to whether this should operate on selected tracks only 
-          or the entire enchillada. uncomment the below line to correct the behaviour 
-          (currently set for all tracks)
+       /* preferentially use *all* ranges in the time selection if we're in range mode
+          to allow discontiguous operation, since get_edit_op_range() currently
+          returns a single range.
        */
+       if (mouse_mode == MouseRange && !selection->time.empty()) {
 
-       for (TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) {    
-       //for (TrackSelection::iterator i = selection->tracks.begin(); i != selection->tracks.end(); ++i) {
-
-               AudioTimeAxisView* atv;
+               separate_regions_between (selection->time);
 
-               if ((atv = dynamic_cast<AudioTimeAxisView*> ((*i))) != 0) {
+       } else {
 
-                       if (atv->is_audio_track()) {
-                                       
-                               if ((playlist = atv->playlist()) != 0) {
-                                        XMLNode *before;
-                                       if (!doing_undo) {
-                                               begin_reversible_command (_("separate"));
-                                               doing_undo = true;
-                                       }
-                                       if (doing_undo) 
-                                            before = &(playlist->get_state());
-                                            
+               nframes64_t start;
+               nframes64_t end;
+               
+               if (get_edit_op_range (start, end)) {
                        
-                                       /* XXX need to consider musical time selections here at some point */
-
-                                       double speed = atv->get_diskstream()->speed();
+                       AudioRange ar (start, end, 1);
+                       TimeSelection ts;
+                       ts.push_back (ar);
 
+                       /* force track selection */
 
-                                       playlist->partition ((nframes_t)(loc.start() * speed), (nframes_t)(loc.end() * speed), true);
-                                       if (doing_undo) 
-                                            session->add_command(new MementoCommand<Playlist>(*playlist, before, &playlist->get_state()));
-                               }
-                       }
+                       ensure_entered_region_selected ();
+                       
+                       separate_regions_between (ts);
                }
        }
-
-       if (doing_undo) commit_reversible_command ();
 }
 
 void
-Editor::crop_region_to_selection ()
+Editor::separate_regions_using_location (Location& loc)
 {
-       if (selection->time.empty()) {
+       if (loc.is_mark()) {
                return;
        }
 
-       vector<boost::shared_ptr<Playlist> > playlists;
-       boost::shared_ptr<Playlist> playlist;
+       AudioRange ar (loc.start(), loc.end(), 1);
+       TimeSelection ts;
 
-       if (clicked_axisview != 0) {
+       ts.push_back (ar);
 
-               if ((playlist = clicked_axisview->playlist()) == 0) {
-                       return;
-               }
+       separate_regions_between (ts);
+}
+
+void
+Editor::crop_region_to_selection ()
+{
+       ensure_entered_region_selected (true);
+
+       if (!selection->time.empty()) {
 
-               playlists.push_back (playlist);
+               crop_region_to (selection->time.start(), selection->time.end_frame());
 
        } else {
+
+               nframes64_t start;
+               nframes64_t end;
+
+               if (get_edit_op_range (start, end)) {
+                       crop_region_to (start, end);
+               }
+       }
                
-               for (TrackSelection::iterator i = selection->tracks.begin(); i != selection->tracks.end(); ++i) {
+}              
+
+void
+Editor::crop_region_to (nframes_t start, nframes_t end)
+{
+       vector<boost::shared_ptr<Playlist> > playlists;
+       boost::shared_ptr<Playlist> playlist;
+       TrackSelection* ts;
 
-                       RouteTimeAxisView* rtv = dynamic_cast<RouteTimeAxisView*>(*i);
+       if (selection->tracks.empty()) {
+               ts = &track_views;
+       } else {
+               sort_track_selection ();
+               ts = &selection->tracks;
+       }
+       
+       for (TrackSelection::iterator i = ts->begin(); i != ts->end(); ++i) {
+               
+               RouteTimeAxisView* rtv;
+               
+               if ((rtv = dynamic_cast<RouteTimeAxisView*> ((*i))) != 0) {
 
-                       if (rtv && rtv->is_track()) {
+                       boost::shared_ptr<Track> t = rtv->track();
 
+                       if (t != 0 && ! t->diskstream()->destructive()) {
+                               
                                if ((playlist = rtv->playlist()) != 0) {
                                        playlists.push_back (playlist);
                                }
@@ -2350,40 +2795,46 @@ Editor::crop_region_to_selection ()
                }
        }
 
-       if (!playlists.empty()) {
-
-               nframes_t start;
-               nframes_t end;
-               nframes_t cnt;
-
-               begin_reversible_command (_("trim to selection"));
-
-               for (vector<boost::shared_ptr<Playlist> >::iterator i = playlists.begin(); i != playlists.end(); ++i) {
-                       
-                       boost::shared_ptr<Region> region;
-                       
-                       start = selection->time.start();
-
-                       if ((region = (*i)->top_region_at(start)) == 0) {
-                               continue;
-                       }
-                       
-                       /* now adjust lengths to that we do the right thing
-                          if the selection extends beyond the region
-                       */
-                       
-                       start = max (start, region->position());
-                       end = min (selection->time.end_frame(), start + region->length() - 1);
-                       cnt = end - start + 1;
-
-                        XMLNode &before = (*i)->get_state();
-                       region->trim_to (start, cnt, this);
-                        XMLNode &after = (*i)->get_state();
-                       session->add_command (new MementoCommand<Playlist>(*(*i), &before, &after));
+       if (playlists.empty()) {
+               return;
+       }
+               
+       nframes_t the_start;
+       nframes_t the_end;
+       nframes_t cnt;
+       
+       begin_reversible_command (_("trim to selection"));
+       
+       for (vector<boost::shared_ptr<Playlist> >::iterator i = playlists.begin(); i != playlists.end(); ++i) {
+               
+               boost::shared_ptr<Region> region;
+       
+               the_start = start;
+       
+               if ((region = (*i)->top_region_at(the_start)) == 0) {
+                       continue;
                }
-
-               commit_reversible_command ();
+               
+               /* now adjust lengths to that we do the right thing
+                  if the selection extends beyond the region
+               */
+               
+               the_start = max (the_start, region->position());
+               if (max_frames - the_start < region->length()) {
+                       the_end = the_start + region->length() - 1;
+               } else {
+                       the_end = max_frames;
+               }
+               the_end = min (end, the_end);
+               cnt = the_end - the_start + 1;
+               
+               XMLNode &before = (*i)->get_state();
+               region->trim_to (the_start, cnt, this);
+               XMLNode &after = (*i)->get_state();
+               session->add_command (new MementoCommand<Playlist>(*(*i), &before, &after));
        }
+       
+       commit_reversible_command ();
 }              
 
 void
@@ -2477,54 +2928,57 @@ Editor::region_fill_selection ()
 }
 
 void
-Editor::set_a_regions_sync_position (boost::shared_ptr<Region> region, nframes_t position)
+Editor::set_region_sync_from_edit_point ()
 {
-
-       if (!region->covers (position)) {
-         error << _("Programming error. that region doesn't cover that position") << __FILE__ << " +" << __LINE__ << endmsg;
-               return;
-       }
-       begin_reversible_command (_("set region sync position"));
-        XMLNode &before = region->playlist()->get_state();
-       region->set_sync_position (position);
-        XMLNode &after = region->playlist()->get_state();
-       session->add_command(new MementoCommand<Playlist>(*(region->playlist()), &before, &after));
-       commit_reversible_command ();
+       nframes64_t where = get_preferred_edit_position ();
+       ensure_entered_region_selected (true);
+       set_sync_point (where, selection->regions);
 }
 
 void
-Editor::set_region_sync_from_edit_cursor ()
+Editor::set_sync_point (nframes64_t where, const RegionSelection& rs)
 {
-       if (clicked_regionview == 0) {
-               return;
-       }
+       bool in_command = false;
 
-       if (!clicked_regionview->region()->covers (edit_cursor->current_frame)) {
-               error << _("Place the edit cursor at the desired sync point") << endmsg;
-               return;
+       for (RegionSelection::const_iterator r = rs.begin(); r != rs.end(); ++r) {
+               
+               if (!(*r)->region()->covers (where)) {
+                       continue;
+               }
+
+               boost::shared_ptr<Region> region ((*r)->region());
+
+               if (!in_command) {
+                       begin_reversible_command (_("set sync point"));
+                       in_command = true;
+               }
+
+               XMLNode &before = region->playlist()->get_state();
+               region->set_sync_position (get_preferred_edit_position());
+               XMLNode &after = region->playlist()->get_state();
+               session->add_command(new MementoCommand<Playlist>(*(region->playlist()), &before, &after));
        }
 
-       boost::shared_ptr<Region> region (clicked_regionview->region());
-       begin_reversible_command (_("set sync from edit cursor"));
-        XMLNode &before = region->playlist()->get_state();
-       region->set_sync_position (edit_cursor->current_frame);
-        XMLNode &after = region->playlist()->get_state();
-       session->add_command(new MementoCommand<Playlist>(*(region->playlist()), &before, &after));
-       commit_reversible_command ();
+       if (in_command) {
+               commit_reversible_command ();
+       }
 }
 
+/** Remove the sync positions of the selection */
 void
 Editor::remove_region_sync ()
 {
-       if (clicked_regionview) {
-               boost::shared_ptr<Region> region (clicked_regionview->region());
-               begin_reversible_command (_("remove sync"));
-                XMLNode &before = region->playlist()->get_state();
-               region->clear_sync_position ();
-                XMLNode &after = region->playlist()->get_state();
-               session->add_command(new MementoCommand<Playlist>(*(region->playlist()), &before, &after));
-               commit_reversible_command ();
+       begin_reversible_command (_("remove sync"));
+
+       for (RegionSelection::iterator i = selection->regions.begin(); i != selection->regions.end(); ++i) {
+               boost::shared_ptr<Region> r = (*i)->region();
+               XMLNode &before = r->playlist()->get_state();
+               r->clear_sync_position ();
+                XMLNode &after = r->playlist()->get_state();
+               session->add_command(new MementoCommand<Playlist>(*(r->playlist()), &before, &after));
        }
+
+       commit_reversible_command ();
 }
 
 void
@@ -2546,13 +3000,33 @@ Editor::naturalize ()
 void
 Editor::align (RegionPoint what)
 {
-       align_selection (what, edit_cursor->current_frame);
+       ensure_entered_region_selected ();
+
+       nframes64_t where = get_preferred_edit_position();
+
+       if (!selection->regions.empty()) {
+               align_selection (what, where, selection->regions);
+       } else {
+
+               RegionSelection rs;
+               rs = get_regions_at (where, selection->tracks);
+               align_selection (what, where, rs);
+       }
 }
 
 void
 Editor::align_relative (RegionPoint what)
 {
-       align_selection_relative (what, edit_cursor->current_frame);
+       nframes64_t where = get_preferred_edit_position();
+
+       if (!selection->regions.empty()) {
+               align_selection_relative (what, where, selection->regions);
+       } else {
+
+               RegionSelection rs;
+               rs = get_regions_at (where, selection->tracks);
+               align_selection_relative (what, where, rs);
+       }
 }
 
 struct RegionSortByTime {
@@ -2562,9 +3036,9 @@ struct RegionSortByTime {
 };
 
 void
-Editor::align_selection_relative (RegionPoint point, nframes_t position)
+Editor::align_selection_relative (RegionPoint point, nframes_t position, const RegionSelection& rs)
 {
-       if (selection->regions.empty()) {
+       if (rs.empty()) {
                return;
        }
 
@@ -2573,34 +3047,64 @@ Editor::align_selection_relative (RegionPoint point, nframes_t position)
        int dir;
 
        list<RegionView*> sorted;
-       selection->regions.by_position (sorted);
+       rs.by_position (sorted);
        boost::shared_ptr<Region> r ((*sorted.begin())->region());
 
        switch (point) {
        case Start:
-               pos = r->first_frame ();
+               pos = position;
+               if (position > r->position()) {
+                       distance = position - r->position();
+                       dir = 1;
+               } else {
+                       distance = r->position() - position;
+                       dir = -1;
+               }
                break;
-
+               
        case End:
-               pos = r->last_frame();
+               if (position > r->last_frame()) {
+                       distance = position - r->last_frame();
+                       pos = r->position() + distance;
+                       dir = 1;
+               } else {
+                       distance = r->last_frame() - position;
+                       pos = r->position() - distance;
+                       dir = -1;
+               }
                break;
 
        case SyncPoint:
-               pos = r->adjust_to_sync (r->first_frame());
+               pos = r->adjust_to_sync (position);
+               if (pos > r->position()) {
+                       distance = pos - r->position();
+                       dir = 1;
+               } else {
+                       distance = r->position() - pos;
+                       dir = -1;
+               }
                break;  
        }
 
-       if (pos > position) {
-               distance = pos - position;
-               dir = -1;
-       } else {
-               distance = position - pos;
-               dir = 1;
+       if (pos == r->position()) {
+               return;
        }
 
        begin_reversible_command (_("align selection (relative)"));
 
-       for (RegionSelection::iterator i = selection->regions.begin(); i != selection->regions.end(); ++i) {
+       /* move first one specially */
+
+       XMLNode &before = r->playlist()->get_state();
+       r->set_position (pos, this);
+       XMLNode &after = r->playlist()->get_state();
+       session->add_command(new MementoCommand<Playlist>(*(r->playlist()), &before, &after));
+
+       /* move rest by the same amount */
+       
+       RegionSelection::const_iterator i = rs.begin();
+       ++i;
+
+       for (; i != rs.end(); ++i) {
 
                boost::shared_ptr<Region> region ((*i)->region());
 
@@ -2621,15 +3125,15 @@ Editor::align_selection_relative (RegionPoint point, nframes_t position)
 }
 
 void
-Editor::align_selection (RegionPoint point, nframes_t position)
+Editor::align_selection (RegionPoint point, nframes_t position, const RegionSelection& rs)
 {
-       if (selection->regions.empty()) {
+       if (rs.empty()) {
                return;
        }
 
        begin_reversible_command (_("align selection"));
 
-       for (RegionSelection::iterator i = selection->regions.begin(); i != selection->regions.end(); ++i) {
+       for (RegionSelection::const_iterator i = rs.begin(); i != rs.end(); ++i) {
                align_region_internal ((*i)->region(), point, position);
        }
 
@@ -2669,66 +3173,173 @@ Editor::align_region_internal (boost::shared_ptr<Region> region, RegionPoint poi
        session->add_command(new MementoCommand<Playlist>(*(region->playlist()), &before, &after));
 }      
 
+/** Trim the end of the selected regions to the position of the edit cursor */
 void
-Editor::trim_region_to_edit_cursor ()
+Editor::trim_region_to_loop ()
 {
-       if (clicked_regionview == 0) {
+       Location* loc = session->locations()->auto_loop_location();
+       if (!loc) {
                return;
        }
-
-       boost::shared_ptr<Region> region (clicked_regionview->region());
-
-       float speed = 1.0f;
-       AudioTimeAxisView *atav;
-
-       if ( clicked_axisview != 0 && (atav = dynamic_cast<AudioTimeAxisView*>(clicked_axisview)) != 0 ) {
-               if (atav->get_diskstream() != 0) {
-                       speed = atav->get_diskstream()->speed();
-               }
-       }
-
-       begin_reversible_command (_("trim to edit"));
-        XMLNode &before = region->playlist()->get_state();
-       region->trim_end( session_frame_to_track_frame(edit_cursor->current_frame, speed), this);
-        XMLNode &after = region->playlist()->get_state();
-       session->add_command(new MementoCommand<Playlist>(*(region->playlist()), &before, &after));
-       commit_reversible_command ();
+       trim_region_to_location (*loc, _("trim to loop"));
 }
 
 void
-Editor::trim_region_from_edit_cursor ()
+Editor::trim_region_to_punch ()
 {
-       if (clicked_regionview == 0) {
+       Location* loc = session->locations()->auto_punch_location();
+       if (!loc) {
                return;
        }
+       trim_region_to_location (*loc, _("trim to punch"));
+}
+void
+Editor::trim_region_to_location (const Location& loc, const char* str)
+{
+       ExclusiveRegionSelection ers (*this, entered_regionview);
 
-       boost::shared_ptr<Region> region (clicked_regionview->region());
+       RegionSelection& rs (get_regions_for_action ());
 
-       float speed = 1.0f;
-       AudioTimeAxisView *atav;
+       begin_reversible_command (str);
+
+       for (RegionSelection::iterator x = rs.begin(); x != rs.end(); ++x) {
+               AudioRegionView* arv = dynamic_cast<AudioRegionView*> (*x);
+
+               if (!arv) {
+                       continue;
+               }
+
+               /* require region to span proposed trim */
+
+               switch (arv->region()->coverage (loc.start(), loc.end())) {
+               case OverlapInternal:
+                       break;
+               default:
+                       continue;
+               }
+                               
+               AudioTimeAxisView* atav = dynamic_cast<AudioTimeAxisView*> (&arv->get_time_axis_view());
+
+               if (!atav) {
+                       return;
+               }
+
+               float speed = 1.0;
+               nframes_t start;
+               nframes_t end;
 
-       if ( clicked_axisview != 0 && (atav = dynamic_cast<AudioTimeAxisView*>(clicked_axisview)) != 0 ) {
                if (atav->get_diskstream() != 0) {
                        speed = atav->get_diskstream()->speed();
                }
+               
+               start = session_frame_to_track_frame (loc.start(), speed);
+               end = session_frame_to_track_frame (loc.end(), speed);
+
+               XMLNode &before = arv->region()->playlist()->get_state();
+               arv->region()->trim_to (start, (end - start), this);
+               XMLNode &after = arv->region()->playlist()->get_state();
+               session->add_command(new MementoCommand<Playlist>(*(arv->region()->playlist()), &before, &after));
        }
 
-       begin_reversible_command (_("trim to edit"));
-        XMLNode &before = region->playlist()->get_state();
-       region->trim_front ( session_frame_to_track_frame(edit_cursor->current_frame, speed), this);
-        XMLNode &after = region->playlist()->get_state();
-       session->add_command(new MementoCommand<Playlist>(*(region->playlist()), &before, &after));
+       commit_reversible_command ();
+}
+
+void
+Editor::trim_region_to_edit_point ()
+{
+       ExclusiveRegionSelection ers (*this, entered_regionview);
+
+       RegionSelection& rs (get_regions_for_action ());
+       nframes64_t where = get_preferred_edit_position();
+
+       begin_reversible_command (_("trim region start to edit point"));
+
+       for (RegionSelection::iterator x = rs.begin(); x != rs.end(); ++x) {
+               AudioRegionView* arv = dynamic_cast<AudioRegionView*> (*x);
+
+               if (!arv) {
+                       continue;
+               }
+
+               /* require region to cover trim */
+
+               if (!arv->region()->covers (where)) {
+                       continue;
+               }
+
+               AudioTimeAxisView* atav = dynamic_cast<AudioTimeAxisView*> (&arv->get_time_axis_view());
+
+               if (!atav) {
+                       return;
+               }
+
+               float speed = 1.0;
+
+               if (atav->get_diskstream() != 0) {
+                       speed = atav->get_diskstream()->speed();
+               }
+
+               XMLNode &before = arv->region()->playlist()->get_state();
+               arv->region()->trim_end( session_frame_to_track_frame(where, speed), this);
+               XMLNode &after = arv->region()->playlist()->get_state();
+               session->add_command(new MementoCommand<Playlist>(*(arv->region()->playlist()), &before, &after));
+       }
+               
+       commit_reversible_command ();
+}
+
+void
+Editor::trim_region_from_edit_point ()
+{
+       ExclusiveRegionSelection ers (*this, entered_regionview);
+
+       RegionSelection& rs (get_regions_for_action ());
+       nframes64_t where = get_preferred_edit_position();
+
+       begin_reversible_command (_("trim region end to edit point"));
+
+       for (RegionSelection::iterator x = rs.begin(); x != rs.end(); ++x) {
+               AudioRegionView* arv = dynamic_cast<AudioRegionView*> (*x);
+
+               if (!arv) {
+                       continue;
+               }
+
+               /* require region to cover trim */
+
+               if (!arv->region()->covers (where)) {
+                       continue;
+               }
+
+               AudioTimeAxisView* atav = dynamic_cast<AudioTimeAxisView*> (&arv->get_time_axis_view());
+
+               if (!atav) {
+                       return;
+               }
+
+               float speed = 1.0;
+
+               if (atav->get_diskstream() != 0) {
+                       speed = atav->get_diskstream()->speed();
+               }
+
+               XMLNode &before = arv->region()->playlist()->get_state();
+               arv->region()->trim_front ( session_frame_to_track_frame(where, speed), this);
+               XMLNode &after = arv->region()->playlist()->get_state();
+               session->add_command(new MementoCommand<Playlist>(*(arv->region()->playlist()), &before, &after));
+       }
+               
        commit_reversible_command ();
 }
 
 void
 Editor::unfreeze_route ()
 {
-       if (clicked_routeview == 0 || !clicked_routeview->is_track()) {
+       if (clicked_routeview == 0 || !clicked_routeview->is_audio_track()) {
                return;
        }
        
-       clicked_routeview->track()->unfreeze ();
+       clicked_routeview->audio_track()->unfreeze ();
 }
 
 void*
@@ -2755,7 +3366,7 @@ Editor::freeze_progress_timeout (void *arg)
 void
 Editor::freeze_route ()
 {
-       if (clicked_routeview == 0 || !clicked_routeview->is_track()) {
+       if (clicked_routeview == 0 || !clicked_routeview->is_audio_track()) {
                return;
        }
        
@@ -2764,8 +3375,10 @@ Editor::freeze_route ()
        if (interthread_progress_window == 0) {
                build_interthread_progress_window ();
        }
-       
-       interthread_progress_window->set_title (_("ardour: freeze"));
+
+       WindowTitle title(Glib::get_application_name());
+       title += _("Freeze");
+       interthread_progress_window->set_title (title.get_string());
        interthread_progress_window->set_position (Gtk::WIN_POS_MOUSE);
        interthread_progress_window->show_all ();
        interthread_progress_bar.set_fraction (0.0f);
@@ -2807,25 +3420,25 @@ Editor::bounce_range_selection ()
                return;
        }
 
-       TrackViewList *views = get_valid_views (selection->time.track, selection->time.group);
+       TrackSelection views = selection->tracks;
 
        nframes_t start = selection->time[clicked_selection].start;
        nframes_t end = selection->time[clicked_selection].end;
        nframes_t cnt = end - start + 1;
-       
+
        begin_reversible_command (_("bounce range"));
 
-       for (TrackViewList::iterator i = views->begin(); i != views->end(); ++i) {
+       for (TrackViewList::iterator i = views.begin(); i != views.end(); ++i) {
 
-               AudioTimeAxisView* atv;
+               RouteTimeAxisView* rtv;
 
-               if ((atv = dynamic_cast<AudioTimeAxisView*> (*i)) == 0) {
+               if ((rtv = dynamic_cast<RouteTimeAxisView*> (*i)) == 0) {
                        continue;
                }
                
                boost::shared_ptr<Playlist> playlist;
                
-               if ((playlist = atv->playlist()) == 0) {
+               if ((playlist = rtv->playlist()) == 0) {
                        return;
                }
 
@@ -2834,30 +3447,60 @@ Editor::bounce_range_selection ()
                itt.done = false;
                itt.cancel = false;
                itt.progress = false;
-               
-                XMLNode &before = playlist->get_state();
-               atv->audio_track()->bounce_range (start, cnt, itt);
-                XMLNode &after = playlist->get_state();
+
+               XMLNode &before = playlist->get_state();
+               rtv->track()->bounce_range (start, cnt, itt);
+               XMLNode &after = playlist->get_state();
                session->add_command (new MementoCommand<Playlist> (*playlist, &before, &after));
        }
        
        commit_reversible_command ();
-       
-       delete views;
 }
 
+/** Cut selected regions, automation points or a time range */
 void
 Editor::cut ()
 {
        cut_copy (Cut);
 }
 
+/** Copy selected regions, automation points or a time range */
 void
 Editor::copy ()
 {
        cut_copy (Copy);
 }
 
+
+/** @return true if a Cut, Copy or Clear is possible */
+bool
+Editor::can_cut_copy () const
+{
+       switch (current_mouse_mode()) {
+               
+       case MouseObject:
+               if (!selection->regions.empty() || !selection->points.empty()) {
+                       return true;
+               }
+               break;
+               
+       case MouseRange:
+               if (!selection->time.empty()) {
+                       return true;
+               }
+               break;
+               
+       default:
+               break;
+       }
+
+       return false;
+}
+
+
+/** Cut, copy or clear selected regions, automation points or a time range.
+ * @param op Operation (Cut, Copy or Clear)
+ */
 void 
 Editor::cut_copy (CutCopyOp op)
 {
@@ -2879,14 +3522,29 @@ Editor::cut_copy (CutCopyOp op)
        
        cut_buffer->clear ();
 
+       if (entered_marker) {
+
+               /* cut/delete op while pointing at a marker */
+
+               bool ignored;
+               Location* loc = find_location_from_marker (entered_marker, ignored);
+
+               if (session && loc) {
+                       Glib::signal_idle().connect (bind (mem_fun(*this, &Editor::really_remove_marker), loc));
+               }
+
+               return;
+       }
+
        switch (current_mouse_mode()) {
        case MouseObject: 
+               cerr << "cutting in object mode\n";
                if (!selection->regions.empty() || !selection->points.empty()) {
 
                        begin_reversible_command (opname + _(" objects"));
 
                        if (!selection->regions.empty()) {
-                               
+                               cerr << "have regions to cut" << endl;
                                cut_copy_regions (op);
                                
                                if (op == Cut) {
@@ -2903,21 +3561,34 @@ Editor::cut_copy (CutCopyOp op)
                        }
 
                        commit_reversible_command ();   
+                       break; // terminate case statement here
+               } 
+               cerr << "nope, now cutting time range" << endl;
+               if (!selection->time.empty()) {
+                       /* don't cause suprises */
+                       break;
                }
-               break;
+               // fall thru if there was nothing selected
                
        case MouseRange:
-               if (!selection->time.empty()) {
-
-                       begin_reversible_command (opname + _(" range"));
-                       cut_copy_ranges (op);
-                       commit_reversible_command ();
-
-                       if (op == Cut) {
-                               selection->clear_time ();
+               if (selection->time.empty()) {
+                       nframes64_t start, end;
+                       cerr << "no time selection, get edit op range" << endl;
+                       if (!get_edit_op_range (start, end)) {
+                               cerr << "no edit op range" << endl;
+                               return;
                        }
+                       selection->set ((TimeAxisView*) 0, start, end);
+               }
                        
+               begin_reversible_command (opname + _(" range"));
+               cut_copy_ranges (op);
+               commit_reversible_command ();
+               
+               if (op == Cut) {
+                       selection->clear_time ();
                }
+
                break;
                
        default:
@@ -2925,6 +3596,9 @@ Editor::cut_copy (CutCopyOp op)
        }
 }
 
+/** Cut, copy or clear selected automation points.
+ * @param op Operation (Cut, Copy or Clear)
+ */
 void
 Editor::cut_copy_points (CutCopyOp op)
 {
@@ -2949,21 +3623,41 @@ struct lt_playlist {
     }
 };
        
+struct PlaylistMapping { 
+    TimeAxisView* tv;
+    boost::shared_ptr<Playlist> pl;
+
+    PlaylistMapping (TimeAxisView* tvp) : tv (tvp) {}
+};
+
+
+/** Cut, copy or clear selected regions.
+ * @param op Operation (Cut, Copy or Clear)
+ */
 void
 Editor::cut_copy_regions (CutCopyOp op)
-{
-        typedef std::map<boost::shared_ptr<AudioPlaylist>,boost::shared_ptr<AudioPlaylist> > PlaylistMapping;
-       PlaylistMapping pmap;
-       nframes_t first_position = max_frames;
+{      
+       /* we can't use a std::map here because the ordering is important, and we can't trivially sort
+          a map when we want ordered access to both elements. i think.
+       */
 
+       vector<PlaylistMapping> pmap;
+
+       nframes_t first_position = max_frames;
+       
        set<PlaylistState, lt_playlist> freezelist;
        pair<set<PlaylistState, lt_playlist>::iterator,bool> insert_result;
+       
+       /* get ordering correct before we cut/copy */
+       
+       selection->regions.sort_by_position_and_track ();
 
        for (RegionSelection::iterator x = selection->regions.begin(); x != selection->regions.end(); ++x) {
+
                first_position = min ((*x)->region()->position(), first_position);
 
                if (op == Cut || op == Clear) {
-                       boost::shared_ptr<AudioPlaylist> pl = boost::dynamic_pointer_cast<AudioPlaylist>((*x)->region()->playlist());
+                       boost::shared_ptr<Playlist> pl = (*x)->region()->playlist();
 
                        if (pl) {
 
@@ -2972,71 +3666,100 @@ Editor::cut_copy_regions (CutCopyOp op)
                                before.before = &pl->get_state();
                                
                                insert_result = freezelist.insert (before);
-
+                               
                                if (insert_result.second) {
                                        pl->freeze ();
                                }
                        }
                }
+
+               TimeAxisView* tv = &(*x)->get_trackview();
+               vector<PlaylistMapping>::iterator z;
+
+               for (z = pmap.begin(); z != pmap.end(); ++z) {
+                       if ((*z).tv == tv) {
+                               break;
+                       }
+               }
+               
+               if (z == pmap.end()) {
+                       pmap.push_back (PlaylistMapping (tv));
+               }
        }
 
        for (RegionSelection::iterator x = selection->regions.begin(); x != selection->regions.end(); ) {
 
-               boost::shared_ptr<AudioPlaylist> pl = boost::dynamic_pointer_cast<AudioPlaylist>((*x)->region()->playlist());
-               boost::shared_ptr<AudioPlaylist> npl;
+               boost::shared_ptr<Playlist> pl = (*x)->region()->playlist();
+               
+               if (!pl) {
+                       /* impossible, but this handles it for the future */
+                       continue;
+               }
+
+               TimeAxisView& tv = (*x)->get_trackview();
+               boost::shared_ptr<Playlist> npl;
                RegionSelection::iterator tmp;
                
                tmp = x;
                ++tmp;
 
-               if (pl) {
-
-                       PlaylistMapping::iterator pi = pmap.find (pl);
-                       
-                       if (pi == pmap.end()) {
-                               // FIXME
-                               npl = boost::dynamic_pointer_cast<AudioPlaylist> (PlaylistFactory::create (DataType::AUDIO, *session, "cutlist", true));
-                               npl->freeze();
-                               pmap[pl] = npl;
-                       } else {
-                               npl = pi->second;
-                       }
-
-                       // FIXME
-                       boost::shared_ptr<AudioRegion> ar = boost::dynamic_pointer_cast<AudioRegion>((*x)->region());
-                       switch (op) {
-                       case Cut:
-                               if (!ar) break;
-
-                               npl->add_region (RegionFactory::create (ar), (*x)->region()->position() - first_position);
-                               pl->remove_region (((*x)->region()));
+               vector<PlaylistMapping>::iterator z;
+               
+               for (z = pmap.begin(); z != pmap.end(); ++z) {
+                       if ((*z).tv == &tv) {
                                break;
+                       }
+               }
+               
+               assert (z != pmap.end());
+               
+               if (!(*z).pl) {
+                       npl = PlaylistFactory::create (pl->data_type(), *session, "cutlist", true);
+                       npl->freeze();
+                       (*z).pl = npl;
+               } else {
+                       npl = (*z).pl;
+               }
+               
+               boost::shared_ptr<Region> r = (*x)->region();
+               boost::shared_ptr<Region> _xx;
 
-                       case Copy:
-                               if (!ar) break;
-
-                               npl->add_region (RegionFactory::create (ar), (*x)->region()->position() - first_position);
-                               break;
+               assert (r != 0);
 
-                       case Clear:
-                               pl->remove_region (((*x)->region()));
-                               break;
-                       }
+               switch (op) {
+               case Cut:
+                       _xx = RegionFactory::create (r);
+                       npl->add_region (_xx, r->position() - first_position);
+                       pl->remove_region (r);
+                       break;
+                       
+               case Copy:
+                       /* copy region before adding, so we're not putting same object into two different playlists */
+                       npl->add_region (RegionFactory::create (r), r->position() - first_position);
+                       break;
+                       
+               case Clear:
+                       pl->remove_region (r);
+                       break;
                }
 
                x = tmp;
        }
-
+       
        list<boost::shared_ptr<Playlist> > foo;
-
-       for (PlaylistMapping::iterator i = pmap.begin(); i != pmap.end(); ++i) {
-               foo.push_back (i->second);
+       
+       /* the pmap is in the same order as the tracks in which selected regions occured */
+       
+       for (vector<PlaylistMapping>::iterator i = pmap.begin(); i != pmap.end(); ++i) {
+               (*i).pl->thaw();
+               foo.push_back ((*i).pl);
        }
+       
 
        if (!foo.empty()) {
                cut_buffer->set (foo);
        }
-       
+
        for (set<PlaylistState, lt_playlist>::iterator pl = freezelist.begin(); pl != freezelist.end(); ++pl) {
                (*pl).playlist->thaw ();
                session->add_command (new MementoCommand<Playlist>(*(*pl).playlist, (*pl).before, &(*pl).playlist->get_state()));
@@ -3046,7 +3769,15 @@ Editor::cut_copy_regions (CutCopyOp op)
 void
 Editor::cut_copy_ranges (CutCopyOp op)
 {
-       for (TrackSelection::iterator i = selection->tracks.begin(); i != selection->tracks.end(); ++i) {
+       TrackSelection* ts;
+
+       if (selection->tracks.empty()) {
+               ts = &track_views;
+       } else {
+               ts = &selection->tracks;
+       }
+
+       for (TrackSelection::iterator i = ts->begin(); i != ts->end(); ++i) {
                (*i)->cut_copy_clear (*selection, op);
        }
 }
@@ -3054,26 +3785,19 @@ Editor::cut_copy_ranges (CutCopyOp op)
 void
 Editor::paste (float times)
 {
-       paste_internal (edit_cursor->current_frame, times);
+       paste_internal (get_preferred_edit_position(), times);
 }
 
 void
 Editor::mouse_paste ()
 {
-       int x, y;
-       double wx, wy;
+       nframes64_t where;
+       bool ignored;
 
-       track_canvas.get_pointer (x, y);
-       track_canvas.window_to_world (x, y, wx, wy);
-       wx += horizontal_adjustment.get_value();
-       wy += vertical_adjustment.get_value();
+       if (!mouse_frame (where, ignored)) {
+               return;
+       }
 
-       GdkEvent event;
-       event.type = GDK_BUTTON_RELEASE;
-       event.button.x = wx;
-       event.button.y = wy;
-       
-       nframes_t where = event_frame (&event, 0, 0);
        snap_to (where);
        paste_internal (where, 1);
 }
@@ -3083,28 +3807,39 @@ Editor::paste_internal (nframes_t position, float times)
 {
        bool commit = false;
 
-       if (cut_buffer->empty() || selection->tracks.empty()) {
+       if (cut_buffer->empty()) {
                return;
        }
 
        if (position == max_frames) {
-               position = edit_cursor->current_frame;
+               position = get_preferred_edit_position();
        }
 
        begin_reversible_command (_("paste"));
 
+       TrackSelection ts;
        TrackSelection::iterator i;
        size_t nth;
 
-       for (nth = 0, i = selection->tracks.begin(); i != selection->tracks.end(); ++i, ++nth) {
-               
+       /* get everything in the correct order */
+
+
+       if (!selection->tracks.empty()) {
+               sort_track_selection ();
+               ts = selection->tracks;
+       } else if (entered_track) {
+               ts.push_back (entered_track);
+       }
+
+       for (nth = 0, i = ts.begin(); i != ts.end(); ++i, ++nth) {
+
                /* undo/redo is handled by individual tracks */
 
                if ((*i)->paste (position, times, *cut_buffer, nth)) {
                        commit = true;
                }
        }
-
+       
        if (commit) {
                commit_reversible_command ();
        }
@@ -3130,18 +3865,20 @@ Editor::paste_named_selection (float times)
        chunk = ns->playlists.begin();
                
        begin_reversible_command (_("paste chunk"));
+       
+       sort_track_selection ();
 
        for (t = selection->tracks.begin(); t != selection->tracks.end(); ++t) {
                
-               AudioTimeAxisView* atv;
+               RouteTimeAxisView* rtv;
                boost::shared_ptr<Playlist> pl;
                boost::shared_ptr<AudioPlaylist> apl;
 
-               if ((atv = dynamic_cast<AudioTimeAxisView*> (*t)) == 0) {
+               if ((rtv = dynamic_cast<RouteTimeAxisView*> (*t)) == 0) {
                        continue;
                }
 
-               if ((pl = atv->playlist()) == 0) {
+               if ((pl = rtv->playlist()) == 0) {
                        continue;
                }
                
@@ -3152,8 +3889,8 @@ Editor::paste_named_selection (float times)
                tmp = chunk;
                ++tmp;
 
-                XMLNode &before = apl->get_state();
-               apl->paste (*chunk, edit_cursor->current_frame, times);
+               XMLNode &before = apl->get_state();
+               apl->paste (*chunk, get_preferred_edit_position(), times);
                session->add_command(new MementoCommand<AudioPlaylist>(*apl, &before, &apl->get_state()));
 
                if (tmp != ns->playlists.end()) {
@@ -3168,8 +3905,9 @@ void
 Editor::duplicate_some_regions (RegionSelection& regions, float times)
 {
        boost::shared_ptr<Playlist> playlist; 
-       RegionSelection sel = regions; // clear (below) will clear the argument list
-               
+       RegionSelection sel = regions; // clear (below) may  clear the argument list if its the current region selection
+       RegionSelection foo;
+
        begin_reversible_command (_("duplicate region"));
 
        selection->clear_regions ();
@@ -3179,8 +3917,9 @@ Editor::duplicate_some_regions (RegionSelection& regions, float times)
                boost::shared_ptr<Region> r ((*i)->region());
 
                TimeAxisView& tv = (*i)->get_time_axis_view();
-               AudioTimeAxisView* atv = dynamic_cast<AudioTimeAxisView*> (&tv);
-               sigc::connection c = atv->view()->RegionViewAdded.connect (mem_fun(*this, &Editor::collect_new_region_view));
+               RouteTimeAxisView* rtv = dynamic_cast<RouteTimeAxisView*> (&tv);
+               latest_regionviews.clear ();
+               sigc::connection c = rtv->view()->RegionViewAdded.connect (mem_fun(*this, &Editor::collect_new_region_view));
                
                playlist = (*i)->region()->playlist();
                 XMLNode &before = playlist->get_state();
@@ -3188,14 +3927,15 @@ Editor::duplicate_some_regions (RegionSelection& regions, float times)
                session->add_command(new MementoCommand<Playlist>(*playlist, &before, &playlist->get_state()));
 
                c.disconnect ();
-
-               if (latest_regionview) {
-                       selection->add (latest_regionview);
-               }
-       }
                
+               foo.insert (foo.end(), latest_regionviews.begin(), latest_regionviews.end());
+       }
 
        commit_reversible_command ();
+
+       if (!foo.empty()) {
+               selection->set (foo);
+       }
 }
 
 void
@@ -3242,8 +3982,6 @@ Editor::reset_point_selection ()
 {
        /* reset all selected points to the relevant default value */
 
-       cerr << "point selection has " << selection->points.size() << " entries\n";
-       
        for (PointSelection::iterator i = selection->points.begin(); i != selection->points.end(); ++i) {
                
                AutomationTimeAxisView* atv = dynamic_cast<AutomationTimeAxisView*>(&(*i).track);
@@ -3258,16 +3996,14 @@ void
 Editor::center_playhead ()
 {
        float page = canvas_width * frames_per_unit;
-
        center_screen_internal (playhead_cursor->current_frame, page);
 }
 
 void
-Editor::center_edit_cursor ()
+Editor::center_edit_point ()
 {
        float page = canvas_width * frames_per_unit;
-
-       center_screen_internal (edit_cursor->current_frame, page);
+       center_screen_internal (get_preferred_edit_position(), page);
 }
 
 void
@@ -3282,15 +4018,15 @@ Editor::clear_playlist (boost::shared_ptr<Playlist> playlist)
 }
 
 void
-Editor::nudge_track (bool use_edit_cursor, bool forwards)
+Editor::nudge_track (bool use_edit, bool forwards)
 {
        boost::shared_ptr<Playlist> playlist; 
        nframes_t distance;
        nframes_t next_distance;
        nframes_t start;
 
-       if (use_edit_cursor) {
-               start = edit_cursor->current_frame;
+       if (use_edit) {
+               start = get_preferred_edit_position();
        } else {
                start = 0;
        }
@@ -3385,6 +4121,8 @@ Editor::denormalize_region ()
                return;
        }
 
+       ExclusiveRegionSelection (*this, entered_regionview);
+
        if (selection->regions.empty()) {
                return;
        }
@@ -3403,55 +4141,129 @@ Editor::denormalize_region ()
        commit_reversible_command ();
 }
 
-
 void
-Editor::reverse_region ()
+Editor::adjust_region_scale_amplitude (bool up)
 {
        if (!session) {
                return;
        }
 
-       Reverse rev (*session);
-       apply_filter (rev, _("reverse regions"));
-}
+       ExclusiveRegionSelection (*this, entered_regionview);
 
-void
-Editor::apply_filter (AudioFilter& filter, string command)
-{
        if (selection->regions.empty()) {
                return;
        }
 
-       begin_reversible_command (command);
-
-       track_canvas.get_window()->set_cursor (*wait_cursor);
-       gdk_flush ();
+       begin_reversible_command ("denormalize");
 
-       for (RegionSelection::iterator r = selection->regions.begin(); r != selection->regions.end(); ) {
+       for (RegionSelection::iterator r = selection->regions.begin(); r != selection->regions.end(); ++r) {
                AudioRegionView* const arv = dynamic_cast<AudioRegionView*>(*r);
                if (!arv)
                        continue;
-
-               boost::shared_ptr<Playlist> playlist = arv->region()->playlist();
-
-               RegionSelection::iterator tmp;
+               XMLNode &before = arv->region()->get_state();
                
-               tmp = r;
-               ++tmp;
-
-               if (arv->audio_region()->apply (filter) == 0) {
+               double fraction = gain_to_slider_position (arv->audio_region()->scale_amplitude ());
+               
+               cerr << "slider pos for " << arv->audio_region()->scale_amplitude ()
+                    << " = " << fraction
+                    << endl;
 
-                        XMLNode &before = playlist->get_state();
-                       playlist->replace_region (arv->region(), filter.results.front(), arv->region()->position());
-                        XMLNode &after = playlist->get_state();
-                       session->add_command(new MementoCommand<Playlist>(*playlist, &before, &after));
+               if (up) {
+                       fraction += 0.05;
+                       fraction = min (fraction, 1.0);
                } else {
-                       goto out;
+                       fraction -= 0.05;
+                       fraction = max (fraction, 0.0);
                }
 
-               r = tmp;
-       }
-
+               if (!up && fraction <= 0) {
+                       continue;
+               }
+
+               if (up && fraction >= 1.0) {
+                       continue;
+               }
+
+               fraction = slider_position_to_gain (fraction);
+               fraction = coefficient_to_dB (fraction);
+               fraction = dB_to_coefficient (fraction);
+               
+               cerr << "set scale amp for " << arv->audio_region()->name() << " to " << fraction << endl;
+
+               arv->audio_region()->set_scale_amplitude (fraction);
+               session->add_command (new MementoCommand<Region>(*(arv->region().get()), &before, &arv->region()->get_state()));
+       }
+
+       commit_reversible_command ();
+}
+
+
+void
+Editor::reverse_region ()
+{
+       if (!session) {
+               return;
+       }
+
+       Reverse rev (*session);
+       apply_filter (rev, _("reverse regions"));
+}
+
+
+void
+Editor::quantize_region ()
+{
+       if (!session) {
+               return;
+       }
+
+       // FIXME: varying meter?
+       Quantize quant (*session, snap_length_beats(0));
+       apply_filter (quant, _("quantize regions"));
+}
+
+void
+Editor::apply_filter (Filter& filter, string command)
+{
+       if (selection->regions.empty()) {
+               return;
+       }
+
+       begin_reversible_command (command);
+
+       track_canvas.get_window()->set_cursor (*wait_cursor);
+       gdk_flush ();
+
+       /* this is ugly. */
+       for (RegionSelection::iterator r = selection->regions.begin(); r != selection->regions.end(); ) {
+               RegionSelection::iterator tmp = r;
+               ++tmp;
+
+               MidiRegionView* const mrv = dynamic_cast<MidiRegionView*>(*r);
+               if (mrv) {
+                       if (mrv->midi_region()->apply(filter) == 0) {
+                               mrv->redisplay_model();
+                       }
+               }
+
+               AudioRegionView* const arv = dynamic_cast<AudioRegionView*>(*r);
+               if (arv) {
+                       boost::shared_ptr<Playlist> playlist = arv->region()->playlist();
+
+                       if (arv->audio_region()->apply (filter) == 0) {
+
+                               XMLNode &before = playlist->get_state();
+                               playlist->replace_region (arv->region(), filter.results.front(), arv->region()->position());
+                               XMLNode &after = playlist->get_state();
+                               session->add_command(new MementoCommand<Playlist>(*playlist, &before, &after));
+                       } else {
+                               goto out;
+                       }
+               }
+
+               r = tmp;
+       }
+
        commit_reversible_command ();
        selection->regions.clear ();
 
@@ -3490,10 +4302,6 @@ Editor::region_selection_op (void (Region::*pmf)(bool), bool yn)
 void
 Editor::external_edit_region ()
 {
-       if (!clicked_regionview) {
-               return;
-       }
-
        /* more to come */
 }
 
@@ -3530,11 +4338,11 @@ Editor::reset_region_gain_envelopes ()
        for (RegionSelection::iterator i = selection->regions.begin(); i != selection->regions.end(); ++i) {
                AudioRegionView* const arv = dynamic_cast<AudioRegionView*>(*i);
                if (arv) {
-                       AutomationList& alist (arv->audio_region()->envelope());
-                       XMLNode& before (alist.get_state());
+                       boost::shared_ptr<AutomationList> alist (arv->audio_region()->envelope());
+                       XMLNode& before (alist->get_state());
 
                        arv->audio_region()->set_default_envelope ();
-                       session->add_command (new MementoCommand<AutomationList>(arv->audio_region()->envelope(), &before, &alist.get_state()));
+                       session->add_command (new MementoCommand<AutomationList>(*arv->audio_region()->envelope().get(), &before, &alist->get_state()));
                }
        }
 
@@ -3547,10 +4355,7 @@ Editor::toggle_gain_envelope_visibility ()
        for (RegionSelection::iterator i = selection->regions.begin(); i != selection->regions.end(); ++i) {
                AudioRegionView* const arv = dynamic_cast<AudioRegionView*>(*i);
                if (arv) {
-                       bool x = region_envelope_visible_item->get_active();
-                       if (x != arv->envelope_visible()) {
-                               arv->set_envelope_visible (x);
-                       }
+                       arv->set_envelope_visible (!arv->envelope_visible());
                }
        }
 }
@@ -3561,10 +4366,7 @@ Editor::toggle_gain_envelope_active ()
        for (RegionSelection::iterator i = selection->regions.begin(); i != selection->regions.end(); ++i) {
                AudioRegionView* const arv = dynamic_cast<AudioRegionView*>(*i);
                if (arv) {
-                       bool x = region_envelope_active_item->get_active();
-                       if (x != arv->audio_region()->envelope_active()) {
-                               arv->audio_region()->set_envelope_active (x);
-                       }
+                       arv->audio_region()->set_envelope_active (!arv->audio_region()->envelope_active());
                }
        }
 }
@@ -3573,13 +4375,7 @@ void
 Editor::toggle_region_lock ()
 {
        for (RegionSelection::iterator i = selection->regions.begin(); i != selection->regions.end(); ++i) {
-               AudioRegionView* const arv = dynamic_cast<AudioRegionView*>(*i);
-               if (arv) {
-                       bool x = region_lock_item->get_active();
-                       if (x != arv->audio_region()->locked()) {
-                               arv->audio_region()->set_locked (x);
-                       }
-               }
+               (*i)->region()->set_locked (!(*i)->region()->locked());
        }
 }
 
@@ -3587,13 +4383,7 @@ void
 Editor::toggle_region_mute ()
 {
        for (RegionSelection::iterator i = selection->regions.begin(); i != selection->regions.end(); ++i) {
-               AudioRegionView* const arv = dynamic_cast<AudioRegionView*>(*i);
-               if (arv) {
-                       bool x = region_mute_item->get_active();
-                       if (x != arv->audio_region()->muted()) {
-                               arv->audio_region()->set_muted (x);
-                       }
-               }
+               (*i)->region()->set_muted (!(*i)->region()->muted());
        }
 }
 
@@ -3601,12 +4391,598 @@ void
 Editor::toggle_region_opaque ()
 {
        for (RegionSelection::iterator i = selection->regions.begin(); i != selection->regions.end(); ++i) {
-               AudioRegionView* const arv = dynamic_cast<AudioRegionView*>(*i);
-               if (arv) {
-                       bool x = region_opaque_item->get_active();
-                       if (x != arv->audio_region()->opaque()) {
-                               arv->audio_region()->set_opaque (x);
+               (*i)->region()->set_opaque (!(*i)->region()->opaque());
+       }
+}
+
+void
+Editor::set_fade_length (bool in)
+{
+       ExclusiveRegionSelection esr (*this, entered_regionview);
+
+       /* we need a region to measure the offset from the start */
+
+       RegionView* rv;
+
+       if (!selection->regions.empty()) {
+               rv = selection->regions.front();
+       } else if (entered_regionview) {
+               rv = entered_regionview;
+       } else {
+               return;
+       }
+
+       nframes64_t pos = get_preferred_edit_position();
+       nframes_t len;
+       char* cmd;
+       
+       if (pos > rv->region()->last_frame() || pos < rv->region()->first_frame()) {
+               /* edit point is outside the relevant region */
+               return;
+       }
+
+       if (in) {
+               if (pos <= rv->region()->position()) {
+                       /* can't do it */
+                       return;
+               }
+               len = pos - rv->region()->position();
+               cmd = _("set fade in length");
+       } else {
+               if (pos >= rv->region()->last_frame()) {
+                       /* can't do it */
+                       return;
+               }
+               len = rv->region()->last_frame() - pos;
+               cmd = _("set fade out length");
+       }
+
+       begin_reversible_command (cmd);
+
+       RegionSelection& rs (get_regions_for_action());
+
+       for (RegionSelection::iterator x = rs.begin(); x != rs.end(); ++x) {
+               AudioRegionView* tmp = dynamic_cast<AudioRegionView*> (*x);
+
+               if (!tmp) {
+                       return;
+               }
+
+               boost::shared_ptr<AutomationList> alist;
+               if (in) {
+                       alist = tmp->audio_region()->fade_in();
+               } else {
+                       alist = tmp->audio_region()->fade_out();
+               }
+
+               XMLNode &before = alist->get_state();
+
+               if (in) {
+                       tmp->audio_region()->set_fade_in_length (len);
+               } else {
+                       tmp->audio_region()->set_fade_out_length (len);
+               }
+               
+               XMLNode &after = alist->get_state();
+               session->add_command(new MementoCommand<AutomationList>(*alist, &before, &after));
+       }
+
+       commit_reversible_command ();
+}
+
+void
+Editor::toggle_fade_active (bool in)
+{
+       ensure_entered_region_selected (true);
+
+       if (selection->regions.empty()) {
+               return;
+       }
+
+       const char* cmd = (in ? _("toggle fade in active") : _("toggle fade out active"));
+       bool have_switch = false;
+       bool yn;
+
+       begin_reversible_command (cmd);
+
+       for (RegionSelection::iterator x = selection->regions.begin(); x != selection->regions.end(); ++x) {
+               AudioRegionView* tmp = dynamic_cast<AudioRegionView*> (*x);
+               
+               if (!tmp) {
+                       return;
+               }
+
+               boost::shared_ptr<AudioRegion> region (tmp->audio_region());
+
+               /* make the behaviour consistent across all regions */
+               
+               if (!have_switch) {
+                       if (in) {
+                               yn = region->fade_in_active();
+                       } else {
+                               yn = region->fade_out_active();
+                       }
+                       have_switch = true;
+               }
+
+               XMLNode &before = region->get_state();
+               if (in) {
+                       region->set_fade_in_active (!yn);
+               } else {
+                       region->set_fade_out_active (!yn);
+               }
+               XMLNode &after = region->get_state();
+               session->add_command(new MementoCommand<AudioRegion>(*region.get(), &before, &after));
+       }
+
+       commit_reversible_command ();
+}
+
+void
+Editor::set_fade_in_shape (AudioRegion::FadeShape shape)
+{
+
+       begin_reversible_command (_("set fade in shape"));
+
+       for (RegionSelection::iterator x = selection->regions.begin(); x != selection->regions.end(); ++x) {
+               AudioRegionView* tmp = dynamic_cast<AudioRegionView*> (*x);
+
+               if (!tmp) {
+                       return;
+               }
+
+               boost::shared_ptr<AutomationList> alist = tmp->audio_region()->fade_in();
+               XMLNode &before = alist->get_state();
+
+               tmp->audio_region()->set_fade_in_shape (shape);
+               
+               XMLNode &after = alist->get_state();
+               session->add_command(new MementoCommand<AutomationList>(*alist.get(), &before, &after));
+       }
+
+       commit_reversible_command ();
+               
+}
+
+void
+Editor::set_fade_out_shape (AudioRegion::FadeShape shape)
+{
+       begin_reversible_command (_("set fade out shape"));
+
+       for (RegionSelection::iterator x = selection->regions.begin(); x != selection->regions.end(); ++x) {
+               AudioRegionView* tmp = dynamic_cast<AudioRegionView*> (*x);
+
+               if (!tmp) {
+                       return;
+               }
+
+               boost::shared_ptr<AutomationList> alist = tmp->audio_region()->fade_out();
+               XMLNode &before = alist->get_state();
+
+               tmp->audio_region()->set_fade_out_shape (shape);
+               
+               XMLNode &after = alist->get_state();
+               session->add_command(new MementoCommand<AutomationList>(*alist.get(), &before, &after));
+       }
+
+       commit_reversible_command ();
+}
+
+void
+Editor::set_fade_in_active (bool yn)
+{
+       begin_reversible_command (_("set fade in active"));
+
+       for (RegionSelection::iterator x = selection->regions.begin(); x != selection->regions.end(); ++x) {
+               AudioRegionView* tmp = dynamic_cast<AudioRegionView*> (*x);
+
+               if (!tmp) {
+                       return;
+               }
+
+
+               boost::shared_ptr<AudioRegion> ar (tmp->audio_region());
+
+               XMLNode &before = ar->get_state();
+
+               ar->set_fade_in_active (yn);
+               
+               XMLNode &after = ar->get_state();
+               session->add_command(new MementoCommand<AudioRegion>(*ar, &before, &after));
+       }
+
+       commit_reversible_command ();
+}
+
+void
+Editor::set_fade_out_active (bool yn)
+{
+       begin_reversible_command (_("set fade out active"));
+
+       for (RegionSelection::iterator x = selection->regions.begin(); x != selection->regions.end(); ++x) {
+               AudioRegionView* tmp = dynamic_cast<AudioRegionView*> (*x);
+
+               if (!tmp) {
+                       return;
+               }
+
+               boost::shared_ptr<AudioRegion> ar (tmp->audio_region());
+
+               XMLNode &before = ar->get_state();
+
+               ar->set_fade_out_active (yn);
+               
+               XMLNode &after = ar->get_state();
+               session->add_command(new MementoCommand<AudioRegion>(*ar, &before, &after));
+       }
+
+       commit_reversible_command ();
+}
+
+
+/** Update crossfade visibility after its configuration has been changed */
+void
+Editor::update_xfade_visibility ()
+{
+       _xfade_visibility = Config->get_xfades_visible ();
+       
+       for (TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) {
+               AudioTimeAxisView* v = dynamic_cast<AudioTimeAxisView*>(*i);
+               if (v) {
+                       if (_xfade_visibility) {
+                               v->show_all_xfades ();
+                       } else {
+                               v->hide_all_xfades ();
                        }
                }
        }
 }
+
+void
+Editor::set_edit_point ()
+{
+       nframes64_t where;
+       bool ignored;
+
+       if (!mouse_frame (where, ignored)) {
+               return;
+       }
+       
+       snap_to (where);
+
+       if (selection->markers.empty()) {
+               
+               mouse_add_new_marker (where);
+
+       } else {
+               bool ignored;
+
+               Location* loc = find_location_from_marker (selection->markers.front(), ignored);
+
+               if (loc) {
+                       loc->move_to (where);
+               }
+       }
+}
+
+void
+Editor::set_playhead_cursor ()
+{
+       if (entered_marker) {
+               session->request_locate (entered_marker->position(), session->transport_rolling());
+       } else {
+               nframes64_t where;
+               bool ignored;
+
+               if (!mouse_frame (where, ignored)) {
+                       return;
+               }
+                       
+               snap_to (where);
+               
+               if (session) {
+                       session->request_locate (where, session->transport_rolling());
+               }
+       }
+}
+
+void
+Editor::split ()
+{
+       ensure_entered_region_selected ();
+       
+       nframes64_t where = get_preferred_edit_position();
+
+       if (!selection->regions.empty()) {
+               
+               split_regions_at (where, selection->regions);
+
+       } else {
+               
+               RegionSelection rs;
+               rs = get_regions_at (where, selection->tracks);
+               split_regions_at (where, rs);
+       }
+}
+
+void
+Editor::ensure_entered_track_selected (bool op_really_wants_one_track_if_none_are_selected)
+{
+       if (entered_track && mouse_mode == MouseObject) {
+               if (!selection->tracks.empty()) {
+                       if (!selection->selected (entered_track)) {
+                               selection->add (entered_track);
+                       }
+               } else {
+                       /* there is no selection, but this operation requires/prefers selected objects */
+
+                       if (op_really_wants_one_track_if_none_are_selected) {
+                               selection->set (entered_track);
+                       }
+               }
+       }
+}
+
+void
+Editor::ensure_entered_region_selected (bool op_really_wants_one_region_if_none_are_selected)
+{
+       if (!entered_regionview || mouse_mode != MouseObject) {
+               return;
+       }
+
+       
+       /* heuristic:
+          
+       - if there is no existing selection, don't change it. the operation will thus apply to "all"
+       
+       - if there is an existing selection, but the entered regionview isn't in it, add it. this
+       avoids key-mouse ops on unselected regions from interfering with an existing selection,
+       but also means that the operation will apply to the pointed-at region.
+       */
+       
+       if (!selection->regions.empty()) {
+               if (!selection->selected (entered_regionview)) {
+                       selection->add (entered_regionview);
+               }
+       } else {
+               /* there is no selection, but this operation requires/prefers selected objects */
+               
+               if (op_really_wants_one_region_if_none_are_selected) {
+                       selection->set (entered_regionview, false);
+               }
+       }
+}
+
+void
+Editor::trim_region_front ()
+{
+       trim_region (true);
+}
+
+void
+Editor::trim_region_back ()
+{
+       trim_region (false);
+}
+
+void
+Editor::trim_region (bool front)
+{
+       ExclusiveRegionSelection ers (*this, entered_regionview);
+
+       nframes64_t where = get_preferred_edit_position();
+       RegionSelection& rs = get_regions_for_action ();
+
+       if (rs.empty()) {
+               return;
+       }
+
+       begin_reversible_command (front ? _("trim front") : _("trim back"));
+
+       for (list<RegionView*>::const_iterator i = rs.by_layer().begin(); i != rs.by_layer().end(); ++i) {
+               if (!(*i)->region()->locked()) {
+                       boost::shared_ptr<Playlist> pl = (*i)->region()->playlist();
+                       XMLNode &before = pl->get_state();
+                       if (front) {
+                               (*i)->region()->trim_front (where, this);       
+                       } else {
+                               (*i)->region()->trim_end (where, this); 
+                       }
+                       XMLNode &after = pl->get_state();
+                       session->add_command(new MementoCommand<Playlist>(*pl.get(), &before, &after));
+               }
+       }
+
+       commit_reversible_command ();
+}
+
+struct EditorOrderRouteSorter {
+    bool operator() (boost::shared_ptr<Route> a, boost::shared_ptr<Route> b) {
+           /* use of ">" forces the correct sort order */
+           return a->order_key ("editor") < b->order_key ("editor");
+    }
+};
+
+void
+Editor::select_next_route()
+{
+       if (selection->tracks.empty()) {
+               selection->set (track_views.front());
+               return;
+       }
+
+       TimeAxisView* current = selection->tracks.front();
+
+       for (TrackViewList::iterator i = track_views.begin(); i != track_views.end(); ++i) {
+               if (*i == current) {
+                       ++i;
+                       if (i != track_views.end()) {
+                               selection->set (*i);
+                       } else {
+                               selection->set (*(track_views.begin()));
+                       }
+                       break;
+               }
+       }
+}
+
+void
+Editor::select_prev_route()
+{
+       if (selection->tracks.empty()) {
+               selection->set (track_views.front());
+               return;
+       }
+
+       TimeAxisView* current = selection->tracks.front();
+
+       for (TrackViewList::reverse_iterator i = track_views.rbegin(); i != track_views.rend(); ++i) {
+               if (*i == current) {
+                       ++i;
+                       if (i != track_views.rend()) {
+                               selection->set (*i);
+                       } else {
+                               selection->set (*(track_views.rbegin()));
+                       }
+                       break;
+               }
+       }
+}
+
+void
+Editor::set_loop_from_selection (bool play)
+{
+       if (session == 0 || selection->time.empty()) {
+               return;
+       }
+
+       nframes_t start = selection->time[clicked_selection].start;
+       nframes_t end = selection->time[clicked_selection].end;
+       
+       set_loop_range (start, end,  _("set loop range from selection"));
+
+       if (play) {
+               session->request_play_loop (true);
+               session->request_locate (start, true);
+       }
+}
+
+void
+Editor::set_loop_from_edit_range (bool play)
+{
+       if (session == 0) {
+               return;
+       }
+
+       nframes64_t start;
+       nframes64_t end;
+       
+       if (!get_edit_op_range (start, end)) {
+               return;
+       }
+
+       set_loop_range (start, end,  _("set loop range from edit range"));
+
+       if (play) {
+               session->request_play_loop (true);
+               session->request_locate (start, true);
+       }
+}
+
+void
+Editor::set_loop_from_region (bool play)
+{
+       nframes64_t start = max_frames;
+       nframes64_t end = 0;
+
+       ExclusiveRegionSelection esr (*this, entered_regionview);
+
+       if (selection->regions.empty()) {
+               info << _("cannot set loop: no region selected") << endmsg;
+               return;
+       }
+
+       for (RegionSelection::iterator i = selection->regions.begin(); i != selection->regions.end(); ++i) {
+               if ((*i)->region()->position() < start) {
+                       start = (*i)->region()->position();
+               }
+               if ((*i)->region()->last_frame() + 1 > end) {
+                       end = (*i)->region()->last_frame() + 1;
+               }
+       }
+
+       set_loop_range (start, end, _("set loop range from region"));
+
+       if (play) {
+               session->request_play_loop (true);
+               session->request_locate (start, true);
+       }
+}
+
+void
+Editor::set_punch_from_selection ()
+{
+       if (session == 0 || selection->time.empty()) {
+               return;
+       }
+
+       nframes_t start = selection->time[clicked_selection].start;
+       nframes_t end = selection->time[clicked_selection].end;
+       
+       set_punch_range (start, end,  _("set punch range from selection"));
+}
+
+void
+Editor::set_punch_from_edit_range ()
+{
+       if (session == 0) {
+               return;
+       }
+
+       nframes64_t start;
+       nframes64_t end;
+       
+       if (!get_edit_op_range (start, end)) {
+               return;
+       }
+
+       set_punch_range (start, end,  _("set punch range from edit range"));
+}
+
+void
+Editor::set_punch_from_region ()
+{
+       nframes64_t start = max_frames;
+       nframes64_t end = 0;
+
+       ExclusiveRegionSelection esr (*this, entered_regionview);
+
+       if (selection->regions.empty()) {
+               info << _("cannot set punch: no region selected") << endmsg;
+               return;
+       }
+
+       for (RegionSelection::iterator i = selection->regions.begin(); i != selection->regions.end(); ++i) {
+               if ((*i)->region()->position() < start) {
+                       start = (*i)->region()->position();
+               }
+               if ((*i)->region()->last_frame() + 1 > end) {
+                       end = (*i)->region()->last_frame() + 1;
+               }
+       }
+
+       set_punch_range (start, end, _("set punch range from region"));
+}
+
+void
+Editor::pitch_shift_regions ()
+{
+       ensure_entered_region_selected (true);
+       
+       if (selection->regions.empty()) {
+               return;
+       }
+
+       pitch_shift (selection->regions, 1.2);
+}
+