[MPlayer-cvslog] r26411 - trunk/libmpdemux/demuxer.c

Diego Biurrun diego at biurrun.de
Sat Apr 12 18:44:58 CEST 2008


On Sat, Apr 12, 2008 at 06:23:08PM +0200, Reimar Döffinger wrote:
> On Sat, Apr 12, 2008 at 05:51:08PM +0200, uau wrote:
> > 
> > Log:
> > Reindent demuxer.c
> 
> This is not only a reindentation, this also does other cosmetic changes,
> which makes this simply unreviewable,

I had no trouble reviewing the output of

svn diff -r 26410:26411 --diff-cmd diff -x -uwbBE

But of course, Subversion is not configured to output this form of diff.
I have attached it for your convenience.

> and also you are supposed to ask about cosmetic changes before for code
> you don't maintain.
> I have simply had enough, if this is the only way you can behave, I
> simply can't and don't want to work with you, no matter how good your
> code may be.
> I'll maintain my code as long as that is desired, but I simply don't
> want to actively work in this environment anymore.

Reimar, you are overreacting.

That said, I'm not happy about the log message.  It should clearly state
that you also remove commented-out code and useless parentheses.

Diego



More information about the MPlayer-cvslog mailing list