[MPlayer-dev-eng] [PATCH] ad_ffmpeg does not honor log level
Vlad Seryakov
vseryakov at gmail.com
Fri Aug 20 20:45:27 CEST 2010
On Aug 20, 2010, at 4:46 AM, Diego Biurrun wrote:
>> --- libmpcodecs/ad_ffmpeg.c (revision 31984)
>> +++ libmpcodecs/ad_ffmpeg.c (working copy)
>> @@ -99,6 +99,18 @@
>> + switch(verbose){
>
> switch (verbose) {
>
>> + case MSGL_FATAL: av_log_set_level(AV_LOG_FATAL); break;
>> + case MSGL_ERR: av_log_set_level(AV_LOG_ERROR); break;
>> + case MSGL_WARN: av_log_set_level(AV_LOG_WARNING); break;
>> + case MSGL_INFO: av_log_set_level(AV_LOG_INFO); break;
>> + case MSGL_V: av_log_set_level(AV_LOG_DEBUG); break;
>
> This could be vertically aligned for better readability.
>
>> + case 7:
>> + case 8:
>> + case 9:
>> + case 10: av_log_set_level(AV_LOG_VERBOSE); break;
>> + }
>
> This could use a default case.
>
I am not sure if INFO is a good choice for default but this is what is default in avutil/log.c so at least it preserves default logging level
Attached
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: avlog.txt
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20100820/0a8f95d5/attachment-0001.txt>
-------------- next part --------------
More information about the MPlayer-dev-eng
mailing list