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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to