[FFmpeg-devel] [PATCH] av_log repeating

Michael Niedermayer michaelni
Tue Sep 14 13:08:12 CEST 2010


On Tue, Sep 14, 2010 at 11:34:51AM +0100, M?ns Rullg?rd wrote:
> 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,

yeah if you define 1=0


> and that change made no difference to this use case.

pre r24573 (which you want) and head do make a difference to this


> 
> > 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.

if you would stop your idiotic attacks and let people do their work it would
be fixed already


> 
> >> > 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?

when you started to behave like a dog with foam infront of its mouth


> 
> >> 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.

right, thats why i commited different code instead of my original patch
after you expressed dislike for the patch.


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Republics decline into democracies and democracies degenerate into
despotisms. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100914/165e4191/attachment.pgp>



More information about the ffmpeg-devel mailing list