Merge pull request #346 from grazder/ringbuffer-fix

fixed ring buffer indexes updates in design patterns
GoogleChromeLabs · Oct 13, 2023 · 6b3e732 · 6b3e732
2 parents bfbdd6e + 43803d9
commit 6b3e732
Showing 1 changed file with 2 additions and 8 deletions.
diff --git a/src/audio-worklet/design-pattern/lib/wasm-audio-helper.js b/src/audio-worklet/design-pattern/lib/wasm-audio-helper.js
@@ -222,10 +222,7 @@ class RingBuffer {
       }
     }
 
-    this._writeIndex += sourceLength;
-    if (this._writeIndex >= this._length) {
-      this._writeIndex = 0;
-    }
+    this._writeIndex = (this._writeIndex + sourceLength) % this._length;
 
     // For excessive frames, the buffer will be overwritten.
     this._framesAvailable += sourceLength;
@@ -258,10 +255,7 @@ class RingBuffer {
       }
     }
 
-    this._readIndex += destinationLength;
-    if (this._readIndex >= this._length) {
-      this._readIndex = 0;
-    }
+    this._readIndex = (this._readIndex + destinationLength) % this._length;
 
     this._framesAvailable -= destinationLength;
     if (this._framesAvailable < 0) {