[FFmpeg-devel] [PATCH 2/3] mpeg12dec: pringt the value that is being ignored in load_matrix()

[Reimar Döffinger]

there is a 'g' too much in "print" in the subject.

On 6 Feb 2012, at 17:36, Michael Niedermayer <michaelni at gmx.at> wrote:

> Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> ---
> libavcodec/mpeg12.c |    2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/libavcodec/mpeg12.c b/libavcodec/mpeg12.c
> index 1dae17f..4875798 100644
> --- a/libavcodec/mpeg12.c
> +++ b/libavcodec/mpeg12.c
> @@ -1497,7 +1497,7 @@ static int load_matrix(MpegEncContext *s, uint16_t matrix0[64], uint16_t matrix1
>             return -1;
>         }
>         if (intra && i == 0 && v != 8) {
> -            av_log(s->avctx, AV_LOG_ERROR, "intra matrix invalid, ignoring\n");
> +            av_log(s->avctx, AV_LOG_ERROR, "intra matrix DC is %d, invalid, ignoring\n", v);

IMO it is more readable as
"intra matrix specifies invalid DC quantizer %d, ignoring"
or something like that (that it is a quantizer is just a guess).