From b845e9c90a4939c796ec30dda98743e368b33e85 Mon Sep 17 00:00:00 2001 From: David Robillard Date: Sat, 17 Mar 2007 06:55:53 +0000 Subject: [PATCH] Fix looping (and other events) at varispeed. git-svn-id: svn://localhost/ardour2/trunk@1612 d708f5d6-7413-0410-9779-e7cbd77b26cf --- libs/ardour/session_process.cc | 28 ++++++++++------------------ 1 file changed, 10 insertions(+), 18 deletions(-) diff --git a/libs/ardour/session_process.cc b/libs/ardour/session_process.cc index 9d5748a332..bab9db928b 100644 --- a/libs/ardour/session_process.cc +++ b/libs/ardour/session_process.cc @@ -284,7 +284,7 @@ Session::process_with_events (nframes_t nframes) return; } - end_frame = _transport_frame + nframes; + end_frame = _transport_frame + (nframes_t)abs(floor(nframes * _transport_speed)); { Event* this_event; @@ -330,20 +330,15 @@ Session::process_with_events (nframes_t nframes) while (nframes) { - if (this_event == 0 || this_event->action_frame > end_frame || this_event->action_frame < _transport_frame) { + this_nframes = nframes; /* real (jack) time relative */ + frames_moved = (long) floor (_transport_speed * nframes); /* transport relative */ - this_nframes = nframes; - - } else { - - /* compute nframes to next event */ - - if (this_event->action_frame < end_frame) { - this_nframes = nframes - (end_frame - this_event->action_frame); - } else { - this_nframes = nframes; - } - } + /* running an event, position transport precisely to its time */ + if (this_event && this_event->action_frame <= end_frame && this_event->action_frame >= _transport_frame) { + /* this isn't quite right for reverse play */ + frames_moved = (long) (this_event->action_frame - _transport_frame); + this_nframes = (nframes_t) abs( floor(frames_moved / _transport_speed) ); + } if (this_nframes) { @@ -362,8 +357,6 @@ Session::process_with_events (nframes_t nframes) nframes -= this_nframes; offset += this_nframes; - frames_moved = (nframes_t) floor (_transport_speed * this_nframes); - if (frames_moved < 0) { decrement_transport_position (-frames_moved); } else { @@ -395,8 +388,7 @@ Session::process_with_events (nframes_t nframes) } /* this is necessary to handle the case of seamless looping */ - /* not sure if it will work in conjuction with varispeed */ - end_frame = _transport_frame + nframes; + end_frame = _transport_frame + (nframes_t) floor (nframes * _transport_speed); } -- 2.30.2