[FFmpeg-devel] [PATCH] H.264 : lowres isn't supported currently so treat a non-zero value as an error.

Stefano Sabatini stefano.sabatini-lala
Sun Jul 4 14:52:31 CEST 2010


On date Sunday 2010-07-04 17:39:26 +0530, Jai Menon encoded:
> ---
>  libavcodec/h264.c |    5 +++++
>  1 files changed, 5 insertions(+), 0 deletions(-)
> 
> diff --git a/libavcodec/h264.c b/libavcodec/h264.c
> index 1054286..5d9ef89 100644
> --- a/libavcodec/h264.c
> +++ b/libavcodec/h264.c
> @@ -903,6 +903,11 @@ av_cold int ff_h264_decode_init(AVCodecContext *avctx){
>      H264Context *h= avctx->priv_data;
>      MpegEncContext * const s = &h->s;
>  
> +    if (avctx->lowres) {
> +        av_log(avctx, AV_LOG_ERROR, "lowres decoding isn't supported by the H.264 decoder\n");
> +        return -1;

AVERROR(EINVAL) or maybe AVERROR_PATCHWELCOME, also "is not" is
favored over "isn't".

Regards.
-- 
FFmpeg = Fancy Furious Mastodontic Portable Epic Gymnast



More information about the ffmpeg-devel mailing list