[PATCH] D114320: [clang-format] Extend AllowShortBlocksOnASingleLine for else blocks

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 22 07:51:11 PST 2021


MyDeveloperDay added a comment.

Could you add the case  where the else is not empty.



================
Comment at: clang/unittests/Format/FormatTest.cpp:5400
+  Style.BraceWrapping.AfterControlStatement = FormatStyle::BWACS_Always;
+  EXPECT_EQ("if (a)\n"
+            "{\n"
----------------
Can you please use verifyFormat  (as this does some extra stuff to ensure its stable)



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114320



More information about the cfe-commits mailing list