[PATCH] clang-format recognizes alternative binary operator keywords

Nikola Smiljanić popizdeh at gmail.com
Tue May 6 21:17:36 PDT 2014


I don't know anything about this code but I'd really expect both LS_Cpp03 and LS_Cpp11 to mean C++. And this seems to be the case since LangOpts.CPlusPlus is always set to 1. I don't see the value in having the language kind parameter since LK_Cpp is used for all C family of languages.

My suggestion would be to just set CXXOperatorNames to 1, unless there are plans to handle C and C++ differently. But let's see what Manuel has to say.

http://reviews.llvm.org/D3634






More information about the cfe-commits mailing list