« get me outta code hell

Fix SoX player - http-music - Command-line music player + utils (not a server!)
about summary refs log tree commit diff
diff options
context:
space:
mode:
authorFlorrie <towerofnix@gmail.com>2017-08-08 11:41:20 -0400
committerFlorrie <towerofnix@gmail.com>2017-08-08 11:41:20 -0400
commitf0758de5069c2958a8efcb2f6e3b0781b77d6ecb (patch)
tree976f42bfd18cab908f6a119c1485fb3b997b7f7d
parent588482d3dad9a3ff023b6a152c490e375eb6746a (diff)
Fix SoX player
-rw-r--r--src/loop-play.js5
-rw-r--r--todo.txt3
2 files changed, 4 insertions, 4 deletions
diff --git a/src/loop-play.js b/src/loop-play.js
index 1e1a75d..7fec995 100644
--- a/src/loop-play.js
+++ b/src/loop-play.js
@@ -145,10 +145,7 @@ class SoXPlayer extends Player {
     // You don't get keyboard controls such as seeking or volume adjusting
     // with SoX, though.
 
-    this.process = spawn('play', [
-      ...this.playOpts,
-      file
-    ])
+    this.process = spawn('play', [file])
 
     return promisifyProcess(this.process)
   }
diff --git a/todo.txt b/todo.txt
index 98ef844..b8c992d 100644
--- a/todo.txt
+++ b/todo.txt
@@ -275,3 +275,6 @@ TODO: Players (MPV, SoX) should be separate (sub-)classes.
 TODO: FIFO doesn't work on Windows.
       (Done! - Use mpv-nofifo player. Would like to automatically check for
       mkfifo command; then use nofifo if that doesn't exist.)
+
+TODO: It looks like the SoX player is broken? - Fix this!
+      (Done!)