<div dir="ltr"><div class="gmail_default" style>Yes, we need an option that describes what clang-format should output and that might be different from what it reads. And, as Chandler said, we need to take the different warnings that users might have enabled into account so that we don't produce code that generates warnings. So this might need some more thinking.</div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jan 16, 2013 at 11:36 AM, Kim Gräsman <span dir="ltr"><<a href="mailto:kim.grasman@gmail.com" target="_blank">kim.grasman@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Wed, Jan 16, 2013 at 10:42 AM, Chandler Carruth <<a href="mailto:chandlerc@google.com">chandlerc@google.com</a>> wrote:<br>

><br>
</div><div class="im">> On Tue, Jan 15, 2013 at 11:30 PM, David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br>
>><br>
>> On Jan 15, 2013 11:21 PM, "Daniel Jasper" <<a href="mailto:djasper@google.com">djasper@google.com</a>> wrote:<br>
>> ><br>
>> > Author: djasper<br>
>> > Date: Wed Jan 16 01:19:28 2013<br>
>> > New Revision: 172601<br>
>> ><br>
>> > URL: <a href="http://llvm.org/viewvc/llvm-project?rev=172601&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=172601&view=rev</a><br>
>> > Log:<br>
>> > Never merge < and ::, as it produces different tokens.<br>
>><br>
>> Shouldn't this be std-specific, since this works in C++11? Speaking of<br>
>> which - what does clang-format do with the >> in nested template types?<br>
><br>
> Agreed on both fronts -- this needs to be conditioned on the standard<br>
> version, and >> in nested templates should be handled similarly....<br>
<br>
</div>Just to be clear -- do you think this should be conditioned on -std or<br>
on a separate compatibility switch?<br>
<br>
I don't think conditioning it on -std works because I may be<br>
formatting portable source code with clang-format that must be<br>
buildable with an older (C++98) or incompatible (e.g. MSVC) compiler.<br>
<br>
I think this is what your follow-up paragraph said, but I wanted to<br>
double-check.<br>
<span class="HOEnZb"><font color="#888888"><br>
- Kim<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">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/mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br></div>