From f0758de5069c2958a8efcb2f6e3b0781b77d6ecb Mon Sep 17 00:00:00 2001 From: Florrie Date: Tue, 8 Aug 2017 11:41:20 -0400 Subject: Fix SoX player --- src/loop-play.js | 5 +---- todo.txt | 3 +++ 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!) -- cgit 1.3.0-6-gf8a5