From 34e1e00f8bd42ac531537de8c7a7a6dc9b23228c Mon Sep 17 00:00:00 2001 From: Colin Fletcher Date: Sun, 14 Jun 2015 18:07:27 +0100 Subject: [PATCH] Insert time: fix moving of range markers When inserting time and moving a range marker, move its end first, before moving its start, so that if the time being inserted is greater than the length of the range, we don't try to set its start to after its end. --- gtk2_ardour/editor_ops.cc | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/gtk2_ardour/editor_ops.cc b/gtk2_ardour/editor_ops.cc index 5c45951e04..c46b49505c 100644 --- a/gtk2_ardour/editor_ops.cc +++ b/gtk2_ardour/editor_ops.cc @@ -7038,10 +7038,11 @@ Editor::insert_time ( if ((*i)->position_lock_style() == AudioTime || glued_markers_too) { if ((*i)->start() >= pos) { - (*i)->set_start ((*i)->start() + frames); + // move end first, in case we're moving by more than the length of the range if (!(*i)->is_mark()) { (*i)->set_end ((*i)->end() + frames); } + (*i)->set_start ((*i)->start() + frames); moved = true; }