[FFmpeg-devel] [PATCH] Fix ljpegenc pix_fmt selection (original) (raw)
Jason Garrett-Glaser jason
Tue Jan 4 19:21:15 CET 2011
- Previous message: [FFmpeg-devel] [PATCH] Fix ljpegenc pix_fmt selection
- Next message: [FFmpeg-devel] [PATCH] Fix ljpegenc pix_fmt selection
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
On Tue, Jan 4, 2011 at 1:18 PM, Michael Niedermayer wrote:
On Sun, Jan 02, 2011 at 12:58:30PM -0500, Jason Garrett-Glaser wrote:
Previously, if you didn't happen to select the right pixfmt via pixfmt, 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 @@ ? ? ?MPVencodeinit, ? ? ?encodepicturelossless, ? ? ?MPVencodeend, + ? ?.pixfmts= (const enum PixelFormat[]){PIXFMTYUVJ420P, PIXFMTYUVJ422P, PIXFMTBGRA, PIXFMTNONE}, ? ? ?.longname = NULLIFCONFIGSMALL("Lossless JPEG"), ?}; that seems not correct if i remove all the pix fmt tests yuv444p works too for example
Yeah, but that should go in a separate patch. This patch is just to fix the current situation: revising the pixfmt checks in mpegvideo is an unrelated problem.
Jason
- Previous message: [FFmpeg-devel] [PATCH] Fix ljpegenc pix_fmt selection
- Next message: [FFmpeg-devel] [PATCH] Fix ljpegenc pix_fmt selection
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]