crossfade hack and slash. removed overlap checks, overlap mode, default length,...
[ardour.git] / gtk2_ardour / selection.cc
index 2eafcf220c555786034e37ee3172884f47515791..395ceda1696ad95aa00b814d292bca6bb473653f 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
 #include "ardour/playlist.h"
 #include "ardour/rc_configuration.h"
 
+#include "gui_thread.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 "control_point.h"
 
 #include "i18n.h"
 
 using namespace std;
 using namespace ARDOUR;
 using namespace PBD;
-using namespace sigc;
 
 struct AudioRangeComparator {
     bool operator()(AudioRange a, AudioRange b) {
@@ -45,6 +47,27 @@ struct AudioRangeComparator {
     }
 };
 
+Selection::Selection (const PublicEditor* e)
+       : tracks (e)
+       , editor (e)
+       , next_time_id (0)
+       , _no_tracks_changed (false)
+{
+       clear ();
+
+       /* we have disambiguate which remove() for the compiler */
+
+       void (Selection::*track_remove)(TimeAxisView*) = &Selection::remove;
+       TimeAxisView::CatchDeletion.connect (*this, MISSING_INVALIDATOR, boost::bind (track_remove, this, _1), gui_context());
+
+       void (Selection::*marker_remove)(Marker*) = &Selection::remove;
+       Marker::CatchDeletion.connect (*this, MISSING_INVALIDATOR, boost::bind (marker_remove, this, _1), gui_context());
+
+       void (Selection::*point_remove)(ControlPoint*) = &Selection::remove;
+       ControlPoint::CatchDeletion.connect (*this, MISSING_INVALIDATOR, boost::bind (point_remove, this, _1), gui_context());
+}
+
+#if 0
 Selection&
 Selection::operator= (const Selection& other)
 {
@@ -53,20 +76,23 @@ Selection::operator= (const Selection& other)
                tracks = other.tracks;
                time = other.time;
                lines = other.lines;
+               midi_regions = other.midi_regions;
+               midi_notes = other.midi_notes;
        }
        return *this;
 }
+#endif
 
 bool
 operator== (const Selection& a, const Selection& b)
 {
        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.playlists == b.playlists &&
+               a.midi_notes == b.midi_notes &&
+               a.midi_regions == b.midi_regions;
 }
 
 /** Clear everything from the Selection */
@@ -79,6 +105,40 @@ Selection::clear ()
        clear_lines();
        clear_time ();
        clear_playlists ();
+       clear_midi_notes ();
+       clear_midi_regions ();
+       clear_markers ();
+}
+
+void
+Selection::clear_objects ()
+{
+       clear_regions ();
+       clear_points ();
+       clear_lines();
+       clear_playlists ();
+       clear_midi_notes ();
+       clear_midi_regions ();
+       clear_markers ();
+}
+
+void
+Selection::clear_tracks ()
+{
+       if (!tracks.empty()) {
+               tracks.clear ();
+               if (!_no_tracks_changed) {
+                       TracksChanged();
+               }
+       }
+}
+
+void
+Selection::clear_time ()
+{
+       time.clear();
+
+       TimeChanged ();
 }
 
 void
@@ -97,26 +157,31 @@ Selection::clear_regions ()
        if (!regions.empty()) {
                regions.clear_all ();
                RegionsChanged();
+               if (Config->get_link_region_and_track_selection()) {
+                       clear_tracks ();
+               }
        }
 }
 
 void
