On Sat, Jul 28, 2012 at 05:14:49PM +0600, Mashiat Sarker Shakkhar wrote:
> 
> --- a/libavcodec/vc1dec.c
> +++ b/libavcodec/vc1dec.c
> @@ -1048,6 +1048,11 @@ static void vc1_mc_4mv_chroma4(VC1Context *v)
>          if ((edges&8) && s->mb_y == (s->mb_height - 1))        \
>              mquant = v->altpq;                                 \
> +        if (!mquant || mquant > 31) {                          \
> +            av_log(v->s.avctx, AV_LOG_ERROR, "Overriding "     \
> +                   invalid mquant %d\n", mquant);              \
> +            mquant = 1;                                        \

nit: Please do not break lines in the middle of strings where
easily avoidable.  It's more readable like this:

> +            av_log(v->s.avctx, AV_LOG_ERROR,                   \
> +                   "Overriding invalid mquant %d\n", mquant);  \

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to