[PATCH] D93938: [clang-format] Fixed AfterEnum handling
MyDeveloperDay via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sun Jan 3 06:39:31 PST 2021
MyDeveloperDay added inline comments.
================
Comment at: clang/unittests/Format/FormatTest.cpp:1346
verifyFormat("enum { A, B, C } ShortEnum1, ShortEnum2;", Style);
Style.AllowShortEnumsOnASingleLine = false;
+ verifyFormat("enum {\n"
----------------
EXPECT_FALSE(Style.BraceWrapping.AfterEnum)
================
Comment at: clang/unittests/Format/FormatTest.cpp:1352
"} ShortEnum1, ShortEnum2;",
Style);
}
----------------
please add
```
verifyFormat("enum {\n"
"A,\n"
"B,\n"
"} ShortEnum",
Style);
```
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D93938/new/
https://reviews.llvm.org/D93938
More information about the cfe-commits
mailing list