[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