From f8ea94e86cde915b999cec6058077b10980a467d Mon Sep 17 00:00:00 2001 From: Ben Loftis Date: Tue, 8 Nov 2022 20:14:45 -0600 Subject: [PATCH] Fix jump_forward_to_mark in the case where you are already located on a marker * first_mark_after() correctly expects a timepos_t to compare with locations * BUT the playhead position is always in samples, by definition * in many cases this meant the 'next' marker was the one you are already located at * also remove the slight distinction between playhead cursor and actual play position (shouldn't matter, but...) --- gtk2_ardour/editor_ops.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gtk2_ardour/editor_ops.cc b/gtk2_ardour/editor_ops.cc index 0b61bac819..fd4e57003e 100644 --- a/gtk2_ardour/editor_ops.cc +++ b/gtk2_ardour/editor_ops.cc @@ -2455,7 +2455,7 @@ Editor::jump_forward_to_mark () return; } - timepos_t pos = _session->locations()->first_mark_after (timepos_t (_playhead_cursor->current_sample())); + timepos_t pos = _session->locations()->first_mark_after (timepos_t (_session->transport_sample()+1)); if (pos == timepos_t::max (Temporal::AudioTime)) { return;