On Fri, Jan 13, 2012 at 02:25:15PM -0600, Daniel Huang wrote:
> 
> --- a/libavformat/mov.c
> +++ b/libavformat/mov.c
> @@ -731,9 +733,11 @@ static int mov_read_mdhd(MOVContext *c, AVIOContext *pb, 
> MOVAtom atom)
>      version = avio_r8(pb);
> -    if (version > 1)
> -        return -1; /* unsupported */
> -
> +    if (version > 1) {
> +        av_log_ask_for_sample(c, "unsupported version %d\n",
> +                              version);
> +        return AVERROR_PATCHWELCOME; /* unsupported */
> +    }

The line is short enough, no need to break it.  Also, the comment
is redundant.

I'll queue the patch with this changed in a moment.

Diego
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to