[SCM] libav/experimental: correctly deal with the alpha channel in 32-bit QT RLE (courtesy of John Koleszar <jkoleszar at on2.com>)

siretart at users.alioth.debian.org siretart at users.alioth.debian.org
Sun Jun 30 15:45:36 UTC 2013


The following commit has been merged in the experimental branch:
commit e84a0475257a230828160a327beb7e4b48f79359
Author: Mike Melanson <mike at multimedia.cx>
Date:   Wed Dec 14 08:02:03 2005 +0000

    correctly deal with the alpha channel in 32-bit QT RLE (courtesy of John
    Koleszar <jkoleszar at on2.com>)
    
    Originally committed as revision 4743 to svn://svn.ffmpeg.org/ffmpeg/trunk

diff --git a/libavcodec/qtrle.c b/libavcodec/qtrle.c
index c5161c0..262dc0e 100644
--- a/libavcodec/qtrle.c
+++ b/libavcodec/qtrle.c
@@ -411,7 +411,7 @@ static void qtrle_decode_32bpp(QtrleContext *s)
     int rle_code;
     int row_ptr, pixel_ptr;
     int row_inc = s->frame.linesize[0];
-    unsigned char r, g, b;
+    unsigned char a, r, g, b;
     unsigned int argb;
     unsigned char *rgb = s->frame.data[0];
     int pixel_limit = s->frame.linesize[0] * s->avctx->height;
@@ -455,11 +455,11 @@ static void qtrle_decode_32bpp(QtrleContext *s)
                 /* decode the run length code */
                 rle_code = -rle_code;
                 CHECK_STREAM_PTR(4);
-                stream_ptr++;  /* skip the alpha (?) byte */
+                a = s->buf[stream_ptr++];
                 r = s->buf[stream_ptr++];
                 g = s->buf[stream_ptr++];
                 b = s->buf[stream_ptr++];
-                argb = (r << 16) | (g << 8) | (b << 0);
+                argb = (a << 24) | (r << 16) | (g << 8) | (b << 0);
 
                 CHECK_PIXEL_PTR(rle_code * 4);
 
@@ -473,11 +473,11 @@ static void qtrle_decode_32bpp(QtrleContext *s)
 
                 /* copy pixels directly to output */
                 while (rle_code--) {
-                    stream_ptr++;  /* skip the alpha (?) byte */
+                    a = s->buf[stream_ptr++];
                     r = s->buf[stream_ptr++];
                     g = s->buf[stream_ptr++];
                     b = s->buf[stream_ptr++];
-                    argb = (r << 16) | (g << 8) | (b << 0);
+                    argb = (a << 24) | (r << 16) | (g << 8) | (b << 0);
                     *(unsigned int *)(&rgb[pixel_ptr]) = argb;
                     pixel_ptr += 4;
                 }

-- 
Libav/FFmpeg packaging



More information about the pkg-multimedia-commits mailing list