MyDeveloperDay added a comment.
I think this is LGTM, however..
I've always felt that we shouldn't require `BreakBeforeBraces` to be `Custom`
in order to be able to use the BraceWrapping.XXXX
Users in my view should be able to say:
BreakBeforeBraces: Allman
BraceWrapping:
BeforeLambdaBody: true
This would mean users could pick the "Closest" style to theirs and then modify
just the fields.
Either that or using BraceWrapping with a non Custom style should be a warning.
It doesn't take much to find .clang-format files on github that look like this
F15089833: image.png <https://reviews.llvm.org/F15089833>
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D94906/new/
https://reviews.llvm.org/D94906
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
