WMA: fix loop unrolling in decode_exp_vlc()
authorMåns Rullgård <mans@mansr.com>
Tue, 29 Sep 2009 12:48:24 +0000 (12:48 +0000)
committerMåns Rullgård <mans@mansr.com>
Tue, 29 Sep 2009 12:48:24 +0000 (12:48 +0000)
The count can be a non-multiple of 4 after all.

Originally committed as revision 20081 to svn://svn.ffmpeg.org/ffmpeg/trunk

libavcodec/wmadec.c

index e216de7..5a31963 100644 (file)
@@ -330,12 +330,12 @@ static int decode_exp_vlc(WMACodecContext *s, int ch)
         iv = iptab[last_exp];
         max_scale = v;
         n = *ptr++;
-        do {
-            *q++ = iv;
-            *q++ = iv;
-            *q++ = iv;
-            *q++ = iv;
-        } while (n -= 4);
+        switch (n & 3) do {
+        case 0: *q++ = iv;
+        case 3: *q++ = iv;
+        case 2: *q++ = iv;
+        case 1: *q++ = iv;
+        } while ((n -= 4) > 0);
     }else
         last_exp = 36;
 
@@ -352,12 +352,12 @@ static int decode_exp_vlc(WMACodecContext *s, int ch)
         if (v > max_scale)
             max_scale = v;
         n = *ptr++;
-        do {
-            *q++ = iv;
-            *q++ = iv;
-            *q++ = iv;
-            *q++ = iv;
-        } while (n -= 4);
+        switch (n & 3) do {
+        case 0: *q++ = iv;
+        case 3: *q++ = iv;
+        case 2: *q++ = iv;
+        case 1: *q++ = iv;
+        } while ((n -= 4) > 0);
     }
     s->max_exponent[ch] = max_scale;
     return 0;