changeset 10711:d2c20d24abbd libavcodec

Change main loop in Interplay Video decoder, so variables x and y really mean coordinates, not offsets.
author kostya
date Sun, 27 Dec 2009 08:15:19 +0000
parents 923f828d1e2c
children 277bf9801257
files interplayvideo.c
diffstat 1 files changed, 6 insertions(+), 5 deletions(-) [+]
line wrap: on
line diff
--- a/interplayvideo.c	Sat Dec 26 11:07:47 2009 +0000
+++ b/interplayvideo.c	Sun Dec 27 08:15:19 2009 +0000
@@ -586,18 +586,19 @@
         + s->avctx->width - 8;
 
     init_get_bits(&gb, s->decoding_map, s->decoding_map_size * 8);
-    for (y = 0; y < (s->stride * s->avctx->height); y += s->stride * 8) {
-        for (x = y; x < y + s->avctx->width; x += 8) {
+    for (y = 0; y < s->avctx->height; y += 8) {
+        for (x = 0; x < s->avctx->width; x += 8) {
             opcode = get_bits(&gb, 4);
 
             debug_interplay("  block @ (%3d, %3d): encoding 0x%X, data ptr @ %p\n",
-                            x - y, y / s->stride, opcode, s->stream_ptr);
+                            x, y, opcode, s->stream_ptr);
 
-            s->pixel_ptr = s->current_frame.data[0] + x;
+            s->pixel_ptr = s->current_frame.data[0] + x
+                          + y*s->current_frame.linesize[0];
             ret = ipvideo_decode_block[opcode](s);
             if (ret != 0) {
                 av_log(s->avctx, AV_LOG_ERROR, " Interplay video: decode problem on frame %d, @ block (%d, %d)\n",
-                       frame, x - y, y / s->stride);
+                       frame, x, y);
                 return;
             }
         }