[FFmpeg-devel] [PATCH]Support SOLID_GRAY lagarith frames

[Carl Eugen Hoyos]

On Tuesday 22 January 2013 12:17:27 pm Paul B Mahol wrote:
> >> >          for (j = 0; j < avctx->height; j++) {
> >> > +            if (frametype == FRAME_SOLID_RGBA) {
> >>
> >> if/else in loop when it is not needed/pointless.
> >
> > Please elaborate.
>
> add another loop for gray case.

Done.

New patch attached, thank you, Carl Eugen
-------------- next part --------------
diff --git a/libavcodec/lagarith.c b/libavcodec/lagarith.c
index 486e326..6074a7f 100644
--- a/libavcodec/lagarith.c
+++ b/libavcodec/lagarith.c
@@ -535,6 +535,14 @@ static int lag_decode_frame(AVCodecContext *avctx,
     switch (frametype) {
     case FRAME_SOLID_RGBA:
         avctx->pix_fmt = AV_PIX_FMT_RGB32;
+    case FRAME_SOLID_GRAY:
+        if (frametype == FRAME_SOLID_GRAY)
+            if (avctx->bits_per_coded_sample == 24) {
+                avctx->pix_fmt = AV_PIX_FMT_RGB24;
+            } else {
+                avctx->pix_fmt = AV_PIX_FMT_0RGB32;
+                planes = 4;
+            }
 
         if (ff_thread_get_buffer(avctx, p) < 0) {
             av_log(avctx, AV_LOG_ERROR, "get_buffer() failed\n");
@@ -542,11 +550,16 @@ static int lag_decode_frame(AVCodecContext *avctx,
         }
 
         dst = p->data[0];
+        if (frametype == FRAME_SOLID_RGBA) {
         for (j = 0; j < avctx->height; j++) {
             for (i = 0; i < avctx->width; i++)
                 AV_WN32(dst + i * 4, offset_gu);
             dst += p->linesize[0];
         }
+        } else {
+            for (j = 0; j < avctx->height; j++)
+                memset(dst, buf[1], avctx->width * planes);
+        }
         break;
     case FRAME_ARITH_RGBA:
         avctx->pix_fmt = AV_PIX_FMT_RGB32;