[PATCH] D44609: [Clang-Format] New option BeforeLambdaBody to manage lambda line break inside function parameter call (in Allman style)

Francois JEAN via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 20 09:21:18 PST 2020


Wawha marked 2 inline comments as done.
Wawha added a comment.

Hello,
@klimek will you have time to review this patch? The current patch integrate the last remarks and also modification due to inline lambda.



================
Comment at: lib/Format/ContinuationIndenter.cpp:1179
+                       Current.is(TT_LambdaLSquare))));
   }
 
----------------
MyDeveloperDay wrote:
> oh boy... I have to say.. that I don't like these kinds of massive if statements without some sort of comment.. as to what it's doing, it's so hard, later on, to read through the various clauses and understand which bit of functionality this section covers
I make some change in the last patch. Hope it's better.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D44609





More information about the cfe-commits mailing list