shithub: sox

Info  •  Files  •  Log  •  Branches

There's a multi-channel in trim.

--- a/TODO

+++ b/TODO

@@ -86,3 +86,5 @@

   o Effects option-parsers should fail if extraneous characters or arguments

     are present. e.g. vol 3mW, speed 2 3, etc. should all fail.

+

+  o Fix the multi-channel bug in trim.

--- a/src/trim.c

+++ b/src/trim.c

@@ -129,6 +129,12 @@

     /* Compute the most samples we can process this time */

     done = ((*isamp < *osamp) ? *isamp : *osamp);

+    if (done % effp->ininfo.channels) {

+      st_fail("FIXME: multi-channel not implemented properly");

+      *isamp = *osamp = 0;

+      return ST_EOF;

+    }

+

     /* Quick check to see if we are trimming off the back side yet.

      * If so then we can skip trimming from the front side.

      */