From 47c265f2eca8fbbcfabadfbcee30ab480ef9016b Mon Sep 17 00:00:00 2001 From: Paul Davis Date: Tue, 26 Feb 2019 10:17:04 -0700 Subject: [PATCH] clarifying comment --- libs/ardour/session_transport.cc | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/libs/ardour/session_transport.cc b/libs/ardour/session_transport.cc index fda5bc4e6d..79bdcd76bb 100644 --- a/libs/ardour/session_transport.cc +++ b/libs/ardour/session_transport.cc @@ -845,9 +845,13 @@ Session::non_realtime_stop (bool abort, int on_entry, bool& finished) synced_to_engine()) { // rg: what is the logic behind this case? - // _requested_return_sample should be ignored when synced_to_engine/slaved. - // currently worked around in MTC_Slave by forcing _requested_return_sample to -1 - // 2016-01-10 + // pd: synced_to_engine() really means "JACK Transport + // Master". We need to tell JACK transport to locate to the + // target. + // + // note: should add clause to never do this if using any other + // transport master. 2019-Feb-26 + if ((auto_return_enabled || synced_to_engine() || _requested_return_sample >= 0) && !(ptw & PostTransportLocate)) {