[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
bigcheesegs at gmail.com
Wed Oct 20 21:16:51 PDT 2010
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
>> 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
More information about the cfe-commits