[FFmpeg-devel] [PATCH] Add log severity level to default log formatting

[hjiodjf 97xgw46]

On Thu, Apr 24, 2014 at 2:18 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
> On Thu, Apr 24, 2014 at 10:31:00AM -0700, hjiodjf 97xgw46 wrote:
>> diff --git a/cmdutils_common_opts.h b/cmdutils_common_opts.h
>> index 190dba6..4dde4fc 100644
>> --- a/cmdutils_common_opts.h
>> +++ b/cmdutils_common_opts.h
>> @@ -22,6 +22,7 @@
>>      { "max_alloc"  , HAS_ARG,  {.func_arg = opt_max_alloc},     "set maximum size of a single allocated block", "bytes" },
>>      { "cpuflags"   , HAS_ARG | OPT_EXPERT, { .func_arg = opt_cpuflags }, "force specific cpu flags", "flags" },
>>      { "hide_banner", OPT_BOOL | OPT_EXPERT, {&hide_banner},     "do not show program banner", "hide_banner" },
>> +    { "print_level", 0,        {(void*)opt_print_level},        "include log level in console output"},
>
> this will fail with av_log() that are run before the main command line
> parsing.
> the log level is special cased so its parsed earlier
>
> also it seems this patch breaks the output
> there seem alot more "Last message repeated" messages than
> actually repeated messages

I forgot to update the code that stores the last printed line. I have
attached a fixed patch that avoids the erroneous "Last message
repeated" events and hooks into parse_loglevel to allow initialization
before option parsing.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Add-AV_LOG_PRINT_LEVEL-flag-to-include-log-severity-.patch
Type: application/octet-stream
Size: 7815 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140424/7f3dac71/attachment.obj>