<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, May 5, 2014 at 12:39 PM, Bobby Moretti <span dir="ltr"><<a href="mailto:bobmoretti@gmail.com" target="_blank">bobmoretti@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thanks everyone for the feedback.<br>
<br>
It sounds like folks are ok with enabling the keyword binary operators<br>
by default, and not adding a new option to clang-format.<br>
<br>
Do we want to enable this even if "Language" is not "Cpp" in the Style options?<br>
<br>
One thing that I find confusing: in the same function,<br>
LangOpts.CPlusPlus is always set to 1. Does this mean that<br>
clang-format always lexes all source files using C++ keywords? Even if<br>
they are C or ObjC files?<br>
<br>
I'd be happy to submit a patch, but I'd like to get the exact behavior<br>
nailed down beforehand, since I'm new to clang/llvm community.<br>
<br>
Also, FWIW, you can also keep MSVC happy with the operator keywords by<br>
passing /Za to the compiler.</blockquote><div><br></div><div>Incidentally: that flag doesn't actually make MSVC handle these keywords properly; they instead appear to be faked up by #defines (which in turn breaks some versions of boost...) </div>
</div></div></div>