[PATCH] D94906: [clang-format] Apply Allman style to lambdas

[MyDeveloperDay via Phabricator via cfe-commits]

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