[MPlayer-cvslog] CVS: main/libmpdemux demux_ogg.c,1.69,1.70

Alexander Strasser eclipse7 at gmx.net
Thu May 5 23:59:41 CEST 2005


Rich Felker wrote:
> On Thu, May 05, 2005 at 11:33:26AM +0200, Reimar Döffinger wrote:
> > Also this IMO wasn't a case of the anti-cosmetics rule, I just found it
> > made no difference to the readability of the resulting code and made the
> > patch a tiny bit easier to understand - well, maybe I was wrong, but
> > after all I posted the patch to -dev-eng first for a reason...
> 
> I agree. The code committed was totally readable to anyone familiar
> with C.

I second this. Not knowing about the lang is no excuse here.
Learn about it and it will be readable to you next time.

  Alex (beastd)




More information about the MPlayer-cvslog mailing list