[PATCH] D59627: [clang-format] Keep protobuf "package" statement on one line

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 21 03:20:52 PDT 2019


MyDeveloperDay added inline comments.


================
Comment at: lib/Format/TokenAnnotator.cpp:1079
     if (Style.Language == FormatStyle::LK_Proto && Line.Level == 0 &&
-        CurrentToken->is(Keywords.kw_option)) {
+        (CurrentToken->is(Keywords.kw_option) ||
+         CurrentToken->is(Keywords.kw_package))) {
----------------
Nit:

could we use
```
CurrentToken->isOneOf(Keywords.kw_option,Keyswords.kw_package)
```


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D59627/new/

https://reviews.llvm.org/D59627





More information about the cfe-commits mailing list