[FFmpeg-cvslog] r26215 - trunk/libavcodec/mpegvideo_enc.c

[Stefano Sabatini]

On date Tuesday 2011-01-04 19:57:16 +0100, michael wrote:
> Author: michael
> Date: Tue Jan  4 19:57:16 2011
> New Revision: 26215
> 
> Log:
> Fix YUV444P LJPEG encoding.
> 
> Modified:
>    trunk/libavcodec/mpegvideo_enc.c
> 
> Modified: trunk/libavcodec/mpegvideo_enc.c
> ==============================================================================
> --- trunk/libavcodec/mpegvideo_enc.c	Tue Jan  4 19:18:01 2011	(r26214)
> +++ trunk/libavcodec/mpegvideo_enc.c	Tue Jan  4 19:57:16 2011	(r26215)
> @@ -252,6 +252,12 @@ av_cold int MPV_encode_init(AVCodecConte
>          }
>          break;
>      case CODEC_ID_LJPEG:
> +        if(avctx->pix_fmt != PIX_FMT_YUVJ420P && avctx->pix_fmt != PIX_FMT_YUVJ422P && avctx->pix_fmt != PIX_FMT_YUVJ444P && avctx->pix_fmt != PIX_FMT_RGB32 &&
> +           ((avctx->pix_fmt != PIX_FMT_YUV420P && avctx->pix_fmt != PIX_FMT_YUV422P && avctx->pix_fmt != PIX_FMT_YUV444P) || avctx->strict_std_compliance>FF_COMPLIANCE_UNOFFICIAL)){
> +            av_log(avctx, AV_LOG_ERROR, "colorspace not supported in LJPEG\n");

While at it, please also provide some hint about what the user has to
do to fix the problem without possibly use a debugger, e.g.:
"Invalid colorspace %s, only yuv420p, yuvj420p, and yuv444p are supported in LJPEG."

> +            return -1;

and please meaningful error codes...

[...]
-- 
FFmpeg = Fabulous & Freak MultiPurpose Extended Ghost