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

[Benoit Fouet]

Limin Wang wrote:
> 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.
>
>   

done

-- 
Ben
Purple Labs S.A.
www.purplelabs.com