[PATCH] D95168: [clang-format] Add InsertBraces option

Owen Pan via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 3 02:06:59 PDT 2021


owenpan added a comment.

In D95168#3105340 <https://reviews.llvm.org/D95168#3105340>, @MyDeveloperDay wrote:

>> I already had an implementation of RemoveBraces for the LLVM style (minus some exceptions).
>
> Why not share the implementation in a review then we can combine them here.

I want to try implementing the remaining exceptions for LLVM and tighten some loose ends before submitting the patch. I don't think we should combine Insert and Remove (even if it's technically possible) before each has been tested against a large codebase.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95168



More information about the cfe-commits mailing list