[FFmpeg-devel] [PATCH] av_log repeating

[Måns Rullgård]

Michael Niedermayer <michaelni at gmx.at> writes:

> as ive said on cvslog, the revission before was buggy
>
> 1. ffmpeg | grep X | wc
>    didnt work

Matter of definition, and that change made no difference to this use case.

> 2. the last repeated message was missing thus leaving you to guess if one
>    or all frames printed it in absence of other messages in between

That could be fixed in a less annoying way.

>> > its also nicer for things like
>> > ffmpeg | grep X | wc
>> > the repeat interfers with such things
>> 
>> So make it do that with -v 2 or something.  I like the repeat
>> suppression,
>
> The problem is that its not compatible with our API
> we would need to require the user to call a av_log_flush() at the end

Not necessarily.

> also theres still the issue that the whole does not mix well with printf()

???

> thus we need a way to disable it, which is what my original patch did.

Disable what?  Which patch do you refer to as original?

>> but I DO NOT want the "repeated X times" message only
>> once, otherwise it is utterly pointless.
>
> you always say something different and your offensive attitude is very
> tireing

As is your stubbornness and unwillingness to listen to others.

-- 
M?ns Rullg?rd
mans at mansr.com