-Selection::clear_tracks ()
+Selection::clear_midi_notes ()
 {
-       if (!tracks.empty()) {
-               tracks.clear ();
-               TracksChanged();
+       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_time ()
+Selection::clear_midi_regions ()
 {
-       time.track = 0;
-       time.group = 0;
-       time.clear();
-
-       TimeChanged ();
+       if (!midi_regions.empty()) {
+               midi_regions.clear ();
+               MidiRegionsChanged ();
+       }
 }
 
 void
@@ -157,6 +222,8 @@ Selection::clear_markers ()
 void
 Selection::toggle (boost::shared_ptr<Playlist> pl)
 {
+       clear_time();  //enforce object/range exclusivity
+       
        PlaylistSelection::iterator i;
 
        if ((i = find (playlists.begin(), playlists.end(), pl)) == playlists.end()) {
@@ -170,10 +237,10 @@ Selection::toggle (boost::shared_ptr<Playlist> pl)
 }
 
 void
-Selection::toggle (const list<TimeAxisView*>& track_list)
+Selection::toggle (const TrackViewList& track_list)
 {
-       for (list<TimeAxisView*>::const_iterator i = track_list.begin(); i != track_list.end(); ++i) {
-               toggle ( (*i) );
+       for (TrackViewList::const_iterator i = track_list.begin(); i != track_list.end(); ++i) {
+               toggle ((*i));
        }
 }
 
@@ -181,21 +248,50 @@ 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));
                tracks.push_back (track);
        } else {
                tracks.erase (i);
        }
 
-       TracksChanged();
+       if (!_no_tracks_changed) {
+               TracksChanged();
+       }
+}
+
+void
+Selection::toggle (const MidiNoteSelection& midi_note_list)
+{
+       clear_time();  //enforce object/range exclusivity
+
+       for (MidiNoteSelection::const_iterator i = midi_note_list.begin(); i != midi_note_list.end(); ++i) {
+               toggle ((*i));
+       }
+}
+
+void
+Selection::toggle (MidiCutBuffer* midi)
+{
+       MidiNoteSelection::iterator i;
+
+       if ((i = find (midi_notes.begin(), midi_notes.end(), midi)) == midi_notes.end()) {
+               midi_notes.push_back (midi);
+       } else {
+               /* remember that we own the MCB */
+               delete *i;
+               midi_notes.erase (i);
+       }
+
+       MidiNotesChanged();
 }
 
+
 void
 Selection::toggle (RegionView* r)
 {
+       clear_time();  //enforce object/range exclusivity
+       
        RegionSelection::iterator i;
 
        if ((i = find (regions.begin(), regions.end(), r)) == regions.end()) {
@@ -207,9 +303,27 @@ Selection::toggle (RegionView* r)
        RegionsChanged ();
 }
 
+void
+Selection::toggle (MidiRegionView* mrv)
+{
+       clear_time();   //enforce object/range exclusivity
+
+       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)
 {
+       clear_time();  //enforce object/range exclusivity
+
        RegionSelection::iterator i;
 
        for (vector<RegionView*>::iterator x = r.begin(); x != r.end(); ++x) {
@@ -224,8 +338,10 @@ Selection::toggle (vector<RegionView*>& r)
 }
 
 long
-Selection::toggle (nframes_t start, nframes_t end)
+Selection::toggle (framepos_t start, framepos_t end)
 {
+       clear_objects();  //enforce object/range exclusivity
+
        AudioRangeComparator cmp;
 
        /* XXX this implementation is incorrect */
@@ -233,7 +349,7 @@ Selection::toggle (nframes_t start, nframes_t end)
        time.push_back (AudioRange (start, end, next_time_id++));
        time.consolidate ();
        time.sort (cmp);
-       
+
        TimeChanged ();
 
        return next_time_id - 1;
@@ -242,6 +358,8 @@ Selection::toggle (nframes_t start, nframes_t end)
 void
 Selection::add (boost::shared_ptr<Playlist> pl)
 {
+       clear_time();  //enforce object/range exclusivity
+       
        if (find (playlists.begin(), playlists.end(), pl) == playlists.end()) {
                pl->use ();
                playlists.push_back(pl);
@@ -252,6 +370,8 @@ Selection::add (boost::shared_ptr<Playlist> pl)
 void
 Selection::add (const list<boost::shared_ptr<Playlist> >& pllist)
 {
+       clear_time();  //enforce object/range exclusivity
+       
        bool changed = false;
 
        for (list<boost::shared_ptr<Playlist> >::const_iterator i = pllist.begin(); i != pllist.end(); ++i) {
@@ -261,51 +381,72 @@ Selection::add (const list<boost::shared_ptr<Playlist> >& pllist)
                        changed = true;
                }
        }
-       
+
        if (changed) {
                PlaylistsChanged ();
        }
 }
 
 void
-Selection::add (const list<TimeAxisView*>& track_list)
+Selection::add (const TrackViewList& track_list)
 {
-       list<TimeAxisView*> added = tracks.add (track_list);
+       TrackViewList added = tracks.add (track_list);
 
-       for (list<TimeAxisView*>::const_iterator i = added.begin(); i != added.end(); ++i) {
-               void (Selection::*pmf)(TimeAxisView*) = &Selection::remove;
-               (*i)->GoingAway.connect (sigc::bind (mem_fun (*this, pmf), (*i)));
-       }
-       
        if (!added.empty()) {
-               TracksChanged ();
+               if (!_no_tracks_changed) {
+                       TracksChanged ();
+               }
        }
 }
 
 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)
+{
+       clear_time();  //enforce object/range exclusivity
+
+       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 (vector<RegionView*>& v)
 {
+       clear_time();  //enforce object/range exclusivity
+
        /* 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());
+                               add (&(*i)->get_time_axis_view());
                        }
                }
        }
@@ -318,20 +459,22 @@ Selection::add (vector<RegionView*>& v)
 void
 Selection::add (const RegionSelection& rs)
 {
+       clear_time();  //enforce object/range exclusivity
+
        /* XXX This method or the add (const vector<RegionView*>&) needs to go
         */
 
        bool changed = false;
-       
+
        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());
+                               add (&(*i)->get_time_axis_view());
                        }
                }
        }
-       
+
        if (changed) {
                RegionsChanged ();
        }
@@ -340,18 +483,41 @@ Selection::add (const RegionSelection& rs)
 void
 Selection::add (RegionView* r)
 {
+       clear_time();  //enforce object/range exclusivity
+
        if (find (regions.begin(), regions.end(), r) == regions.end()) {
-               regions.add (r);
+               bool changed = regions.add (r);
+                if (Config->get_link_region_and_track_selection() && changed) {
+                        add (&r->get_time_axis_view());
+                }
+                if (changed) {
+                        RegionsChanged ();
+                }
+       }
+}
+
+void
+Selection::add (MidiRegionView* mrv)
+{
+       clear_time();  //enforce object/range exclusivity
+
+       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 (&r->get_trackview());
+                       add (&mrv->get_time_axis_view());
                }
-               RegionsChanged ();
+#endif
+               MidiRegionsChanged ();
        }
 }
 
 long
