[FFmpeg-devel] [PATCH] avformat/dump.c: fix mixed log levels
t.rapp at noa-archive.com
Thu May 12 16:38:41 CEST 2016
On 12.05.2016 16:00, Michael Niedermayer wrote:
> On Thu, May 12, 2016 at 10:26:03AM +0200, Tobias Rapp wrote:
>> Previously a partial log message without newline was printed in case of
>> Signed-off-by: Tobias Rapp <t.rapp at noa-archive.com>
>> libavformat/dump.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>> diff --git a/libavformat/dump.c b/libavformat/dump.c
>> index d6a3249..9eb6146 100644
>> --- a/libavformat/dump.c
>> +++ b/libavformat/dump.c
>> @@ -422,7 +422,7 @@ static void dump_sidedata(void *ctx, AVStream *st, const char *indent)
>> dump_mastering_display_metadata(ctx, &sd);
>> - av_log(ctx, AV_LOG_WARNING,
>> + av_log(ctx, AV_LOG_INFO,
> this is ok if its wanted to change the log level
> is that wanted ?
> if not the \n would have to be printed with the max log level
> that was used so its not missning
I guess this wouldn't be enough. Also the whitespace separator log level
has to be adapted, which will then produce confusing output in case of
mixed known/unknown side data together with loglevel=warning.
A better option would be to assemble the log message into a buffer first
using av_bprintf and then av_log the complete message either with
AV_LOG_INFO or AV_LOG_WARNING depending on whether only known side data
was found or not?
Or maybe always print the (assembled) side data info with AV_LOG_INFO
plus additional AV_LOG_WARNING for each unknown side data item?
More information about the ffmpeg-devel