[FFmpeg-devel] [PATCH] Fix ljpegenc pix_fmt selection

Michael Niedermayer michaelni
Tue Jan 4 19:18:49 CET 2011


On Sun, Jan 02, 2011 at 12:58:30PM -0500, Jason Garrett-Glaser wrote:
> Previously, if you didn't happen to select the right pix_fmt via
> pix_fmt, ffmpeg would fail to convert to the pixel formats supported
> by ljpeg.
> 
> Jason

> Index: libavcodec/ljpegenc.c
> ===================================================================
> --- libavcodec/ljpegenc.c	(revision 26200)
> +++ libavcodec/ljpegenc.c	(working copy)
> @@ -194,5 +194,6 @@
>      MPV_encode_init,
>      encode_picture_lossless,
>      MPV_encode_end,
> +    .pix_fmts= (const enum PixelFormat[]){PIX_FMT_YUVJ420P, PIX_FMT_YUVJ422P, PIX_FMT_BGRA, PIX_FMT_NONE},
>      .long_name = NULL_IF_CONFIG_SMALL("Lossless JPEG"),
>  };

that seems not correct
if i remove all the pix fmt tests yuv444p works too for example

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

I do not agree with what you have to say, but I'll defend to the death your
right to say it. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20110104/17b2f5f8/attachment.pgp>



More information about the ffmpeg-devel mailing list