[PATCH] D68296: [clang-format] Add ability to wrap braces after multi-line control statements

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 2 13:07:56 PDT 2019


MyDeveloperDay requested changes to this revision.
MyDeveloperDay added a comment.
This revision now requires changes to proceed.

Looks good but I think the option would be better as just `MultiLine`, remember to mark the comments "Done" when you are complete



================
Comment at: clang/docs/ClangFormatStyleOptions.rst:798
+    * ``BWACS_OnlyMultiLine`` (in configuration: ``OnlyMultiLine``)
+      Only wrap braces after a multi-line control statement.
+
----------------
Change to MultiLine to match AlwaysBreakBeforeMultilineStrings 


Repository:
  rC Clang

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

https://reviews.llvm.org/D68296





More information about the cfe-commits mailing list