From 62fd8ecd353a7f1f6a3070a68b84f3c4caf438a1 Mon Sep 17 00:00:00 2001 From: Ben Loftis Date: Tue, 3 May 2022 11:51:46 -0500 Subject: [PATCH] crop_region_to_selection: remove redundant check for a Range get_edit_op_range() already checks for the existence of a Range --- gtk2_ardour/editor_ops.cc | 25 +++++++------------------ 1 file changed, 7 insertions(+), 18 deletions(-) diff --git a/gtk2_ardour/editor_ops.cc b/gtk2_ardour/editor_ops.cc index bff50c8d2a..931c152e8e 100644 --- a/gtk2_ardour/editor_ops.cc +++ b/gtk2_ardour/editor_ops.cc @@ -3452,27 +3452,16 @@ Editor::separate_under_selected_regions () void Editor::crop_region_to_selection () { - if (!selection->time.empty()) { + timepos_t start; + timepos_t end; + + if (get_edit_op_range (start, end)) { + begin_reversible_command (_("Crop Regions to Edit Range")); + + crop_region_to (start, end); - begin_reversible_command (_("Crop Regions to Time Selection")); - for (std::list::iterator i = selection->time.begin(); i != selection->time.end(); ++i) { - crop_region_to ((*i).start(), (*i).end()); - } commit_reversible_command(); - } else { - - timepos_t start; - timepos_t end; - - if (get_edit_op_range (start, end)) { - begin_reversible_command (_("Crop Regions to Edit Range")); - - crop_region_to (start, end); - - commit_reversible_command(); - } } - } void