-Selection::add (nframes_t start, nframes_t end)
+Selection::add (framepos_t start, framepos_t end)
 {
+       clear_objects();  //enforce object/range exclusivity
+
        AudioRangeComparator cmp;
 
        /* XXX this implementation is incorrect */
@@ -359,15 +525,17 @@ Selection::add (nframes_t start, 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, nframes_t start, nframes_t end)
+Selection::replace (uint32_t sid, framepos_t start, framepos_t end)
 {
+       clear_objects();  //enforce object/range exclusivity
+       
        for (list<AudioRange>::iterator i = time.begin(); i != time.end(); ++i) {
                if ((*i).id == sid) {
                        time.erase (i);
@@ -388,12 +556,13 @@ Selection::replace (uint32_t sid, nframes_t start, nframes_t end)
 void
 Selection::add (boost::shared_ptr<Evoral::ControlList> cl)
 {
+       clear_time();  //enforce object/range exclusivity
+
        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;
        }
        if (find (lines.begin(), lines.end(), al) == lines.end()) {
                lines.push_back (al);
@@ -407,27 +576,72 @@ Selection::remove (TimeAxisView* track)
        list<TimeAxisView*>::iterator i;
        if ((i = find (tracks.begin(), tracks.end(), track)) != tracks.end()) {
                tracks.erase (i);
-               TracksChanged();
+               if (!_no_tracks_changed) {
+                       TracksChanged();
+               }
        }
 }
 
 void
-Selection::remove (const list<TimeAxisView*>& track_list)
+Selection::remove (ControlPoint* p)
 {
-       bool changed = false;
+       PointSelection::iterator i = find (points.begin(), points.end(), p);
+       if (i != points.end ()) {
+               points.erase (i);
+       }
+}
 
-       for (list<TimeAxisView*>::const_iterator i = track_list.begin(); i != track_list.end(); ++i) {
+void
+Selection::remove (const TrackViewList& track_list)
+{
+       bool changed = false;
 
-               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;
                }
        }
 
        if (changed) {
-               TracksChanged();
+               if (!_no_tracks_changed) {
+                       TracksChanged();
+               }
+       }
+}
+
+void
+Selection::remove (const MidiNoteSelection& midi_list)
+{
+       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 ();
        }
 }
 
@@ -468,11 +682,29 @@ Selection::remove (RegionView* r)
                RegionsChanged ();
        }
 
-       if (Config->get_link_region_and_track_selection() && !regions.involves (r->get_trackview())) {
-               remove (&r->get_trackview());
+       if (Config->get_link_region_and_track_selection() && !regions.involves (r->get_time_axis_view())) {
+               remove (&r->get_time_axis_view());
        }
 }
 
+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_time_axis_view())) {
+               remove (&r->get_time_axis_view());
+       }
+#endif
+}
+
 
 void
 Selection::remove (uint32_t selection_id)
