[FFmpeg-devel] [PATCH] use av_log_level for x264 log

Limin Wang lance.lmwang
Thu Jul 12 09:35:42 CEST 2007


Hi,

> > Index: libavcodec/libx264.c
> > ===================================================================
> > --- libavcodec/libx264.c	(revision 9606)
> > +++ libavcodec/libx264.c	(working copy)
> > @@ -36,17 +36,7 @@
> >  static void
> >  X264_log(void *p, int level, const char *fmt, va_list args)
> >  {
> > -    static const int level_map[] = {
> > -        [X264_LOG_ERROR]   = AV_LOG_ERROR,
> > -        [X264_LOG_WARNING] = AV_LOG_ERROR,
> > -        [X264_LOG_INFO]    = AV_LOG_INFO,
> > -        [X264_LOG_DEBUG]   = AV_LOG_DEBUG
> > -    };
> > -
> > -    if(level < 0 || level > X264_LOG_DEBUG)
> > -        return;
> > -
> > -    av_vlog(p, level_map[level], fmt, args);
> > +    av_vlog(p, level, fmt, args);
> 
> Loren already pointed out that removing the mapping table is not the 
> correct solution; I just wanted to add that we now have an 
> AV_LOG_WARNING error level, so the table can be updated accordingly.

Yes, didn't notice that. Please add it.

Thanks,
Limin




More information about the ffmpeg-devel mailing list