[FFmpeg-devel] [PATCH]Silence a warning for one sample from ticket 973

Carl Eugen Hoyos cehoyos at ag.or.at
Fri Feb 10 17:55:25 CET 2012


On Monday 06 February 2012 05:01:38 pm Michael Niedermayer wrote:

> > Attached trial-and-error patch does not break pink.mpg (issue 1046), but
> > silences the error messages for the first sample in ticket #973, decoding
> > is unchanged.
> 
> iam not sure this makes sense
> the value surely is ignored so the message isnt wrong, maybe its log
> level could be reduced.

New patch attached.

Please comment, Carl Eugen
-------------- next part --------------
diff --git a/libavcodec/mpeg12.c b/libavcodec/mpeg12.c
index 5e325cf..28c17e6 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 specifies invalid DC quantizer %d, ignoring\n", v);
+            av_log(s->avctx, AV_LOG_DEBUG, "intra matrix specifies invalid DC quantizer %d, ignoring\n", v);
             v = 8; // needed by pink.mpg / issue1046
         }
         matrix0[j] = v;


More information about the ffmpeg-devel mailing list