From f194becadfc15e8eca9baae44a6caaaf3e51bce8 Mon Sep 17 00:00:00 2001
From: Chris Cannam <cannam@all-day-breakfast.com>
Date: Thu, 29 Sep 2022 15:45:47 +0100
Subject: [PATCH] Use number of actually-populated bands, not total number

---
 src/finer/PhaseAdvance.h | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/finer/PhaseAdvance.h b/src/finer/PhaseAdvance.h
index 6015a22..956ddf2 100644
--- a/src/finer/PhaseAdvance.h
+++ b/src/finer/PhaseAdvance.h
@@ -135,7 +135,8 @@ public:
             for (int i = lowest; i <= highest; ++i) {
                 m_currentPeaks[c][i] = i;
             }
-            for (const auto &band : guidance[c]->phaseLockBands) {
+            for (int i = 0; i < guidance[c]->phaseLockBandCount; ++i) {
+                const auto &band = guidance[c]->phaseLockBands[i];
                 int startBin = binForFrequency
                     (band.f0, m_parameters.fftSize, m_parameters.sampleRate);
                 int endBin = binForFrequency
@@ -187,7 +188,8 @@ public:
             for (int i = lowest; i <= highest; ++i) {
                 process_t f = frequencyForBin
                     (i, m_parameters.fftSize, m_parameters.sampleRate);
-                while (f > g->phaseLockBands[phaseLockBand].f1) {
+                while (f > g->phaseLockBands[phaseLockBand].f1 &&
+                       phaseLockBand + 1 < g->phaseLockBandCount) {
                     ++phaseLockBand;
                 }
                 process_t ph = 0.0;