Whitespace cleanups

[Kai Blin]

On Wednesday 25 November 2009 19:02:08 Ken Thomases wrote:
> On Nov 25, 2009, at 11:13 AM, Nate Gallaher wrote:
> > Ken Thomases wrote:
> >> One important reason to avoid whitespace-only changes is it makes
> >> git-blame essentially useless for finding the real source of functional
> >> changes.
> >
> > But is that really important?  git-bisect would put you on the other side
> > of any whitespace changes, and a prudent bug-hunter should be referring
> > to the commit logs anyway.
>
> Who says git-blame is only useful on occasions when it makes sense to
> git-bisect?  Or that one is hunting bugs.
>
> I often use git-blame precisely to figure out which commit log I need to be
> looking at to understand why the code is the way it is.  I see something in
> the code which is done in a way I don't understand/expect.  I git-blame to
> see what commit introduced the code in that form.  I get a commit ID.  I
> look up the commit to see what it contained, what the log message was, what
> commits were around it, etc.

There _is_ a -w option to git-blame that ignores whitespace, though. Adding 
two characters (possibly via an alias) is a small price to pay for 
non-braindead formatting imho.

Cheers,
Kai

-- 
Kai Blin
WorldForge developer  http://www.worldforge.org/
Wine developer        http://wiki.winehq.org/KaiBlin
Samba team member     http://www.samba.org/samba/team/
--
Will code for cotton.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part.
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20091125/ea06a679/attachment.pgp>