ardour/ardour
ardour/ardour
13
1
Fork 0
Code Pull Requests Releases Activity

Fix iterator off-by-one: Allow to move top-most track. via ctrl up/down

Browse Source
This commit is contained in:
Robin Gareus 2017-02-20 21:37:50 +01:00
parent 3a50975c3d
commit 4ece934e66
1 changed files with 4 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
gtk2_ardour/editor_routes.cc
View File

@ -1663,9 +1663,9 @@ EditorRoutes::move_selected_tracks (bool up)
PresentationInfo::ChangeSuspender cs; PresentationInfo::ChangeSuspender cs;
if (up) { if (up) {
unselected_neighbour = view_stripables.end (); unselected_neighbour = view_stripables.begin ();
++unselected_neighbour;
vsi = view_stripables.begin(); vsi = view_stripables.begin();
++vsi;
while (vsi != view_stripables.end()) { while (vsi != view_stripables.end()) {
@ -1699,10 +1699,11 @@ EditorRoutes::move_selected_tracks (bool up)
unselected_neighbour = view_stripables.end(); unselected_neighbour = view_stripables.end();
vsi = unselected_neighbour; vsi = unselected_neighbour;
--vsi;
do { do {
--vsi;
if (vsi->stripable->presentation_info().selected()) { if (vsi->stripable->presentation_info().selected()) {
if (unselected_neighbour != view_stripables.end()) { if (unselected_neighbour != view_stripables.end()) {
@ -1726,8 +1727,6 @@ EditorRoutes::move_selected_tracks (bool up)
} }
--vsi;
} while (vsi != view_stripables.begin()); } while (vsi != view_stripables.begin());
} }
} }