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

[Michael Niedermayer]

On Mon, Feb 06, 2012 at 08:02:03PM +0100, Reimar Döffinger wrote:
> 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"

changed locally

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

In a rich man's house there is no place to spit but his face.
-- Diogenes of Sinope
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120206/5e84c273/attachment.asc>