smokephil/ardour
1
0
Fork 0
mirror of https://github.com/Ardour/ardour.git synced 2025-12-07 23:35:03 +01:00
Code Issues Projects Releases Wiki Activity

Rework region selection XML

Browse source 
Ensures that selection is restored in the same object type order
it was created in, which may some day make a difference.
This commit is contained in:
nick_m 2017-06-21 23:51:12 +10:00
parent 1d8548306b
commit 515cdb6b32
2 changed files with 32 additions and 23 deletions
Download patch file Download diff file Expand all files Collapse all files

2
gtk2_ardour/editor_ops.cc
View file

@ -3342,7 +3342,7 @@ Editor::separate_under_selected_regions ()
PlaylistState before; PlaylistState before;
before.playlist = playlist; before.playlist = playlist;
before.before = &playlist->get_state(); before.before = &playlist->get_state();
playlist->clear_changes ();
playlist->freeze (); playlist->freeze ();
playlists.push_back(before); playlists.push_back(before);
} }

53
gtk2_ardour/selection.cc
View file

@ -1150,9 +1150,12 @@ Selection::get_state () const
} }
} }
for (RegionSelection::const_iterator i = regions.begin(); i != regions.end(); ++i) { if (!regions.empty()) {
XMLNode* r = node->add_child (X_("Region")); XMLNode* parent = node->add_child (X_("Regions"));
r->set_property (X_("id"), (*i)->region ()->id ()); for (RegionSelection::const_iterator i = regions.begin(); i != regions.end(); ++i) {
XMLNode* r = parent->add_child (X_("Region"));
r->set_property (X_("id"), (*i)->region ()->id ());
}
} }
/* midi region views have thir own internal selection. */ /* midi region views have thir own internal selection. */
@ -1225,8 +1228,6 @@ Selection::set_state (XMLNode const & node, int)
clear_time (); clear_time ();
clear_markers (); clear_markers ();
RegionSelection selected_regions;
/* NOTE: stripable/time-axis-view selection is saved/restored by /* NOTE: stripable/time-axis-view selection is saved/restored by
* ARDOUR::CoreSelection, not this Selection object * ARDOUR::CoreSelection, not this Selection object
*/ */
@ -1235,23 +1236,34 @@ Selection::set_state (XMLNode const & node, int)
XMLNodeList children = node.children (); XMLNodeList children = node.children ();
for (XMLNodeList::const_iterator i = children.begin(); i != children.end(); ++i) { for (XMLNodeList::const_iterator i = children.begin(); i != children.end(); ++i) {
if ((*i)->name() == X_("Region")) { if ((*i)->name() == X_("Regions")) {
RegionSelection selected_regions;
XMLNodeList children = (*i)->children ();
for (XMLNodeList::const_iterator ci = children.begin(); ci != children.end(); ++ci) {
PBD::ID id;
if (!(*i)->get_property (X_("id"), id)) { if (!(*ci)->get_property (X_("id"), id)) {
assert(false); continue;
}
RegionSelection rs;
editor->get_regionviews_by_id (id, rs);
if (!rs.empty ()) {
for (RegionSelection::const_iterator i = rs.begin(); i != rs.end(); ++i) {
selected_regions.push_back (*i);
}
} else {
/*
regionviews haven't been constructed - stash the region IDs
so we can identify them in Editor::region_view_added ()
*/
regions.pending.push_back (id);
}
} }
RegionSelection rs; if (!selected_regions.empty()) {
editor->get_regionviews_by_id (id, rs); add (selected_regions);
if (!rs.empty ()) {
selected_regions.insert (selected_regions.end(), rs.begin(), rs.end());
} else {
/*
regionviews haven't been 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_("MIDINotes")) { } else if ((*i)->name() == X_("MIDINotes")) {
@ -1406,9 +1418,6 @@ Selection::set_state (XMLNode const & node, int)
} }
// now add regions to selection at once
add (selected_regions);
return 0; return 0;
} }