r174504 - Optionally derive formatting information from the input file.
silvas at purdue.edu
Wed Feb 6 15:07:47 PST 2013
> Yes, this comes up every time one argues with C++ devs :)
> There are multiple issues at play here:
> - this change only detects configuration options, it doesn't actually change
> the behavior depending on how code was formatted before in a non-canonical
Ah, that's an astute observation.
> - we already configure code according to the previous whitespaces -
> obviously macros need this, but we also derive where comments belong to by
> looking at whitespace around the comment
Makes sense. Interesting.
> - applying "don't touch" to StringSwitch is a pretty strong example for the
> other end of the spectrum, and I'd strongly vote against doing this for as
> long as possible ;)
The thing about StringSwitch (at least for me) isn't "don't touch",
but rather recognize "these should be one per line".
-- Sean Silva
More information about the cfe-commits