[MPlayer-cvslog] r34228 - trunk/stream/stream.h

Diego Biurrun diego at biurrun.de
Fri Oct 21 17:17:44 CEST 2011


On Fri, Oct 21, 2011 at 11:42:47AM +0000, Carl Eugen Hoyos wrote:
> Reimar Döffinger <Reimar.Doeffinger <at> gmx.de> writes:
> 
> > Including the actual error or warning for error or warning fixes is
> > certainly recommended.
> 
> While I don't see an actual warning included in the original commit,

So one idiocy justifies the next now?

My commit eliminated more than a half dozen warnings.  Pasting them into
the log message creates quite a bit of clutter for little gain; the gcc
warnings from wrong printf modifiers are not terribly exciting.

Details about what configuration a printf conversion specifier broke on
the other hand is far from obvious.  I tested on x86_64 and x86_32 and
you can of course assume that it compiled for me.

> I still believe testing on DragonFly would be more important.

Apparently nobody cares about DragonFly...

> Can anybody do this? I already have to spend hours to test the traitors
> regressions....

lol

Diego


More information about the MPlayer-cvslog mailing list