Re: [GIT PATCH] ACPI patches for 2.6.26-rc5

From: Len Brown
Date: Wed Jun 11 2008 - 23:08:37 EST




> > 27 files changed, 255 insertions(+), 180 deletions(-)
>
> Hmm. I get very different statistics:
>
> 27 files changed, 434 insertions(+), 359 deletions(-)
>
> You seem to use "git diff -w" to ignore whitespace changes, which explains
> it, but please don't - it makes it harder to compare with expected
> results, and it hides some changes entirely.
>
> I do agree that using '-w' is often useful, but especially for the purpose
> of letting people know what changed it kind of avoids the point by hiding
> many things.

Yes, I've been using "-b" recently:

git diff -b -M --stat linus..$RELEASE

and if I remove the "-b", we match:

27 files changed, 434 insertions(+), 359 deletions(-)

I have a vague memory of having a good reason to cut over to "-b"
a while back and noting that in my pull request, but if you prefer
no -b, no problem.

thanks,
-Len
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/