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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
