[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