#34 Don't trigger a pipe if already triggered.

Author: Archie3d

Source/aeolus/division.cpp

@@ -609,6 +609,9 @@ bool Division::triggerVoicesForStop(int stopIndex, int note)
 {
     jassert(isPositiveAndBelow(stopIndex, _stops.size()));
 
+    if (isAlreadyVoiced(stopIndex, note))
+        return true;
+
     const auto& stop = _stops[stopIndex];
 
     if (!stop.isEnabled())
@@ -640,4 +643,19 @@ bool Division::triggerVoicesForStop(int stopIndex, int note)
     return voiceTriggered;
 }
 
+bool Division::isAlreadyVoiced(int stopIndex, int note)
+{
+    auto* voice = _activeVoices.first();
+
+    while (voice != nullptr) {
+        if (voice->isActive() && voice->stopIndex() == stopIndex && voice->isForNote(note)) {
+            return true;
+        }
+
+        voice = voice->next();
+    }
+
+    return false;
+}
+
 AEOLUS_NAMESPACE_END

Source/aeolus/division.h

@@ -166,6 +166,8 @@ class Division
 
     bool triggerVoicesForStop(int stopIndex, int note);
 
+    bool isAlreadyVoiced(int stopIndex, int node);
+
     Engine& _engine;
 
     juce::String _name;     ///< The division name.

Source/aeolus/voice.h

@@ -61,7 +61,7 @@ class Voice : public ListItem<Voice>
 private:
     Engine& _engine;
     Pipewave::State _state; ///< Pipe state associated with this voice.
-    
+
     /// Index of the stop associated with this voice.
     /// This is used to tell which stops are voiced.
     int _stopIndex;
@@ -80,7 +80,7 @@ class Voice : public ListItem<Voice>
 
     /// Stereo spatial modeller.
     dsp::SpatialSource _spatialSource;
-    
+
     /// Counter to account for the delayed sound before recycling the voice.
     size_t _postReleaseCounter;
 };