On 02/01/2014 15:19, Alexander Kornienko wrote:
On Thu, Jan 2, 2014 at 11:54 AM, Alexander Kornienko
<[email protected] <mailto:[email protected]>> wrote:
On Thu, Jan 2, 2014 at 11:37 AM, Alp Toker <[email protected]
<mailto:[email protected]>> wrote:
On 02/01/2014 10:24, Manuel Klimek wrote:
On Thu, Jan 2, 2014 at 11:21 AM, Alp Toker <[email protected]
<mailto:[email protected]> <mailto:[email protected]
<mailto:[email protected]>>> wrote:
Just saw this after Manuel replied.
Is it OK to reformat IWYU pragmas in other coding
styles, say in
the LLVM style?
I think we mainly want to not have *all* pragmas in *all*
coding styles. IWYU is just an example, and we can easily
add it to other styles.
If not, that's an indication this kind of setting
should be
separated from individual coding style descriptions as
it appears
equally applicable to WebKit, Chromium, LLVM and
custom styles.
Well, you can set it for all of those? What is your
proposed solution?
A quick fix without major surgery on the proposed patch would
be to have the same default value, or empty default value,
shared between all built-in coding styles so that switching
between them doesn't change behaviour.
This makes sense because non-reformatted comment pragmas are
project-level / user-level settings not related to or really
defined by any coding style.
This'll be great for lit RUN lines too(!)
And for CHECK.*: lines as well (i'll update the patch). But this
is a good example of why it doesn't make sense to make this a
global setting. I can easily imagine comments starting with " RUN:
" or " CHECK: ", that don't have special meaning in any code, that
is not lit tests. And then it would be undesirable to leave these
comments on a single line, if they exceed column limit.
Thinking a bit more, it seems like a better idea to put the setting
for RUN: and CHECK.*: lines in .clang-format inside the tests/
directories.
Agree fully. Ditto for IWYU -- no need to hard-code it in GoogleStyle.
Alp.
--
http://www.nuanti.com
the browser experts
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
