From 5ebc13be93dd5a740887ae558aca8f69a1fdfd0d Mon Sep 17 00:00:00 2001 From: Paul Licameli Date: Thu, 1 Aug 2019 07:11:43 -0400 Subject: [PATCH] Bug2167 residual... ... See comment 9 of bug report See commit 0192af2e97339f90853a76a9eaaed4f6e780f995 The stopping before restart just needed to be delayed a little further --- src/AdornedRulerPanel.cpp | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/AdornedRulerPanel.cpp b/src/AdornedRulerPanel.cpp index 5cfd76d99..ca05308c2 100644 --- a/src/AdornedRulerPanel.cpp +++ b/src/AdornedRulerPanel.cpp @@ -1686,11 +1686,6 @@ void AdornedRulerPanel::StartQPPlay(bool looped, bool cutPreview) // Looping a tiny selection may freeze, so just play it once. loopEnabled = ((end - start) > 0.001)? true : false; - // Stop only after deciding where to start again, because an event - // callback may change the play region back to the selection - auto &projectAudioManager = ProjectAudioManager::Get( *mProject ); - projectAudioManager.Stop(); - auto options = DefaultPlayOptions( *mProject ); options.playLooped = (loopEnabled && looped); @@ -1705,6 +1700,11 @@ void AdornedRulerPanel::StartQPPlay(bool looped, bool cutPreview) : options.playLooped ? PlayMode::loopedPlay : PlayMode::normalPlay; + // Stop only after deciding where to start again, because an event + // callback may change the play region back to the selection + auto &projectAudioManager = ProjectAudioManager::Get( *mProject ); + projectAudioManager.Stop(); + // Change play region display while playing playRegion.SetTimes( start, end ); Refresh();