[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