<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Jan 2, 2014 at 4:28 PM, Alp Toker <span dir="ltr"><<a href="mailto:alp@nuanti.com" target="_blank">alp@nuanti.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
On 02/01/2014 15:19, Alexander Kornienko wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
On Thu, Jan 2, 2014 at 11:54 AM, Alexander Kornienko <<a href="mailto:alexfh@google.com" target="_blank">alexfh@google.com</a> <mailto:<a href="mailto:alexfh@google.com" target="_blank">alexfh@google.com</a>>> wrote:<br>

<br>
    On Thu, Jan 2, 2014 at 11:37 AM, Alp Toker <<a href="mailto:alp@nuanti.com" target="_blank">alp@nuanti.com</a><br></div><div class="im">
    <mailto:<a href="mailto:alp@nuanti.com" target="_blank">alp@nuanti.com</a>>> wrote:<br>
<br>
<br>
        On 02/01/2014 10:24, Manuel Klimek wrote:<br>
<br>
            On Thu, Jan 2, 2014 at 11:21 AM, Alp Toker <<a href="mailto:alp@nuanti.com" target="_blank">alp@nuanti.com</a><br></div>
            <mailto:<a href="mailto:alp@nuanti.com" target="_blank">alp@nuanti.com</a>> <mailto:<a href="mailto:alp@nuanti.com" target="_blank">alp@nuanti.com</a><div><div class="h5"><br>
            <mailto:<a href="mailto:alp@nuanti.com" target="_blank">alp@nuanti.com</a>>>> wrote:<br>
<br>
                Just saw this after Manuel replied.<br>
<br>
                Is it OK to reformat IWYU pragmas in other coding<br>
            styles, say in<br>
                the LLVM style?<br>
<br>
<br>
            I think we mainly want to not have *all* pragmas in *all*<br>
            coding styles. IWYU is just an example, and we can easily<br>
            add it to other styles.<br>
<br>
                If not, that's an indication this kind of setting<br>
            should be<br>
                separated from individual coding style descriptions as<br>
            it appears<br>
                equally applicable to WebKit, Chromium, LLVM and<br>
            custom styles.<br>
<br>
<br>
            Well, you can set it for all of those? What is your<br>
            proposed solution?<br>
<br>
<br>
        A quick fix without major surgery on the proposed patch would<br>
        be to have the same default value, or empty default value,<br>
        shared between all built-in coding styles so that switching<br>
        between them doesn't change behaviour.<br>
<br>
        This makes sense because non-reformatted comment pragmas are<br>
        project-level / user-level settings not related to or really<br>
        defined by any coding style.<br>
<br>
        This'll be great for lit RUN lines too(!)<br>
<br>
<br>
    And for CHECK.*: lines as well (i'll update the patch). But this<br>
    is a good example of why it doesn't make sense to make this a<br>
    global setting. I can easily imagine comments starting with " RUN:<br>
    " or " CHECK: ", that don't have special meaning in any code, that<br>
    is not lit tests. And then it would be undesirable to leave these<br>
    comments on a single line, if they exceed column limit.<br>
<br>
<br>
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.<br>
</div></div></blockquote>
<br>
Agree fully. Ditto for IWYU -- no need to hard-code it in GoogleStyle.</blockquote><div><br></div><div>IWYU pragmas are neither limited to a certain narrow kind of C++ files, nor too generic (it's rather unlikely, that someone uses this substring for a different purpose ;). So they can be safely put into all styles (which is done in the committed version of the patch).</div>
</div>
</div></div>