[cfe-commits] r144171 - in /cfe/trunk/include/clang: AST/ Analysis/ Analysis/Analyses/ Frontend/ Index/ Lex/ Sema/ Serialization/

David Blaikie dblaikie at gmail.com
Wed Nov 9 07:30:35 PST 2011


> Please *do not* make whitespace changes like this. It screws up the results of "svn annotate", which we lean on heavily, and has no actual benefits. Whitespace/formatting tweaks are okay within the specific code that you're working on anyway, as part of an actual semantic change.

Ah, sorry Doug - I was under the impression that 80 col fixes were
desirable even in the absence of other changes (& I've made some like
this before, I guess those ones were small enough). The trailing
whitespace removal was just out of convenience since I was touching
the same files & in some cases the 80 col violations were due to
trailing whitespace.

- David




More information about the cfe-commits mailing list