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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
