[FFmpeg-devel] [PATCH]Implement raw 2bpp in mov

[Michael Niedermayer]

On Tue, Jan 05, 2010 at 01:50:30AM +0100, Carl Eugen Hoyos wrote:
> Hi!
>
> Attached patch fixes issue 1528.
>
> Please comment, Carl Eugen

>  rawdec.c |   11 ++++++++++-
>  1 file changed, 10 insertions(+), 1 deletion(-)
> 65841c799e26abc6a5f543839d18a9aef3206dc7  patch1528.diff
> Index: libavcodec/rawdec.c
> ===================================================================
> --- libavcodec/rawdec.c	(revision 21026)
> +++ libavcodec/rawdec.c	(working copy)
> @@ -47,6 +47,7 @@
>  
>  static const PixelFormatTag pixelFormatBpsMOV[] = {
>      { PIX_FMT_MONOWHITE, 1 },
> +    { PIX_FMT_PAL8,      2 },
>      { PIX_FMT_PAL8,      4 },
>      { PIX_FMT_PAL8,      8 },
>      // FIXME swscale does not support 16 bit in .mov, sample 16bit.mov
> @@ -115,15 +116,23 @@
>      frame->top_field_first = avctx->coded_frame->top_field_first;
>  
>      //4bpp raw in avi and mov (yes this is ugly ...)
> -    if(avctx->bits_per_coded_sample == 4 && avctx->pix_fmt==PIX_FMT_PAL8 &&
> +    if(avctx->bits_per_coded_sample <= 4 && avctx->pix_fmt==PIX_FMT_PAL8 &&

i think this should be ==2 || ==4 oterwise ok

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Let us carefully observe those good qualities wherein our enemies excel us
and endeavor to excel them, by avoiding what is faulty, and imitating what
is excellent in them. -- Plutarch
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100105/93561d01/attachment.pgp>