From cf3c32c722c525639f39eb1bd50a395a69042f52 Mon Sep 17 00:00:00 2001
From: Paul Walker <paul@pwjw.com>
Date: Sun, 6 Feb 2022 18:06:09 -0500
Subject: [PATCH] Remove PPQ advance check in looping mode

I confused 'isLooping' to mean "is looping" when it actually
means "would be looping if playing" which meant my ppqpos
reset was improper.

Closes #5812
---
 src/surge-xt/SurgeSynthProcessor.cpp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/surge-xt/SurgeSynthProcessor.cpp b/src/surge-xt/SurgeSynthProcessor.cpp
index a480761f615..c2d54f4f207 100644
--- a/src/surge-xt/SurgeSynthProcessor.cpp
+++ b/src/surge-xt/SurgeSynthProcessor.cpp
@@ -209,7 +209,8 @@ void SurgeSynthProcessor::processBlock(juce::AudioBuffer<float> &buffer,
         playhead->getCurrentPosition(cp);
         surge->time_data.tempo = cp.bpm;
 
-        if (cp.isPlaying || cp.isRecording || cp.isLooping)
+        // isRecording should always imply isPlaying but better safe than sorry
+        if (cp.isPlaying || cp.isRecording)
             surge->time_data.ppqPos = cp.ppqPosition;
 
         surge->time_data.timeSigNumerator = cp.timeSigNumerator;