[FFmpeg-devel] [PATCH] av_log repeating

Måns Rullgård mans
Tue Sep 14 09:11:50 CEST 2010


Michael Niedermayer <michaelni at gmx.at> writes:

> On Tue, Sep 14, 2010 at 01:30:06AM +0100, M?ns Rullg?rd wrote:
>> Michael Niedermayer <michaelni at gmx.at> writes:
>> 
>> > On Mon, Sep 13, 2010 at 11:46:15PM +0100, M?ns Rullg?rd wrote:
>> >> Michael Niedermayer <michaelni at gmx.at> writes:
>> >> 
>> >> > On Mon, Sep 13, 2010 at 11:13:36PM +0100, M?ns Rullg?rd wrote:
>> >> >> Michael Niedermayer <michaelni at gmx.at> writes:
>> >> >> 
>> >> >> > Hi
>> >> >> >
>> >> >> > patch below disabled the (hated by some) repeating in av_log() and adds an
>> >> >> > option for apps to turn it on again
>> >> >> >
>> >> >> > comments?
>> >> >> >
>> >> >> > (yes ill bump vers & add appchanges entry if people agree to this)
>> >> >> >
>> >> >> > alternatively we can also completely drop the repeat code ...
>> >> >> 
>> >> >> It was fine before your last change.  Now it spews just as many
>> >> >> messages as without repeat detection.  It makes log files horrible to
>> >> >> read if there are many repeated messages.
>> >> >> 
>> >> >> Printing the increasing counter with \r might be acceptable on a
>> >> >> terminal (although it makes the next non-repeated message unreadable
>> >> >> if it is shorter), but not otherwise.
>> >> >
>> >> > we need a solution not complaints
>> >> >
>> >> > btw, isnt there something on posix that can clean these \r in a terminal
>> >> > simulating manner out of a file ?
>> >> 
>> >> Of course something can be done with awk or sed, but that's not the
>> >> point.
>> >
>> > i didnt really mean it as solution for this otherwise i wouldnt have posted
>> > that patch, i was more curious if there was a easy way to flatten all the
>> > codes into a nice ascii/utf8 file. ffmpeg isnt the only thing from where
>> > one can end up with things that need that ...
>> 
>> tr '\r' '\n' might do what you want in some cases.
>> 
>> >> The proper solution is to not print the repeated messages in the first
>> >> place.  We already check for a terminal to enable the colouring of log
>> >> output.  Do the same thing if you insist on printing those lines.
>> >
>> > ok and done
>> 
>> No, not done.
>
> i prefer it as it is now and it fixes the issue that you reported.

No it doesn't.  The issue I reported would have been fixed by
reverting r24573.  I even asked you to do that, as did several others.

> 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, but I DO NOT want the "repeated X times" message only
once, otherwise it is utterly pointless.

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



More information about the ffmpeg-devel mailing list