Fixed issue with flush reading array out of bounds (#3)

* Fixed issue with flush reading array out of bounds * updated minor version number * ran build commands
velochy · Jul 12, 2019 · 2f40479 · 2f40479
1 parent a7873bd
commit 2f40479
Show file tree

Hide file tree

Showing 8 changed files with 8 additions and 8 deletions.
diff --git a/dist/AudioTempoChanger.js b/dist/AudioTempoChanger.js
diff --git a/dist/AudioTempoChanger.js.map b/dist/AudioTempoChanger.js.map
diff --git a/dist/AudioTempoChanger.min.js b/dist/AudioTempoChanger.min.js
diff --git a/dist/AudioTempoChanger.min.js.map b/dist/AudioTempoChanger.min.js.map
diff --git a/dist/test.js b/dist/test.js
diff --git a/dist/test.js.map b/dist/test.js.map
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "audio-tempo-changer.js",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "Portable pure-JS implementation of a phase vocoder for changing audio tempo",
   "main": "dist/AudioTempoChanger.js",
   "repository": {

diff --git a/src/audio_tempo_changer.js b/src/audio_tempo_changer.js
@@ -109,7 +109,7 @@
 
 				// Clear the now-made-future tempo changes (if any)
 				var ci = changes.length-1;
-				while(out_time<=changes[ci].out_time && ci>=0) { changes.pop(); ci--; }
+				while(ci > 0 && out_time <= changes[ci].out_time) { changes.pop(); ci--; }
 
 				// Add a tempo change reflecting current state
 				changes.push({