[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