@@ -484,7 +716,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;
                }
@@ -492,7 +724,7 @@ Selection::remove (uint32_t selection_id)
 }
 
 void
-Selection::remove (nframes_t start, nframes_t end)
+Selection::remove (framepos_t /*start*/, framepos_t /*end*/)
 {
 }
 
@@ -514,39 +746,62 @@ 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 (const MidiNoteSelection& midi_list)
+{
+       clear_time ();  //enforce region/object exclusivity
+       clear_objects ();
+       add (midi_list);
+}
+
 void
 Selection::set (boost::shared_ptr<Playlist> playlist)
 {
-       clear_playlists ();
+       clear_time ();  //enforce region/object exclusivity
+       clear_objects ();
        add (playlist);
 }
 
 void
 Selection::set (const list<boost::shared_ptr<Playlist> >& pllist)
 {
-       clear_playlists ();
+       clear_time();  //enforce region/object exclusivity
+       clear_objects ();
        add (pllist);
 }
 
 void
 Selection::set (const RegionSelection& rs)
 {
-       clear_regions();
+       clear_time();  //enforce region/object exclusivity
+       clear_objects();
        regions = rs;
        RegionsChanged(); /* EMIT SIGNAL */
 }
 
+void
+Selection::set (MidiRegionView* mrv)
+{
+       clear_time();  //enforce region/object exclusivity
+       clear_objects ();
+       add (mrv);
+}
+
 void
 Selection::set (RegionView* r, bool also_clear_tracks)
 {
-       clear_regions ();
-       if (also_clear_tracks) {
+       clear_time();  //enforce region/object exclusivity
+       clear_objects ();
+       if (also_clear_tracks && !Config->get_link_region_and_track_selection()) {
+               /* clear_regions() will have done this if the link preference
+                * is enabled
+                */
                clear_tracks ();
        }
        add (r);
@@ -555,18 +810,35 @@ Selection::set (RegionView* r, bool also_clear_tracks)
 void
 Selection::set (vector<RegionView*>& v)
 {
-       clear_regions ();
+       bool had_regions = !regions.empty();
+
+       clear_time();  //enforce region/object exclusivity
+       clear_objects();
+
        if (Config->get_link_region_and_track_selection()) {
-               clear_tracks ();
-               // make sure to deselect any automation selections
-               clear_points();
+               if (had_regions) {
+                       /* there were regions before, so we're changing the
+                        * region selection (likely), thus link region/track
+                        * selection. relevant tracks will get selected
+                        * as we ::add() below.
+                        */
+                       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, nframes_t start, nframes_t end)
+Selection::set (framepos_t start, framepos_t end)
 {
+       clear_objects();  //enforce region/object exclusivity
+       clear_time();
+
        if ((start == 0 && end == 0) || end < start) {
                return 0;
        }
@@ -583,25 +855,49 @@ Selection::set (TimeAxisView* track, nframes_t start, nframes_t end)
                time.front().end = end;
        }
 
-       if (track) {
-               time.track = track;
-               time.group = track->route_group();
+       time.consolidate ();
+
+       TimeChanged ();
+
+       return time.front().id;
+}
+
+/** Set the start and end of the range selection.  If more than one range
+ *  is currently selected, the start of the earliest range and the end of the
+ *  latest range are set.  If no range is currently selected, this method
+ *  selects a single range from start to end.
+ *
+ *  @param start New start time.
+ *  @param end New end time.
+ */
+void
+Selection::set_preserving_all_ranges (framepos_t start, framepos_t end)
+{
+       clear_objects();  //enforce region/object exclusivity
+
+       if ((start == 0 && end == 0) || (end < start)) {
+               return;
+       }
+
+       if (time.empty ()) {
+               time.push_back (AudioRange (start, end, next_time_id++));
        } else {
-               time.track = 0;
-               time.group = 0;
+               time.sort (AudioRangeComparator ());
+               time.front().start = start;
+               time.back().end = end;
        }
 
        time.consolidate ();
 
        TimeChanged ();
-
-       return time.front().id;
 }
 
 void
 Selection::set (boost::shared_ptr<Evoral::ControlList> ac)
 {
-       lines.clear();
+       clear_time();  //enforce region/object exclusivity
+       clear_objects();
+       
        add (ac);
 }
 
@@ -624,48 +920,77 @@ Selection::selected (RegionView* rv)
 }
 
 bool
-Selection::empty ()
+Selection::selected (ControlPoint* cp)
 {
-       return regions.empty () &&
+       return find (points.begin(), points.end(), cp) != points.end();
+}
+
+bool
+Selection::empty (bool internal_selection)
+{
+       bool object_level_empty =  regions.empty () &&
                tracks.empty () &&
-               points.empty () && 
-               playlists.empty () && 
+               points.empty () &&
+               playlists.empty () &&
                lines.empty () &&
                time.empty () &&
                playlists.empty () &&
-               markers.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)
+Selection::toggle (ControlPoint* cp)
 {
-       for (vector<AutomationSelectable*>::const_iterator x = autos.begin(); x != autos.end(); ++x) {
-               if ((*x)->get_selected()) {
-                       points.remove (**x);
-               } else {
-                       points.push_back (**x);
-               }
+       clear_time();  //enforce region/object exclusivity
 
-               delete *x;
+       cp->set_selected (!cp->get_selected ());
+       PointSelection::iterator i = find (points.begin(), points.end(), cp);
+       if (i == points.end()) {
+               points.push_back (cp);
+       } else {
+               points.erase (i);
        }
 
        PointsChanged (); /* EMIT SIGNAL */
 }
 
 void
-Selection::toggle (list<Selectable*>& selectables)
+Selection::toggle (vector<ControlPoint*> const & cps)
+{
+       clear_time();  //enforce region/object exclusivity
+
+       for (vector<ControlPoint*>::const_iterator i = cps.begin(); i != cps.end(); ++i) {
+               toggle (*i);
+       }
+}
+
+void
+Selection::toggle (list<Selectable*> const & selectables)
 {
+       clear_time();  //enforce region/object exclusivity
+       
        RegionView* rv;
-       AutomationSelectable* as;
+       ControlPoint* cp;
        vector<RegionView*> rvs;
-       vector<AutomationSelectable*> autos;
+       vector<ControlPoint*> cps;
 
-       for (std::list<Selectable*>::iterator i = selectables.begin(); i != selectables.end(); ++i) {
+       for (std::list<Selectable*>::const_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 if ((cp = dynamic_cast<ControlPoint*> (*i)) != 0) {
+                       cps.push_back (cp);
                } else {
                        fatal << _("programming error: ")
                              << X_("unknown selectable type passed to Selection::toggle()")
@@ -676,35 +1001,51 @@ Selection::toggle (list<Selectable*>& selectables)
 
        if (!rvs.empty()) {
                toggle (rvs);
-       } 
+       }
 
-       if (!autos.empty()) {
-               toggle (autos);
-       } 
+       if (!cps.empty()) {
+               toggle (cps);
+       }
 }
 
 void
-Selection::set (list<Selectable*>& selectables)
+Selection::set (list<Selectable*> const & selectables)
 {
-       clear_regions();
-       clear_points ();
+       clear_time ();  //enforce region/object exclusivity
+       clear_objects ();
+
+       if (Config->get_link_region_and_track_selection ()) {
+               clear_tracks ();
+       }
+
        add (selectables);
 }
 
+void
+Selection::add (PointSelection const & s)
+{
+       clear_time ();  //enforce region/object exclusivity
+
+       for (PointSelection::const_iterator i = s.begin(); i != s.end(); ++i) {
+               points.push_back (*i);
+       }
+}
 
 void
-Selection::add (list<Selectable*>& selectables)
+Selection::add (list<Selectable*> const & selectables)
 {
+       clear_time ();  //enforce region/object exclusivity
+
        RegionView* rv;
-       AutomationSelectable* as;
+       ControlPoint* cp;
        vector<RegionView*> rvs;
-       vector<AutomationSelectable*> autos;
+       vector<ControlPoint*> cps;
 
-       for (std::list<Selectable*>::iterator i = selectables.begin(); i != selectables.end(); ++i) {
+       for (std::list<Selectable*>::const_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 if ((cp = dynamic_cast<ControlPoint*> (*i)) != 0) {
+                       cps.push_back (cp);
                } else {
                        fatal << _("programming error: ")
                              << X_("unknown selectable type passed to Selection::add()")
@@ -715,11 +1056,11 @@ Selection::add (list<Selectable*>& selectables)
 
        if (!rvs.empty()) {
                add (rvs);
-       } 
+       }
 
-       if (!autos.empty()) {
-               add (autos);
-       } 
+       if (!cps.empty()) {
+               add (cps);
+       }
 }
 
 void
@@ -732,27 +1073,60 @@ Selection::clear_points ()
 }
 
 void
-Selection::add (vector<AutomationSelectable*>& autos)
+Selection::add (ControlPoint* cp)
+{
+       clear_time ();  //enforce region/object exclusivity
+
+       cp->set_selected (true);
+       points.push_back (cp);
+       PointsChanged (); /* EMIT SIGNAL */
+}
+
+void
+Selection::add (vector<ControlPoint*> const & cps)
+{
+       clear_time ();  //enforce region/object exclusivity
+
+       for (vector<ControlPoint*>::const_iterator i = cps.begin(); i != cps.end(); ++i) {
+               (*i)->set_selected (true);
+               points.push_back (*i);
+       }
+       PointsChanged (); /* EMIT SIGNAL */
+}
+
+void
+Selection::set (ControlPoint* cp)
 {
-       for (vector<AutomationSelectable*>::iterator i = autos.begin(); i != autos.end(); ++i) {
-               points.push_back (**i);
+       clear_time ();  //enforce region/object exclusivity
+
+       if (cp->get_selected()) {
+               return;
        }
 
-       PointsChanged ();
+       for (uint32_t i = 0; i < cp->line().npoints(); ++i) {
+               cp->line().nth (i)->set_selected (false);
+       }
+
+       clear_objects ();
+       add (cp);
 }
 
 void
 Selection::set (Marker* m)
 {
-       clear_markers ();
+       clear_time ();  //enforce region/object exclusivity
+       clear_objects();
+
        add (m);
 }
 
 void
 Selection::toggle (Marker* m)
 {
+       clear_time ();  //enforce region/object exclusivity
+
        MarkerSelection::iterator i;
-       
+
        if ((i = find (markers.begin(), markers.end(), m)) == markers.end()) {
                add (m);
        } else {
@@ -774,14 +1148,9 @@ Selection::remove (Marker* m)
 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 (bind (mem_fun (*this, pmf), m));
+       clear_time ();  //enforce region/object exclusivity
 
+       if (find (markers.begin(), markers.end(), m) == markers.end()) {
                markers.push_back (m);
                MarkersChanged();
        }
@@ -790,21 +1159,26 @@ Selection::add (Marker* m)
 void
 Selection::add (const list<Marker*>& m)
 {
+       clear_time ();  //enforce region/object exclusivity
+
        markers.insert (markers.end(), m.begin(), m.end());
+       markers.sort ();
+       markers.unique ();
+       
        MarkersChanged ();
 }
 
 void
-MarkerSelection::range (nframes64_t& s, nframes64_t& e)
+MarkerSelection::range (framepos_t& s, framepos_t& e)
 {
-       s = max_frames;
+       s = max_framepos;
        e = 0;
 
        for (MarkerSelection::iterator i = begin(); i != end(); ++i) {
 
                if ((*i)->position() < s) {
                        s = (*i)->position();
-               } 
+               }
 
                if ((*i)->position() > e) {
                        e = (*i)->position();
@@ -814,3 +1188,123 @@ MarkerSelection::range (nframes64_t& s, nframes64_t& e)
        s = std::min (s, e);
        e = std::max (s, e);
 }
+
+XMLNode&
+Selection::get_state () const
+{
+       /* XXX: not complete; just sufficient to get track selection state
+          so that re-opening plugin windows for editor mixer strips works
+       */
+
+       XMLNode* node = new XMLNode (X_("Selection"));
+
+       for (TrackSelection::const_iterator i = tracks.begin(); i != tracks.end(); ++i) {
+               RouteTimeAxisView* rtv = dynamic_cast<RouteTimeAxisView*> (*i);
+               AutomationTimeAxisView* atv = dynamic_cast<AutomationTimeAxisView*> (*i);
+               if (rtv) {
+                       XMLNode* t = node->add_child (X_("RouteView"));
+                       t->add_property (X_("id"), atoi (rtv->route()->id().to_s().c_str()));
+               } else if (atv) {
+                       XMLNode* t = node->add_child (X_("AutomationView"));
+                       t->add_property (X_("id"), atoi (atv->parent_route()->id().to_s().c_str()));
+                       t->add_property (X_("parameter"), EventTypeMap::instance().to_symbol (atv->parameter ()));
+               }
+       }
+
+       for (MarkerSelection::const_iterator i = markers.begin(); i != markers.end(); ++i) {
+               XMLNode* t = node->add_child (X_("Marker"));
+
+               bool is_start;
+               Location* loc = editor->find_location_from_marker (*i, is_start);
+
+               t->add_property (X_("id"), atoi (loc->id().to_s().c_str()));
+               t->add_property (X_("start"), is_start ? X_("yes") : X_("no"));
+       }
+
+       return *node;
+}
+
+int
+Selection::set_state (XMLNode const & node, int)
+{
+       if (node.name() != X_("Selection")) {
+               return -1;
+       }
+
+       XMLNodeList children = node.children ();
+       for (XMLNodeList::const_iterator i = children.begin(); i != children.end(); ++i) {
+               if ((*i)->name() == X_("RouteView")) {
+
+                       XMLProperty* prop_id = (*i)->property (X_("id"));
+                       assert (prop_id);
+                       PBD::ID id (prop_id->value ());
+                       RouteTimeAxisView* rtv = editor->get_route_view_by_route_id (id);
+                       if (rtv) {
+                               add (rtv);
+                       }
+
+               } else if ((*i)->name() == X_("AutomationView")) {
+
+                       XMLProperty* prop_id = (*i)->property (X_("id"));
+                       XMLProperty* prop_parameter = (*i)->property (X_("parameter"));
+
+                       assert (prop_id);
+                       assert (prop_parameter);
+
+                       PBD::ID id (prop_id->value ());
+                       RouteTimeAxisView* rtv = editor->get_route_view_by_route_id (id);
+
+                       if (rtv) {
+                               boost::shared_ptr<AutomationTimeAxisView> atv = rtv->automation_child (EventTypeMap::instance().new_parameter (prop_parameter->value ()));
+
+                               /* the automation could be for an entity that was never saved
+                                  in the session file. Don't freak out if we can't find
+                                  it.
+                               */
+
+                               if (atv) {
+                                       add (atv.get());
+                               }
+                       }
+
+               } else if ((*i)->name() == X_("Marker")) {
+
+                       XMLProperty* prop_id = (*i)->property (X_("id"));
+                       XMLProperty* prop_start = (*i)->property (X_("start"));
+                       assert (prop_id);
+                       assert (prop_start);
+
+                       PBD::ID id (prop_id->value ());
+                       Marker* m = editor->find_marker_from_location_id (id, string_is_affirmative (prop_start->value ()));
+                       if (m) {
+                               add (m);
+                       }
+                       
+               }
+               
+       }
+
+       return 0;
+}
+
+void
+Selection::remove_regions (TimeAxisView* t)
+{
+       RegionSelection::iterator i = regions.begin();
+       while (i != regions.end ()) {
+               RegionSelection::iterator tmp = i;
+               ++tmp;
+
+               if (&(*i)->get_time_axis_view() == t) {
+                       remove (*i);
+               }
+
+               i = tmp;
+       }
+}
+
+void
+Selection::block_tracks_changed (bool yn)
+{
+       _no_tracks_changed = yn;
+}