[cfe-commits] r116990 - in /cfe/trunk: include/clang/Basic/LangOptions.h include/clang/Driver/CC1Options.td include/clang/Driver/Options.td lib/Basic/Targets.cpp lib/Driver/Tools.cpp lib/Frontend/CompilerInvocation.cpp lib/Serialization/ASTReader

[Michael Spencer]

On Wed, Oct 20, 2010 at 11:48 PM, Douglas Gregor <dgregor at apple.com> wrote:
>
> On Oct 20, 2010, at 8:16 PM, Michael J. Spencer wrote:
>
>> Author: mspencer
>> Date: Wed Oct 20 22:16:25 2010
>> New Revision: 116990
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=116990&view=rev
>> Log:
>> Fix Whitespace.
>
> Please don't do this. The only thing it does is confuse "svn blame". Fix whitespace locally when you're making a change to that code anyway.
>
>        - Doug

I was about to make a commit that changed all these files. I wanted to
keep the changes separate.

As for blame:
svn blame -x-w (or -x-b)

However, I will not do this in the future on clang.

(I still think we should add a svn commit hook to reject any commits
with trailing whitespace).

- Michael Spencer