Hi,
someone broke the wav codec 6 weeks ago (it only affected the DVI_ADPCM format). Here's the (quick) fix.

BTW, I don't think "exit" (which is a standard C function name) is a good name for a label. EXIT would be better IMHO.

fred
Index: apps/codecs/wav.c
===================================================================
RCS file: /cvsroot/rockbox/apps/codecs/wav.c,v
retrieving revision 1.20
diff -u -r1.20 wav.c
--- apps/codecs/wav.c   18 Jan 2006 20:22:03 -0000      1.20
+++ apps/codecs/wav.c   3 Mar 2006 17:17:12 -0000
@@ -532,12 +532,14 @@
                                  blockalign, channels, bitspersample,
                                  int16_samples+i*samplesperblock*channels,
                                  &decodedsize)
-                != CODEC_OK)
+                != CODEC_OK) {
                 i = CODEC_ERROR;
-            goto exit;
-            if (decodedsize != samplesperblock)
+                goto exit;
+            }
+            if (decodedsize != samplesperblock) {
                 i = CODEC_ERROR;
-            goto exit;
+                goto exit;
+            }
         }
         wavbufsize = nblocks*samplesperblock*channels*2;
     }

Reply via email to