[PATCH] D116316: [clang-format] Add an experimental option to remove optional control statement braces in LLVM C++ code

[Marek Kurdej via Phabricator via cfe-commits]

curdeius accepted this revision.
curdeius added a comment.
This revision is now accepted and ready to land.

LGTM.



================
Comment at: clang/docs/ClangFormatStyleOptions.rst:3398
 
+**RemoveBracesLLVM** (``Boolean``) :versionbadge:`clang-format 14`
+  Remove optional braces of control statements (``if``, ``else``, ``for``,
----------------
owenpan wrote:
> MyDeveloperDay wrote:
> > Can we agree on one set of options that can be used for both insertion and removal even if this patch only does removal 
> > Can we agree on one set of options that can be used for both insertion and removal even if this patch only does removal 
> 
> One idea is to run insertion for all missing braces and then this option to remove the optional ones for LLVM, i.e., an `AutomaticBraces` if you will.
Have you agreed upon this?


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

https://reviews.llvm.org/D116316