Remove confusing track / group members in TimeSelection struct; time selection
[ardour.git] / gtk2_ardour / selection.cc
index 3c1aea1ec85caa30ff33236cf07a925c7e8311e8..da706a34d7a4b344866ebd98cadb1b716616e1b6 100644 (file)
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2002 Paul Davis 
+    Copyright (C) 2002 Paul Davis
 
     This program is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
     along with this program; if not, write to the Free Software
     Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
 
-    $Id$
 */
 
 #include <algorithm>
 #include <sigc++/bind.h>
-#include <pbd/error.h>
+#include "pbd/error.h"
+#include "pbd/stacktrace.h"
 
-#include <ardour/playlist.h>
+#include "ardour/playlist.h"
+#include "ardour/rc_configuration.h"
 
-#include "regionview.h"
+#include "midi_cut_buffer.h"
+#include "region_view.h"
 #include "selection.h"
 #include "selection_templates.h"
 #include "time_axis_view.h"
 #include "automation_time_axis.h"
+#include "public_editor.h"
 
 #include "i18n.h"
 
+using namespace std;
 using namespace ARDOUR;
-using namespace sigc;
+using namespace PBD;
 
 struct AudioRangeComparator {
     bool operator()(AudioRange a, AudioRange b) {
@@ -45,10 +49,12 @@ Selection&
 Selection::operator= (const Selection& other)
 {
        if (&other != this) {
-               audio_regions = other.audio_regions;
+               regions = other.regions;
                tracks = other.tracks;
                time = other.time;
                lines = other.lines;
+               midi_regions = other.midi_regions;
+               midi_notes = other.midi_notes;
        }
        return *this;
 }
@@ -56,52 +62,44 @@ Selection::operator= (const Selection& other)
 bool
 operator== (const Selection& a, const Selection& b)
 {
-       return a.audio_regions == b.audio_regions &&
+       return a.regions == b.regions &&
                a.tracks == b.tracks &&
-               a.time.track == b.time.track &&
-               a.time.group == b.time.group && 
                a.time == b.time &&
                a.lines == b.lines &&
                a.playlists == b.playlists &&
-               a.redirects == b.redirects;
+               a.midi_notes == b.midi_notes &&
+               a.midi_regions == b.midi_regions;
 }
 
+/** Clear everything from the Selection */
 void
 Selection::clear ()
 {
        clear_tracks ();
-       clear_audio_regions ();
+       clear_regions ();
        clear_points ();
        clear_lines();
        clear_time ();
        clear_playlists ();
-       clear_redirects ();
+       clear_midi_notes ();
+       clear_midi_regions ();
 }
 
 void
 Selection::dump_region_layers()
 {
        cerr << "region selection layer dump" << endl;
-       for (AudioRegionSelection::iterator i = audio_regions.begin(); i != audio_regions.end(); ++i) {
-               cerr << "layer: " << (int)(*i)->region.layer() << endl;
+       for (RegionSelection::iterator i = regions.begin(); i != regions.end(); ++i) {
+               cerr << "layer: " << (int)(*i)->region()->layer() << endl;
        }
 }
 
 
 void
-Selection::clear_redirects ()
+Selection::clear_regions ()
 {
-       if (!redirects.empty()) {
-               redirects.clear ();
-               RedirectsChanged ();
-       }
-}
-
-void
-Selection::clear_audio_regions ()
-{
-       if (!audio_regions.empty()) {
-               audio_regions.clear_all ();
+       if (!regions.empty()) {
+               regions.clear_all ();
                RegionsChanged();
        }
 }
@@ -115,11 +113,30 @@ Selection::clear_tracks ()
        }
 }
 
+void
+Selection::clear_midi_notes ()
+{
+       if (!midi_notes.empty()) {
+               for (MidiNoteSelection::iterator x = midi_notes.begin(); x != midi_notes.end(); ++x) {
+                       delete *x;
+               }
+               midi_notes.clear ();
+               MidiNotesChanged ();
+       }
+}
+
+void
+Selection::clear_midi_regions ()
+{
+       if (!midi_regions.empty()) {
+               midi_regions.clear ();
+               MidiRegionsChanged ();
+       }
+}
+
 void
 Selection::clear_time ()
 {
-       time.track = 0;
-       time.group = 0;
        time.clear();
 
        TimeChanged ();
@@ -131,7 +148,9 @@ Selection::clear_playlists ()
        /* Selections own their playlists */
 
        for (PlaylistSelection::iterator i = playlists.begin(); i != playlists.end(); ++i) {
-               (*i)->unref ();
+               /* selections own their own regions, which are copies of the "originals". make them go away */
+               (*i)->drop_regions ();
+               (*i)->release ();
        }
 
        if (!playlists.empty()) {
@@ -150,26 +169,21 @@ Selection::clear_lines ()
 }
 
 void
-Selection::toggle (Redirect* r)
+Selection::clear_markers ()
 {
-       RedirectSelection::iterator i;
-
-       if ((i = find (redirects.begin(), redirects.end(), r)) == redirects.end()) {
-               redirects.push_back (r);
-       } else {
-               redirects.erase (i);
+       if (!markers.empty()) {
+               markers.clear ();
+               MarkersChanged();
        }
-       RedirectsChanged();
-
 }
 
 void
-Selection::toggle (Playlist* pl)
+Selection::toggle (boost::shared_ptr<Playlist> pl)
 {
        PlaylistSelection::iterator i;
 
        if ((i = find (playlists.begin(), playlists.end(), pl)) == playlists.end()) {
-               pl->ref ();
+               pl->use ();
                playlists.push_back(pl);
        } else {
                playlists.erase (i);
@@ -178,14 +192,22 @@ Selection::toggle (Playlist* pl)
        PlaylistsChanged ();
 }
 
+void
+Selection::toggle (const TrackViewList& track_list)
+{
+       for (TrackViewList::const_iterator i = track_list.begin(); i != track_list.end(); ++i) {
+               toggle ((*i));
+       }
+}
+
 void
 Selection::toggle (TimeAxisView* track)
 {
        TrackSelection::iterator i;
-       
+
        if ((i = find (tracks.begin(), tracks.end(), track)) == tracks.end()) {
                void (Selection::*pmf)(TimeAxisView*) = &Selection::remove;
-               track->GoingAway.connect (sigc::bind (mem_fun (*this, pmf), track));
+               track->GoingAway.connect (sigc::bind (sigc::mem_fun (*this, pmf), track));
                tracks.push_back (track);
        } else {
                tracks.erase (i);
@@ -195,29 +217,68 @@ Selection::toggle (TimeAxisView* track)
 }
 
 void
-Selection::toggle (AudioRegionView* r)
+Selection::toggle (const MidiNoteSelection& midi_note_list)
+{
+       for (MidiNoteSelection::const_iterator i = midi_note_list.begin(); i != midi_note_list.end(); ++i) {
+               toggle ((*i));
+       }
+}
+
+void
+Selection::toggle (MidiCutBuffer* midi)
 {
-       AudioRegionSelection::iterator i;
+       MidiNoteSelection::iterator i;
 
-       if ((i = find (audio_regions.begin(), audio_regions.end(), r)) == audio_regions.end()) {
-               audio_regions.add (r);
+       if ((i = find (midi_notes.begin(), midi_notes.end(), midi)) == midi_notes.end()) {
+               midi_notes.push_back (midi);
        } else {
-               audio_regions.erase (i);
+               /* remember that we own the MCB */
+               delete *i;
+               midi_notes.erase (i);
+       }
+
+       MidiNotesChanged();
+}
+
+
+void
+Selection::toggle (RegionView* r)
+{
+       RegionSelection::iterator i;
+
+       if ((i = find (regions.begin(), regions.end(), r)) == regions.end()) {
+               add (r);
+       } else {
+               remove (*i);
        }
 
        RegionsChanged ();
 }
 
 void
-Selection::toggle (vector<AudioRegionView*>& r)
+Selection::toggle (MidiRegionView* mrv)
+{
+       MidiRegionSelection::iterator i;
+
+       if ((i = find (midi_regions.begin(), midi_regions.end(), mrv)) == midi_regions.end()) {
+               add (mrv);
+       } else {
+               midi_regions.erase (i);
+       }
+
+       MidiRegionsChanged ();
+}
+
+void
+Selection::toggle (vector<RegionView*>& r)
 {
-       AudioRegionSelection::iterator i;
+       RegionSelection::iterator i;
 
-       for (vector<AudioRegionView*>::iterator x = r.begin(); x != r.end(); ++x) {
-               if ((i = find (audio_regions.begin(), audio_regions.end(), (*x))) == audio_regions.end()) {
-                       audio_regions.add ((*x));
+       for (vector<RegionView*>::iterator x = r.begin(); x != r.end(); ++x) {
+               if ((i = find (regions.begin(), regions.end(), (*x))) == regions.end()) {
+                       add ((*x));
                } else {
-                       audio_regions.erase (i);
+                       remove (*x);
                }
        }
 
@@ -225,7 +286,7 @@ Selection::toggle (vector<AudioRegionView*>& r)
 }
 
 long
-Selection::toggle (jack_nframes_t start, jack_nframes_t end)
+Selection::toggle (nframes_t start, nframes_t end)
 {
        AudioRangeComparator cmp;
 
@@ -234,65 +295,51 @@ Selection::toggle (jack_nframes_t start, jack_nframes_t end)
        time.push_back (AudioRange (start, end, next_time_id++));
        time.consolidate ();
        time.sort (cmp);
-       
+
        TimeChanged ();
 
        return next_time_id - 1;
 }
 
-
-void
-Selection::add (Redirect* r)
-{
-       if (find (redirects.begin(), redirects.end(), r) == redirects.end()) {
-               redirects.push_back (r);
-               RedirectsChanged();
-       }
-}
-
 void
-Selection::add (Playlist* pl)
+Selection::add (boost::shared_ptr<Playlist> pl)
 {
        if (find (playlists.begin(), playlists.end(), pl) == playlists.end()) {
-               pl->ref ();
+               pl->use ();
                playlists.push_back(pl);
                PlaylistsChanged ();
        }
 }
 
 void
-Selection::add (const list<Playlist*>& pllist)
+Selection::add (const list<boost::shared_ptr<Playlist> >& pllist)
 {
        bool changed = false;
 
-       for (list<Playlist*>::const_iterator i = pllist.begin(); i != pllist.end(); ++i) {
+       for (list<boost::shared_ptr<Playlist> >::const_iterator i = pllist.begin(); i != pllist.end(); ++i) {
                if (find (playlists.begin(), playlists.end(), (*i)) == playlists.end()) {
-                       (*i)->ref ();
+                       (*i)->use ();
                        playlists.push_back (*i);
                        changed = true;
                }
        }
-       
+
        if (changed) {
                PlaylistsChanged ();
        }
 }
 
 void
-Selection::add (const list<TimeAxisView*>& track_list)
+Selection::add (const TrackViewList& track_list)
 {
-       bool changed = false;
+       TrackViewList added = tracks.add (track_list);
 
-       for (list<TimeAxisView*>::const_iterator i = track_list.begin(); i != track_list.end(); ++i) {
-               if (find (tracks.begin(), tracks.end(), (*i)) == tracks.end()) {
-                       void (Selection::*pmf)(TimeAxisView*) = &Selection::remove;
-                       (*i)->GoingAway.connect (sigc::bind (mem_fun (*this, pmf), (*i)));
-                       tracks.push_back (*i);
-                       changed = true;
-               }
+       for (list<TimeAxisView*>::const_iterator i = added.begin(); i != added.end(); ++i) {
+               void (Selection::*pmf)(TimeAxisView*) = &Selection::remove;
+               (*i)->GoingAway.connect (sigc::bind (sigc::mem_fun (*this, pmf), (*i)));
        }
-       
-       if (changed) {
+
+       if (!added.empty()) {
                TracksChanged ();
        }
 }
@@ -300,32 +347,70 @@ Selection::add (const list<TimeAxisView*>& track_list)
 void
 Selection::add (TimeAxisView* track)
 {
-       if (find (tracks.begin(), tracks.end(), track) == tracks.end()) {
-               void (Selection::*pmf)(TimeAxisView*) = &Selection::remove;
-               track->GoingAway.connect (sigc::bind (mem_fun (*this, pmf), track));
-               tracks.push_back (track);
-               TracksChanged();
+       TrackViewList tr;
+       tr.push_back (track);
+       add (tr);
+}
+
+void
+Selection::add (const MidiNoteSelection& midi_list)
+{
+       const MidiNoteSelection::const_iterator b = midi_list.begin();
+       const MidiNoteSelection::const_iterator e = midi_list.end();
+
+       if (!midi_list.empty()) {
+               midi_notes.insert (midi_notes.end(), b, e);
+               MidiNotesChanged ();
+       }
+}
+
+void
+Selection::add (MidiCutBuffer* midi)
+{
+       /* we take ownership of the MCB */
+
+       if (find (midi_notes.begin(), midi_notes.end(), midi) == midi_notes.end()) {
+               midi_notes.push_back (midi);
+               MidiNotesChanged ();
        }
 }
 
 void
-Selection::add (AudioRegionView* r)
+Selection::add (vector<RegionView*>& v)
 {
-       if (find (audio_regions.begin(), audio_regions.end(), r) == audio_regions.end()) {
-               audio_regions.add (r);
+       /* XXX This method or the add (const RegionSelection&) needs to go
+        */
+
+       bool changed = false;
+
+       for (vector<RegionView*>::iterator i = v.begin(); i != v.end(); ++i) {
+               if (find (regions.begin(), regions.end(), (*i)) == regions.end()) {
+                       changed = regions.add ((*i));
+                       if (Config->get_link_region_and_track_selection() && changed) {
+                               add (&(*i)->get_trackview());
+                       }
+               }
+       }
+
+       if (changed) {
                RegionsChanged ();
        }
 }
 
 void
-Selection::add (vector<AudioRegionView*>& v)
+Selection::add (const RegionSelection& rs)
 {
+       /* XXX This method or the add (const vector<RegionView*>&) needs to go
+        */
+
        bool changed = false;
 
-       for (vector<AudioRegionView*>::iterator i = v.begin(); i != v.end(); ++i) {
-               if (find (audio_regions.begin(), audio_regions.end(), (*i)) == audio_regions.end()) {
-                       audio_regions.add ((*i));
-                       changed = true;
+       for (RegionSelection::const_iterator i = rs.begin(); i != rs.end(); ++i) {
+               if (find (regions.begin(), regions.end(), (*i)) == regions.end()) {
+                       changed = regions.add ((*i));
+                       if (Config->get_link_region_and_track_selection() && changed) {
+                               add (&(*i)->get_trackview());
+                       }
                }
        }
 
@@ -334,8 +419,35 @@ Selection::add (vector<AudioRegionView*>& v)
        }
 }
 
+void
+Selection::add (RegionView* r)
+{
+       if (find (regions.begin(), regions.end(), r) == regions.end()) {
+               regions.add (r);
+               if (Config->get_link_region_and_track_selection()) {
+                       add (&r->get_trackview());
+               }
+               RegionsChanged ();
+       }
+}
+
+void
+Selection::add (MidiRegionView* mrv)
+{
+       if (find (midi_regions.begin(), midi_regions.end(), mrv) == midi_regions.end()) {
+               midi_regions.push_back (mrv);
+               /* XXX should we do this? */
+#if 0
+               if (Config->get_link_region_and_track_selection()) {
+                       add (&mrv->get_trackview());
+               }
+#endif
+               MidiRegionsChanged ();
+       }
+}
+
 long
-Selection::add (jack_nframes_t start, jack_nframes_t end)
+Selection::add (nframes_t start, nframes_t end)
 {
        AudioRangeComparator cmp;
 
@@ -344,14 +456,14 @@ Selection::add (jack_nframes_t start, jack_nframes_t end)
        time.push_back (AudioRange (start, end, next_time_id++));
        time.consolidate ();
        time.sort (cmp);
-       
+
        TimeChanged ();
 
        return next_time_id - 1;
 }
 
 void
-Selection::replace (uint32_t sid, jack_nframes_t start, jack_nframes_t end)
+Selection::replace (uint32_t sid, nframes_t start, nframes_t end)
 {
        for (list<AudioRange>::iterator i = time.begin(); i != time.end(); ++i) {
                if ((*i).id == sid) {
@@ -371,21 +483,18 @@ Selection::replace (uint32_t sid, jack_nframes_t start, jack_nframes_t end)
 }
 
 void
-Selection::add (AutomationList* ac)
+Selection::add (boost::shared_ptr<Evoral::ControlList> cl)
 {
-       if (find (lines.begin(), lines.end(), ac) == lines.end()) {
-               lines.push_back (ac);
-               LinesChanged();
+       boost::shared_ptr<ARDOUR::AutomationList> al
+               = boost::dynamic_pointer_cast<ARDOUR::AutomationList>(cl);
+       if (!al) {
+               warning << "Programming error: Selected list is not an ARDOUR::AutomationList" << endmsg;
+               return;
+               return;
        }
-}
-
-void
-Selection::remove (Redirect* r)
-{
-       list<Redirect*>::iterator i;
-       if ((i = find (redirects.begin(), redirects.end(), r)) != redirects.end()) {
-               redirects.erase (i);
-               RedirectsChanged ();
+       if (find (lines.begin(), lines.end(), al) == lines.end()) {
+               lines.push_back (al);
+               LinesChanged();
        }
 }
 
@@ -400,15 +509,14 @@ Selection::remove (TimeAxisView* track)
 }
 
 void
-Selection::remove (const list<TimeAxisView*>& track_list)
+Selection::remove (const TrackViewList& track_list)
 {
        bool changed = false;
 
-       for (list<TimeAxisView*>::const_iterator i = track_list.begin(); i != track_list.end(); ++i) {
-
-               list<TimeAxisView*>::iterator x;
+       for (TrackViewList::const_iterator i = track_list.begin(); i != track_list.end(); ++i) {
 
-               if ((x = find (tracks.begin(), tracks.end(), (*i))) != tracks.end()) {
+               TrackViewList::iterator x = find (tracks.begin(), tracks.end(), *i);
+               if (x != tracks.end()) {
                        tracks.erase (x);
                        changed = true;
                }
@@ -420,9 +528,42 @@ Selection::remove (const list<TimeAxisView*>& track_list)
 }
 
 void
-Selection::remove (Playlist* track)
+Selection::remove (const MidiNoteSelection& midi_list)
 {
-       list<Playlist*>::iterator i;
+       bool changed = false;
+
+       for (MidiNoteSelection::const_iterator i = midi_list.begin(); i != midi_list.end(); ++i) {
+
+               MidiNoteSelection::iterator x;
+
+               if ((x = find (midi_notes.begin(), midi_notes.end(), (*i))) != midi_notes.end()) {
+                       midi_notes.erase (x);
+                       changed = true;
+               }
+       }
+
+       if (changed) {
+               MidiNotesChanged();
+       }
+}
+
+void
+Selection::remove (MidiCutBuffer* midi)
+{
+       MidiNoteSelection::iterator x;
+
+       if ((x = find (midi_notes.begin(), midi_notes.end(), midi)) != midi_notes.end()) {
+               /* remember that we own the MCB */
+               delete *x;
+               midi_notes.erase (x);
+               MidiNotesChanged ();
+       }
+}
+
+void
+Selection::remove (boost::shared_ptr<Playlist> track)
+{
+       list<boost::shared_ptr<Playlist> >::iterator i;
        if ((i = find (playlists.begin(), playlists.end(), track)) != playlists.end()) {
                playlists.erase (i);
                PlaylistsChanged();
@@ -430,13 +571,13 @@ Selection::remove (Playlist* track)
 }
 
 void
-Selection::remove (const list<Playlist*>& pllist)
+Selection::remove (const list<boost::shared_ptr<Playlist> >& pllist)
 {
        bool changed = false;
 
-       for (list<Playlist*>::const_iterator i = pllist.begin(); i != pllist.end(); ++i) {
+       for (list<boost::shared_ptr<Playlist> >::const_iterator i = pllist.begin(); i != pllist.end(); ++i) {
 
-               list<Playlist*>::iterator x;
+               list<boost::shared_ptr<Playlist> >::iterator x;
 
                if ((x = find (playlists.begin(), playlists.end(), (*i))) != playlists.end()) {
                        playlists.erase (x);
@@ -450,10 +591,33 @@ Selection::remove (const list<Playlist*>& pllist)
 }
 
 void
-Selection::remove (AudioRegionView* r)
+Selection::remove (RegionView* r)
 {
-       audio_regions.remove (r);
-       RegionsChanged ();
+       if (regions.remove (r)) {
+               RegionsChanged ();
+       }
+
+       if (Config->get_link_region_and_track_selection() && !regions.involves (r->get_trackview())) {
+               remove (&r->get_trackview());
+       }
+}
+
+void
+Selection::remove (MidiRegionView* mrv)
+{
+       MidiRegionSelection::iterator x;
+
+       if ((x = find (midi_regions.begin(), midi_regions.end(), mrv)) != midi_regions.end()) {
+               midi_regions.erase (x);
+               MidiRegionsChanged ();
+       }
+
+#if 0
+       /* XXX fix this up ? */
+       if (Config->get_link_region_and_track_selection() && !regions.involves (r->get_trackview())) {
+               remove (&r->get_trackview());
+       }
+#endif
 }
 
 
@@ -467,7 +631,7 @@ Selection::remove (uint32_t selection_id)
        for (list<AudioRange>::iterator i = time.begin(); i != time.end(); ++i) {
                if ((*i).id == selection_id) {
                        time.erase (i);
-                                               
+
                        TimeChanged ();
                        break;
                }
@@ -475,27 +639,20 @@ Selection::remove (uint32_t selection_id)
 }
 
 void
-Selection::remove (jack_nframes_t start, jack_nframes_t end)
+Selection::remove (nframes_t /*start*/, nframes_t /*end*/)
 {
 }
 
 void
-Selection::remove (AutomationList *ac)
+Selection::remove (boost::shared_ptr<ARDOUR::AutomationList> ac)
 {
-       list<AutomationList*>::iterator i;
+       AutomationSelection::iterator i;
        if ((i = find (lines.begin(), lines.end(), ac)) != lines.end()) {
                lines.erase (i);
                LinesChanged();
        }
 }
 
-void
-Selection::set (Redirect *r)
-{
-       clear_redirects ();
-       add (r);
-}
-
 void
 Selection::set (TimeAxisView* track)
 {
@@ -504,45 +661,75 @@ Selection::set (TimeAxisView* track)
 }
 
 void
-Selection::set (const list<TimeAxisView*>& track_list)
+Selection::set (const TrackViewList& track_list)
 {
        clear_tracks ();
        add (track_list);
 }
 
 void
-Selection::set (Playlist* playlist)
+Selection::set (const MidiNoteSelection& midi_list)
+{
+       clear_midi_notes ();
+       add (midi_list);
+}
+
+void
+Selection::set (boost::shared_ptr<Playlist> playlist)
 {
        clear_playlists ();
        add (playlist);
 }
 
 void
-Selection::set (const list<Playlist*>& pllist)
+Selection::set (const list<boost::shared_ptr<Playlist> >& pllist)
 {
        clear_playlists ();
        add (pllist);
 }
 
 void
-Selection::set (AudioRegionView* r)
+Selection::set (const RegionSelection& rs)
 {
-       clear_audio_regions ();
-       add (r);
+       clear_regions();
+       regions = rs;
+       RegionsChanged(); /* EMIT SIGNAL */
+}
+
+void
+Selection::set (MidiRegionView* mrv)
+{
+       clear_midi_regions ();
+       add (mrv);
 }
 
 void
-Selection::set (vector<AudioRegionView*>& v)
+Selection::set (RegionView* r, bool also_clear_tracks)
 {
+       clear_regions ();
+       if (also_clear_tracks) {
+               clear_tracks ();
+       }
+       add (r);
+}
 
-       clear_audio_regions ();
-       // make sure to deselect any automation selections
-       clear_points();
+void
+Selection::set (vector<RegionView*>& v)
+{
+       clear_regions ();
+       if (Config->get_link_region_and_track_selection()) {
+               clear_tracks ();
+               // make sure to deselect any automation selections
+               clear_points();
+       }
        add (v);
 }
 
+/** Set the start and end time of the time selection, without changing
+ *  the list of tracks it applies to.
+ */
 long
-Selection::set (TimeAxisView* track, jack_nframes_t start, jack_nframes_t end)
+Selection::set (nframes_t start, nframes_t end)
 {
        if ((start == 0 && end == 0) || end < start) {
                return 0;
@@ -560,14 +747,6 @@ Selection::set (TimeAxisView* track, jack_nframes_t start, jack_nframes_t end)
                time.front().end = end;
        }
 
-       if (track) {
-               time.track = track;
-               time.group = track->edit_group();
-       } else {
-               time.track = 0;
-               time.group = 0;
-       }
-
        time.consolidate ();
 
        TimeChanged ();
@@ -576,12 +755,18 @@ Selection::set (TimeAxisView* track, jack_nframes_t start, jack_nframes_t end)
 }
 
 void
-Selection::set (AutomationList *ac)
+Selection::set (boost::shared_ptr<Evoral::ControlList> ac)
 {
        lines.clear();
        add (ac);
 }
 
+bool
+Selection::selected (Marker* m)
+{
+       return find (markers.begin(), markers.end(), m) != markers.end();
+}
+
 bool
 Selection::selected (TimeAxisView* tv)
 {
@@ -589,61 +774,119 @@ Selection::selected (TimeAxisView* tv)
 }
 
 bool
-Selection::selected (AudioRegionView* arv)
+Selection::selected (RegionView* rv)
 {
-       return find (audio_regions.begin(), audio_regions.end(), arv) != audio_regions.end();
+       return find (regions.begin(), regions.end(), rv) != regions.end();
 }
 
 bool
-Selection::empty ()
+Selection::empty (bool internal_selection)
 {
-       return audio_regions.empty () &&
+       bool object_level_empty =  regions.empty () &&
                tracks.empty () &&
-               points.empty () && 
-               playlists.empty () && 
+               points.empty () &&
+               playlists.empty () &&
                lines.empty () &&
                time.empty () &&
                playlists.empty () &&
-               redirects.empty ()
+               markers.empty() &&
+               midi_regions.empty()
                ;
+
+       if (!internal_selection) {
+               return object_level_empty;
+       }
+
+       /* this is intended to really only apply when using a Selection
+          as a cut buffer.
+       */
+
+       return object_level_empty && midi_notes.empty();
+}
+
+void
+Selection::toggle (const vector<AutomationSelectable*>& autos)
+{
+       for (vector<AutomationSelectable*>::const_iterator x = autos.begin(); x != autos.end(); ++x) {
+               if ((*x)->get_selected()) {
+                       points.remove (**x);
+               } else {
+                       points.push_back (**x);
+               }
+
+               delete *x;
+       }
+
+       PointsChanged (); /* EMIT SIGNAL */
+}
+
+void
+Selection::toggle (list<Selectable*>& selectables)
+{
+       RegionView* rv;
+       AutomationSelectable* as;
+       vector<RegionView*> rvs;
+       vector<AutomationSelectable*> autos;
+
+       for (std::list<Selectable*>::iterator i = selectables.begin(); i != selectables.end(); ++i) {
+               if ((rv = dynamic_cast<RegionView*> (*i)) != 0) {
+                       rvs.push_back (rv);
+               } else if ((as = dynamic_cast<AutomationSelectable*> (*i)) != 0) {
+                       autos.push_back (as);
+               } else {
+                       fatal << _("programming error: ")
+                             << X_("unknown selectable type passed to Selection::toggle()")
+                             << endmsg;
+                       /*NOTREACHED*/
+               }
+       }
+
+       if (!rvs.empty()) {
+               toggle (rvs);
+       }
+
+       if (!autos.empty()) {
+               toggle (autos);
+       }
 }
 
 void
 Selection::set (list<Selectable*>& selectables)
 {
-       clear_audio_regions();
+       clear_regions();
        clear_points ();
        add (selectables);
 }
 
+
 void
 Selection::add (list<Selectable*>& selectables)
 {
-       AudioRegionView* arv;
+       RegionView* rv;
        AutomationSelectable* as;
-       vector<AudioRegionView*> arvs;
+       vector<RegionView*> rvs;
        vector<AutomationSelectable*> autos;
 
        for (std::list<Selectable*>::iterator i = selectables.begin(); i != selectables.end(); ++i) {
-               if ((arv = dynamic_cast<AudioRegionView*> (*i)) != 0) {
-                       arvs.push_back (arv);
+               if ((rv = dynamic_cast<RegionView*> (*i)) != 0) {
+                       rvs.push_back (rv);
                } else if ((as = dynamic_cast<AutomationSelectable*> (*i)) != 0) {
                        autos.push_back (as);
                } else {
                        fatal << _("programming error: ")
-                             << X_("unknown selectable type passed to Selection::set()")
+                             << X_("unknown selectable type passed to Selection::add()")
                              << endmsg;
                        /*NOTREACHED*/
                }
        }
 
-       if (!arvs.empty()) {
-               add (arvs);
-       } 
+       if (!rvs.empty()) {
+               add (rvs);
+       }
 
        if (!autos.empty()) {
                add (autos);
-       } 
+       }
 }
 
 void
@@ -660,8 +903,81 @@ Selection::add (vector<AutomationSelectable*>& autos)
 {
        for (vector<AutomationSelectable*>::iterator i = autos.begin(); i != autos.end(); ++i) {
                points.push_back (**i);
-               delete *i;
        }
 
        PointsChanged ();
 }
+
+void
+Selection::set (Marker* m)
+{
+       clear_markers ();
+       add (m);
+}
+
+void
+Selection::toggle (Marker* m)
+{
+       MarkerSelection::iterator i;
+
+       if ((i = find (markers.begin(), markers.end(), m)) == markers.end()) {
+               add (m);
+       } else {
+               remove (m);
+       }
+}
+
+void
+Selection::remove (Marker* m)
+{
+       MarkerSelection::iterator i;
+
+       if ((i = find (markers.begin(), markers.end(), m)) != markers.end()) {
+               markers.erase (i);
+               MarkersChanged();
+       }
+}
+
+void
+Selection::add (Marker* m)
+{
+       if (find (markers.begin(), markers.end(), m) == markers.end()) {
+
+               /* disambiguate which remove() for the compiler */
+
+               void (Selection::*pmf)(Marker*) = &Selection::remove;
+
+               m->GoingAway.connect (sigc::bind (sigc::mem_fun (*this, pmf), m));
+
+               markers.push_back (m);
+               MarkersChanged();
+       }
+}
+
+void
+Selection::add (const list<Marker*>& m)
+{
+       markers.insert (markers.end(), m.begin(), m.end());
+       MarkersChanged ();
+}
+
+void
+MarkerSelection::range (nframes64_t& s, nframes64_t& e)
+{
+       s = max_frames;
+       e = 0;
+
+       for (MarkerSelection::iterator i = begin(); i != end(); ++i) {
+
+               if ((*i)->position() < s) {
+                       s = (*i)->position();
+               }
+
+               if ((*i)->position() > e) {
+                       e = (*i)->position();
+               }
+       }
+
+       s = std::min (s, e);
+       e = std::max (s, e);
+}