<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Jan 2, 2014 at 11:21 AM, 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">
Just saw this after Manuel replied.<br>
<br>
Is it OK to reformat IWYU pragmas in other coding styles, say in the LLVM style?<br></blockquote><div><br></div><div>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.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
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.<br></blockquote><div><br></div>
<div>Well, you can set it for all of those? What is your proposed solution?</div><div><br></div><div>Cheers,</div><div>/Manuel</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
It seems prudent not to add a public config setting with a commitment of stability until that's addressed or it'll be hard to fix after the fact.<br>
<br>
Alp.<div><div class="h5"><br>
<br>
On 02/01/2014 09:58, Manuel Klimek wrote:<br>
</div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5">
   lg<br>
<br>
<br>
================<br>
Comment at: unittests/Format/FormatTest.<u></u>cpp:1053<br>
@@ +1052,3 @@<br>
+      "// IWYU pragma: aaaaaaaaaaaaaaaaaa bbbbbbbbbbbbbb",<br>
+      format("// IWYU pragma: aaaaaaaaaaaaaaaaaa bbbbbbbbbbbbbb", Pragmas));<br>
+}<br>
----------------<br>
Please add a short test for block comments.<br>
<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D2460" target="_blank">http://llvm-reviews.chandlerc.<u></u>com/D2460</a><br>
<br>
BRANCH<br>
   svn<br>
<br>
ARCANIST PROJECT<br>
   clang<br></div></div>
______________________________<u></u>_________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/cfe-commits</a><span class="HOEnZb"><font color="#888888"><br>
</font></span></blockquote><span class="HOEnZb"><font color="#888888">
<br>
-- <br>
<a href="http://www.nuanti.com" target="_blank">http://www.nuanti.com</a><br>
the browser experts<br>
<br>
</font></span></blockquote></div><br></div></div>