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

Don't use the all_route_group() in Editor::set_selected_track_as_side_effect ().

Browse Source 
- we don't appear to actually use it.
This commit is contained in:
nick_m 2015-10-05 00:42:09 +11:00
parent 52a239a254
commit afcba4d3d3
1 changed files with 4 additions and 29 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

33
gtk2_ardour/editor_selection.cc
View File

@ -192,17 +192,10 @@ Editor::set_selected_track_as_side_effect (Selection::Operation op)
group = clicked_routeview->route()->route_group();
}
bool had_tracks = !selection->tracks.empty();
RouteGroup& arg (_session->all_route_group());
switch (op) {
case Selection::Toggle:
if (selection->selected (clicked_axisview)) {
if (arg.is_select() && arg.is_active()) {
for (TrackViewList::iterator i = track_views.begin(); i != track_views.end (); ++i) {
selection->remove(*i);
}
} else if (group && group->is_active()) {
if (group && group->is_active()) {
for (TrackViewList::iterator i = track_views.begin(); i != track_views.end (); ++i) {
if ((*i)->route_group() == group) {
selection->remove(*i);
@ -212,11 +205,7 @@ Editor::set_selected_track_as_side_effect (Selection::Operation op)
selection->remove (clicked_axisview);
}
} else {
if (arg.is_select() && arg.is_active()) {
for (TrackViewList::iterator i = track_views.begin(); i != track_views.end (); ++i) {
selection->add(*i);
}
} else if (group && group->is_active()) {
if (group && group->is_active()) {
for (TrackViewList::iterator i = track_views.begin(); i != track_views.end (); ++i) {
if ((*i)->route_group() == group) {
selection->add(*i);
@ -229,14 +218,7 @@ Editor::set_selected_track_as_side_effect (Selection::Operation op)
break;
case Selection::Add:
if (!had_tracks && arg.is_select() && arg.is_active()) {
/* nothing was selected already, and all group is active etc. so use
all tracks.
*/
for (TrackViewList::iterator i = track_views.begin(); i != track_views.end (); ++i) {
selection->add(*i);
}
} else if (group && group->is_active()) {
if (group && group->is_active()) {
for (TrackViewList::iterator i = track_views.begin(); i != track_views.end (); ++i) {
if ((*i)->route_group() == group) {
selection->add(*i);
@ -249,14 +231,7 @@ Editor::set_selected_track_as_side_effect (Selection::Operation op)
case Selection::Set:
selection->clear();
if (!had_tracks && arg.is_select() && arg.is_active()) {
/* nothing was selected already, and all group is active etc. so use
all tracks.
*/
for (TrackViewList::iterator i = track_views.begin(); i != track_views.end (); ++i) {
selection->add(*i);
}
} else if (group && group->is_active()) {
if (group && group->is_active()) {
for (TrackViewList::iterator i = track_views.begin(); i != track_views.end (); ++i) {
if ((*i)->route_group() == group) {
selection->add(*i);