#include <algorithm>
#include <sigc++/bind.h>
+
#include "pbd/error.h"
#include "pbd/stacktrace.h"
/* we have disambiguate which remove() for the compiler */
void (Selection::*track_remove)(TimeAxisView*) = &Selection::remove;
- TimeAxisView::CatchDeletion.connect (*this, MISSING_INVALIDATOR, ui_bind (track_remove, this, _1), gui_context());
+ 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, ui_bind (marker_remove, this, _1), gui_context());
+ 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
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 ();
+}
+
+void
+Selection::clear_tracks ()
+{
+ if (!tracks.empty()) {
+ for (TrackViewList::iterator x = tracks.begin(); x != tracks.end(); ++x) {
+ (*x)->set_selected (false);
+ }
+ tracks.clear ();
+ if (!_no_tracks_changed) {
+ TracksChanged();
+ }
+ }
+}
+
+void
+Selection::clear_time ()
+{
+ time.clear();
+
+ TimeChanged ();
}
void
if (!regions.empty()) {
regions.clear_all ();
RegionsChanged();
- if (Config->get_link_region_and_track_selection()) {
- clear_tracks ();
- }
- }
-}
-
-void
-Selection::clear_tracks ()
-{
- if (!tracks.empty()) {
- tracks.clear ();
- if (!_no_tracks_changed) {
- TracksChanged();
- }
}
}
midi_notes.clear ();
MidiNotesChanged ();
}
+
+ /* The note selection is actually stored in MidiRegionView, emit signal to
+ tell them to clear their selection. */
+ ClearMidiNoteSelection(); /* EMIT SIGNAL */
}
void
}
}
-void
-Selection::clear_time ()
-{
- time.clear();
-
- TimeChanged ();
-}
-
void
Selection::clear_playlists ()
{
void
Selection::toggle (boost::shared_ptr<Playlist> pl)
{
+ clear_time(); //enforce object/range exclusivity
+ clear_tracks(); //enforce object/track exclusivity
+
PlaylistSelection::iterator i;
if ((i = find (playlists.begin(), playlists.end(), pl)) == playlists.end()) {
TrackSelection::iterator i;
if ((i = find (tracks.begin(), tracks.end(), track)) == tracks.end()) {
+ track->set_selected (true);
tracks.push_back (track);
} else {
+ track->set_selected (false);
tracks.erase (i);
}
void
Selection::toggle (const MidiNoteSelection& midi_note_list)
{
+ clear_time(); //enforce object/range exclusivity
+ clear_tracks(); //enforce object/track exclusivity
+
for (MidiNoteSelection::const_iterator i = midi_note_list.begin(); i != midi_note_list.end(); ++i) {
toggle ((*i));
}
void
Selection::toggle (RegionView* r)
{
+ clear_time(); //enforce object/range exclusivity
+ clear_tracks(); //enforce object/track exclusivity
+
RegionSelection::iterator i;
if ((i = find (regions.begin(), regions.end(), r)) == regions.end()) {
void
Selection::toggle (MidiRegionView* mrv)
{
+ clear_time(); //enforce object/range exclusivity
+ clear_tracks(); //enforce object/track exclusivity
+
MidiRegionSelection::iterator i;
if ((i = find (midi_regions.begin(), midi_regions.end(), mrv)) == midi_regions.end()) {
void
Selection::toggle (vector<RegionView*>& r)
{
+ clear_time(); //enforce object/range exclusivity
+ clear_tracks(); //enforce object/track exclusivity
+
RegionSelection::iterator i;
for (vector<RegionView*>::iterator x = r.begin(); x != r.end(); ++x) {
long
Selection::toggle (framepos_t start, framepos_t end)
{
+ clear_objects(); //enforce object/range exclusivity
+
AudioRangeComparator cmp;
/* XXX this implementation is incorrect */
- time.push_back (AudioRange (start, end, next_time_id++));
+ time.push_back (AudioRange (start, end, ++next_time_id));
time.consolidate ();
time.sort (cmp);
TimeChanged ();
- return next_time_id - 1;
+ return next_time_id;
}
void
Selection::add (boost::shared_ptr<Playlist> pl)
{
+ clear_time(); //enforce object/range exclusivity
+ clear_tracks(); //enforce object/track exclusivity
+
if (find (playlists.begin(), playlists.end(), pl) == playlists.end()) {
pl->use ();
playlists.push_back(pl);
void
Selection::add (const list<boost::shared_ptr<Playlist> >& pllist)
{
+ clear_time(); //enforce object/range exclusivity
+ clear_tracks(); //enforce object/track exclusivity
+
bool changed = false;
for (list<boost::shared_ptr<Playlist> >::const_iterator i = pllist.begin(); i != pllist.end(); ++i) {
void
Selection::add (const TrackViewList& track_list)
{
+ clear_objects(); //enforce object/range exclusivity
+
TrackViewList added = tracks.add (track_list);
if (!added.empty()) {
+ for (TrackViewList::iterator x = added.begin(); x != added.end(); ++x) {
+ (*x)->set_selected (true);
+ }
if (!_no_tracks_changed) {
TracksChanged ();
}
void
Selection::add (TimeAxisView* track)
{
+ clear_objects(); //enforce object/range exclusivity
+
TrackViewList tr;
+ track->set_selected (true);
tr.push_back (track);
add (tr);
}
void
Selection::add (const MidiNoteSelection& midi_list)
{
+ clear_time(); //enforce object/range exclusivity
+ clear_tracks(); //enforce object/track exclusivity
+
const MidiNoteSelection::const_iterator b = midi_list.begin();
const MidiNoteSelection::const_iterator e = midi_list.end();
void
Selection::add (vector<RegionView*>& v)
{
+ clear_time(); //enforce object/range exclusivity
+ clear_tracks(); //enforce object/track exclusivity
+
/* XXX This method or the add (const RegionSelection&) needs to go
*/
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_time_axis_view());
- }
}
}
void
Selection::add (const RegionSelection& rs)
{
+ clear_time(); //enforce object/range exclusivity
+ clear_tracks(); //enforce object/track exclusivity
+
/* XXX This method or the add (const vector<RegionView*>&) needs to go
*/
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_time_axis_view());
- }
}
}
void
Selection::add (RegionView* r)
{
+ clear_time(); //enforce object/range exclusivity
+ clear_tracks(); //enforce object/track exclusivity
+
if (find (regions.begin(), regions.end(), r) == regions.end()) {
bool changed = regions.add (r);
- if (Config->get_link_region_and_track_selection() && changed) {
- add (&r->get_time_axis_view());
- }
- if (changed) {
- RegionsChanged ();
- }
+ if (changed) {
+ RegionsChanged ();
+ }
}
}
void
Selection::add (MidiRegionView* mrv)
{
+ clear_time(); //enforce object/range exclusivity
+ clear_tracks(); //enforce object/track 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 (&mrv->get_time_axis_view());
- }
-#endif
MidiRegionsChanged ();
}
}
long
Selection::add (framepos_t start, framepos_t end)
{
+ clear_objects(); //enforce object/range exclusivity
+
AudioRangeComparator cmp;
/* XXX this implementation is incorrect */
- time.push_back (AudioRange (start, end, next_time_id++));
+ time.push_back (AudioRange (start, end, ++next_time_id));
time.consolidate ();
time.sort (cmp);
TimeChanged ();
- return next_time_id - 1;
+ return next_time_id;
+}
+
+void
+Selection::move_time (framecnt_t distance)
+{
+ if (distance == 0) {
+ return;
+ }
+
+ for (list<AudioRange>::iterator i = time.begin(); i != time.end(); ++i) {
+ (*i).start += distance;
+ (*i).end += distance;
+ }
+
+ TimeChanged ();
}
void
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);
void
Selection::add (boost::shared_ptr<Evoral::ControlList> cl)
{
+ clear_time(); //enforce object/range exclusivity
+ clear_tracks(); //enforce object/track 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);
{
list<TimeAxisView*>::iterator i;
if ((i = find (tracks.begin(), tracks.end(), track)) != tracks.end()) {
+ track->set_selected (false);
tracks.erase (i);
if (!_no_tracks_changed) {
TracksChanged();
TrackViewList::iterator x = find (tracks.begin(), tracks.end(), *i);
if (x != tracks.end()) {
+ (*i)->set_selected (false);
tracks.erase (x);
changed = true;
}
}
}
+void
+Selection::remove (ControlPoint* p)
+{
+ PointSelection::iterator i = find (points.begin(), points.end(), p);
+ if (i != points.end ()) {
+ points.erase (i);
+ }
+}
+
void
Selection::remove (const MidiNoteSelection& midi_list)
{
if (regions.remove (r)) {
RegionsChanged ();
}
-
- if (Config->get_link_region_and_track_selection() && !regions.involves (r->get_time_axis_view())) {
- remove (&r->get_time_axis_view());
- }
}
void
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::set (TimeAxisView* track)
{
+ clear_objects(); //enforce object/range exclusivity
clear_tracks ();
add (track);
}
void
Selection::set (const TrackViewList& track_list)
{
+ clear_objects(); //enforce object/range exclusivity
clear_tracks ();
add (track_list);
}
void
Selection::set (const MidiNoteSelection& midi_list)
{
- clear_midi_notes ();
+ clear_time (); //enforce region/object exclusivity
+ clear_tracks(); //enforce object/track exclusivity
+ clear_objects ();
add (midi_list);
}
void
Selection::set (boost::shared_ptr<Playlist> playlist)
{
- clear_playlists ();
+ clear_time (); //enforce region/object exclusivity
+ clear_tracks(); //enforce object/track 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_tracks(); //enforce object/track exclusivity
+ clear_objects();
regions = rs;
RegionsChanged(); /* EMIT SIGNAL */
}
void
Selection::set (MidiRegionView* mrv)
{
- clear_midi_regions ();
+ clear_time(); //enforce region/object exclusivity
+ clear_tracks(); //enforce object/track exclusivity
+ clear_objects ();
add (mrv);
}
void
-Selection::set (RegionView* r, bool also_clear_tracks)
+Selection::set (RegionView* r, bool /*also_clear_tracks*/)
{
- clear_regions ();
- 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 ();
- }
+ clear_time(); //enforce region/object exclusivity
+ clear_tracks(); //enforce object/track exclusivity
+ clear_objects ();
add (r);
}
void
Selection::set (vector<RegionView*>& v)
{
- bool had_regions = !regions.empty();
-
- clear_regions ();
+ clear_time(); //enforce region/object exclusivity
+ clear_tracks(); //enforce object/track exclusivity
+ clear_objects();
- if (Config->get_link_region_and_track_selection()) {
- 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);
}
long
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;
}
if (time.empty()) {
- time.push_back (AudioRange (start, end, next_time_id++));
+ time.push_back (AudioRange (start, end, ++next_time_id));
} else {
/* reuse the first entry, and remove all the rest */
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++));
+ time.push_back (AudioRange (start, end, ++next_time_id));
} else {
time.sort (AudioRangeComparator ());
time.front().start = start;
void
Selection::set (boost::shared_ptr<Evoral::ControlList> ac)
{
- lines.clear();
+ clear_time(); //enforce region/object exclusivity
+ clear_tracks(); //enforce object/track exclusivity
+ clear_objects();
+
add (ac);
}
bool
Selection::selected (TimeAxisView* tv)
{
- return find (tracks.begin(), tracks.end(), tv) != tracks.end();
+ return tv->get_selected ();
}
bool
return find (regions.begin(), regions.end(), rv) != regions.end();
}
+bool
+Selection::selected (ControlPoint* cp)
+{
+ return find (points.begin(), points.end(), cp) != points.end();
+}
+
bool
Selection::empty (bool internal_selection)
{
as a cut buffer.
*/
- return object_level_empty && midi_notes.empty();
+ return object_level_empty && midi_notes.empty() && points.empty();
}
void
Selection::toggle (ControlPoint* cp)
{
+ clear_time(); //enforce region/object exclusivity
+ clear_tracks(); //enforce object/track exclusivity
+
cp->set_selected (!cp->get_selected ());
- set_point_selection_from_line (cp->line ());
+ 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 (vector<ControlPoint*> const & cps)
{
+ clear_time(); //enforce region/object exclusivity
+ clear_tracks(); //enforce object/track exclusivity
+
for (vector<ControlPoint*>::const_iterator i = cps.begin(); i != cps.end(); ++i) {
- (*i)->set_selected (!(*i)->get_selected ());
+ toggle (*i);
}
-
- set_point_selection_from_line (cps.front()->line ());
}
void
Selection::toggle (list<Selectable*> const & selectables)
{
+ clear_time(); //enforce region/object exclusivity
+ clear_tracks(); //enforce object/track exclusivity
+
RegionView* rv;
ControlPoint* cp;
vector<RegionView*> rvs;
fatal << _("programming error: ")
<< X_("unknown selectable type passed to Selection::toggle()")
<< endmsg;
- /*NOTREACHED*/
+ abort(); /*NOTREACHED*/
}
}
void
Selection::set (list<Selectable*> const & selectables)
{
- clear_regions();
- clear_points ();
-
- if (Config->get_link_region_and_track_selection ()) {
- clear_tracks ();
- }
+ clear_time (); //enforce region/object exclusivity
+ clear_tracks(); //enforce object/track exclusivity
+ clear_objects ();
add (selectables);
}
+void
+Selection::add (PointSelection const & s)
+{
+ clear_time (); //enforce region/object exclusivity
+ clear_tracks(); //enforce object/track exclusivity
+
+ for (PointSelection::const_iterator i = s.begin(); i != s.end(); ++i) {
+ points.push_back (*i);
+ }
+}
void
Selection::add (list<Selectable*> const & selectables)
{
+ clear_time (); //enforce region/object exclusivity
+ clear_tracks(); //enforce object/track exclusivity
+
RegionView* rv;
ControlPoint* cp;
vector<RegionView*> rvs;
fatal << _("programming error: ")
<< X_("unknown selectable type passed to Selection::add()")
<< endmsg;
- /*NOTREACHED*/
+ abort(); /*NOTREACHED*/
}
}
void
Selection::add (ControlPoint* cp)
{
+ clear_time (); //enforce region/object exclusivity
+ clear_tracks(); //enforce object/track exclusivity
+
cp->set_selected (true);
- set_point_selection_from_line (cp->line ());
+ points.push_back (cp);
+ PointsChanged (); /* EMIT SIGNAL */
}
void
Selection::add (vector<ControlPoint*> const & cps)
{
+ clear_time (); //enforce region/object exclusivity
+ clear_tracks(); //enforce object/track exclusivity
+
for (vector<ControlPoint*>::const_iterator i = cps.begin(); i != cps.end(); ++i) {
(*i)->set_selected (true);
+ points.push_back (*i);
}
-
- set_point_selection_from_line (cps.front()->line ());
+ PointsChanged (); /* EMIT SIGNAL */
}
void
Selection::set (ControlPoint* cp)
{
+ clear_time (); //enforce region/object exclusivity
+ clear_tracks(); //enforce object/track exclusivity
+
if (cp->get_selected()) {
return;
}
- /* We're going to set up the PointSelection from the selected ControlPoints
- on this point's line, so we need to deselect all ControlPoints before
- we re-add this one.
- */
-
for (uint32_t i = 0; i < cp->line().npoints(); ++i) {
cp->line().nth (i)->set_selected (false);
}
- vector<ControlPoint*> cps;
- cps.push_back (cp);
- add (cps);
+ clear_objects ();
+ add (cp);
}
void
Selection::set (Marker* m)
{
- clear_markers ();
+ clear_time (); //enforce region/object exclusivity
+ clear_tracks(); //enforce object/track exclusivity
+ markers.clear ();
+
add (m);
}
void
Selection::add (Marker* m)
{
+ clear_time (); //enforce region/object exclusivity
+ clear_tracks(); //enforce object/track exclusivity
+
if (find (markers.begin(), markers.end(), m) == markers.end()) {
markers.push_back (m);
MarkersChanged();
void
Selection::add (const list<Marker*>& m)
{
+ clear_time (); //enforce region/object exclusivity
+ clear_tracks(); //enforce object/track exclusivity
+
markers.insert (markers.end(), m.begin(), m.end());
markers.sort ();
markers.unique ();
e = std::max (s, e);
}
-/** Automation control point selection is mostly manipulated using the selected state
- * of the ControlPoints themselves. For example, to add a point to a selection, its
- * ControlPoint is marked as selected and then this method is called. It sets up
- * our PointSelection from the selected ControlPoints of a given AutomationLine.
- *
- * We can't use ControlPoints directly in the selection, as we need to express a
- * selection of not just a visible ControlPoint but also (possibly) some invisible
- * points nearby. Hence the selection stores AutomationRanges, and these are synced
- * with ControlPoint selection state using AutomationLine::set_selected_points.
- */
-
-void
-Selection::set_point_selection_from_line (AutomationLine const & line)
-{
- points.clear ();
-
- AutomationRange current (DBL_MAX, 0, 1, 0, &line.trackview);
-
- for (uint32_t i = 0; i < line.npoints(); ++i) {
- ControlPoint const * cp = line.nth (i);
-
- if (cp->get_selected()) {
- /* x and y position of this control point in coordinates suitable for
- an AutomationRange (ie model time and fraction of track height)
- */
- double const x = (*(cp->model()))->when;
- double const y = 1 - (cp->get_y() / line.trackview.current_height ());
-
- /* work out the position of a rectangle the size of a control point centred
- on this point
- */
-
- double const size = cp->size ();
- double const x_size = line.time_converter().from (line.trackview.editor().pixel_to_frame (size));
- double const y_size = size / line.trackview.current_height ();
-
- double const x1 = max (0.0, x - x_size / 2);
- double const x2 = x + x_size / 2;
- double const y1 = max (0.0, y - y_size / 2);
- double const y2 = y + y_size / 2;
-
- /* extend the current AutomationRange to put this point in */
- current.start = min (current.start, x1);
- current.end = max (current.end, x2);
- current.low_fract = min (current.low_fract, y1);
- current.high_fract = max (current.high_fract, y2);
-
- } else {
- /* this point isn't selected; if the current AutomationRange has some
- stuff in it, push it onto the list and make a new one
- */
- if (current.start < DBL_MAX) {
- points.push_back (current);
- current = AutomationRange (DBL_MAX, 0, 1, 0, &line.trackview);
- }
- }
- }
-
- /* Maybe push the current AutomationRange, as above */
- if (current.start < DBL_MAX) {
- points.push_back (current);
- current = AutomationRange (DBL_MAX, 0, 1, 0, &line.trackview);
- }
-
- PointsChanged (); /* EMIT SIGNAL */
-}
-
XMLNode&
Selection::get_state () const
{
so that re-opening plugin windows for editor mixer strips works
*/
+ char buf[32];
XMLNode* node = new XMLNode (X_("Selection"));
for (TrackSelection::const_iterator i = tracks.begin(); i != tracks.end(); ++i) {
}
}
+ for (RegionSelection::const_iterator i = regions.begin(); i != regions.end(); ++i) {
+ XMLNode* r = node->add_child (X_("Region"));
+ r->add_property (X_("id"), atoi ((*i)->region ()->id ().to_s ().c_str()));
+
+ }
+
+ for (TimeSelection::const_iterator i = time.begin(); i != time.end(); ++i) {
+ XMLNode* t = node->add_child (X_("AudioRange"));
+ snprintf(buf, sizeof(buf), "%ld", (*i).start);
+ t->add_property (X_("start"), string(buf));
+ snprintf(buf, sizeof(buf), "%ld", (*i).end);
+ t->add_property (X_("end"), string(buf));
+ }
+
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);
return -1;
}
+ clear_regions ();
+ clear_time ();
+ clear_tracks ();
+ clear_markers ();
+
XMLNodeList children = node.children ();
for (XMLNodeList::const_iterator i = children.begin(); i != children.end(); ++i) {
if ((*i)->name() == X_("RouteView")) {
add (rtv);
}
+ } else if ((*i)->name() == X_("Region")) {
+ XMLProperty* prop_id = (*i)->property (X_("id"));
+ assert (prop_id);
+ PBD::ID id (prop_id->value ());
+
+ RegionSelection rs;
+ editor->get_regionviews_by_id (id, rs);
+
+ if (!rs.empty ()) {
+ add (rs);
+ } else {
+ /*
+ regionviews are being constructed - stash the region IDs
+ so we can identify them in Editor::region_view_added ()
+ */
+ regions.pending.push_back (id);
+ }
+
+ } else if ((*i)->name() == X_("AudioRange")) {
+ XMLProperty* prop_start = (*i)->property (X_("start"));
+ XMLProperty* prop_end = (*i)->property (X_("end"));
+
+ assert (prop_start);
+ assert (prop_end);
+
+ framepos_t s (atol (prop_start->value ().c_str()));
+ framepos_t e (atol (prop_end->value ().c_str()));
+
+ set_preserving_all_ranges (s, e);
+
} else if ((*i)->name() == X_("AutomationView")) {
XMLProperty* prop_id = (*i)->property (X_("id"));
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 ()));
+ boost::shared_ptr<AutomationTimeAxisView> atv = rtv->automation_child (EventTypeMap::instance().from_symbol (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