[PATCH] D44609: [Clang-Format] New option BreakBeforeLambdaBody to manage lambda line break inside function parameter call

[Manuel Klimek via Phabricator via cfe-commits]

klimek added a comment.

In https://reviews.llvm.org/D44609#1143895, @Wawha wrote:

>   Hello,
>
> after my last modification (require by previous comment), I do not see any feedback or validation for this patch.
>  Is their something special to do in order to go forward on this patch?
>
> Lambda are more an more used in modern C++, and it's very annoying to not have a way to format them in allman style.


Sorry for the delay, pinging it once a week until somebody replies is the right approach.



================
Comment at: lib/Format/ContinuationIndenter.cpp:1308
+       (Style.BraceWrapping.BeforeLambdaBody &&
+        State.Line->containsAfter(Current, TT_LambdaLSquare)));
   State.Stack.back().IsInsideObjCArrayLiteral =
----------------
Why don't lambdas already count as nested blocks? Generally, I'd rather fix BlockParameterCount than introduce another linear scan over the line.


Repository:
  rC Clang

https://reviews.llvm.org/D44609