IFF: decode last 8 pixels per line

The decodeplane8() function processes one byte of input less than
it should.  Also, the for loop has an unusual style with side-effects
in the controlling expression; replaced with a more intuitive while
loop.

10l to Basty.

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

diff --git a/libavcodec/iff.c b/libavcodec/iff.c
index e563de1..d23837e 100644 (file)
--- a/libavcodec/iff.c
+++ b/libavcodec/iff.c
@@ -141,9 +141,10 @@ static av_cold int decode_init(AVCodecContext *avctx)
 static void decodeplane8(uint8_t *dst, const uint8_t *buf, int buf_size, int bps, int plane)
 {
     const uint64_t *lut = plane8_lut[plane];
-    for(; --buf_size != 0; dst += 8) {
+    while (buf_size--) {
         uint64_t v = AV_RN64A(dst) | lut[*buf++];
         AV_WN64A(dst, v);
+        dst += 8;
     }
 }