[MPlayer-cvslog] r23703 - in trunk/gui: app.h mplayer/gmplayer.h mplayer/gtk/mb.h mplayer/gtk/opts.h mplayer/widgets.h skin/skin.h wm/ws.h wm/wskeys.h wm/wsxdnd.h

Diego Biurrun diego at biurrun.de
Wed Jul 4 18:06:22 CEST 2007


On Wed, Jul 04, 2007 at 06:57:46PM +0300, Uoti Urpala wrote:
> On Wed, 2007-07-04 at 12:11 +0200, Diego Biurrun wrote:
> > On Tue, Jul 03, 2007 at 05:42:15PM +0300, Uoti Urpala wrote:
> > > What do you mean by "splitting"? Didn't you change all of those lines
> > > again, thus not making the later commits any smaller? To me it looks
> > > like you made a big commit even bigger by committing completely useless
> > > partially-fixed versions AND then the fixed version.
> > 
> > Just removing the underscores made the GUI not compile anymore (some
> > multiple inclusion guards were duplicated), so I had to change those
> > first in order to be able to have one commit just remove the
> > underscores.  Some other inconsistencies I noticed in between and
> > committed them in batches as I found them.
> 
> Was there any benefit compared to picking a consistent naming scheme and
> then changing everything to use that at once (or divided in parts by
> header, but not changing any one header more than once)?

Maybe, maybe not.  But you know what?  I don't really give a damn.  It's
done now, we can enjoy the end result.

Diego



More information about the MPlayer-cvslog mailing list