[PATCH] clang-format recognizes alternative binary operator keywords

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


Theoretically yes but this is true for any C++ keyword as proto compiler turns message fields into data members with decorated names (appends underscore).

I'd say that this test is broken. If clang format is supposed to format proto files the same way it does C++ files than proto definitions can't use C++ tokens as field names. I'd say that this test needs to be modified. I think the same would happen if test declared message field like this 'optional int32 class'?

http://reviews.llvm.org/D3634






More information about the cfe-commits mailing list