[PATCH] D93846: [clang-format] PR16518 Add flag to suppress empty line insertion before access modifier

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 31 07:11:42 PST 2020


MyDeveloperDay added inline comments.


================
Comment at: clang/unittests/Format/FormatTest.cpp:8544
+TEST_F(FormatTest, FormatsAccessModifiers) {
+  EXPECT_EQ("struct foo {\n"
+            "  int i;\n"
----------------
any reason this can't be verifyFormat?


================
Comment at: clang/unittests/Format/FormatTest.cpp:8565
+               "}\n");
+  EXPECT_EQ("struct foo {\n"
+            "  int i;\n"
----------------
verifyFormat? why not


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

https://reviews.llvm.org/D93846



More information about the cfe-commits mailing list