christophe-calmejane added a comment. In D44609#1875172 <https://reviews.llvm.org/D44609#1875172>, @Wawha wrote:
> In D44609#1871612 <https://reviews.llvm.org/D44609#1871612>, @MyDeveloperDay > wrote: > > > Correct follow that description on how to request commit access > > > It's done. I have the commit right, and push that change on github : > https://github.com/llvm/llvm-project/commit/fa0118e6e588fe303b08e7e06ba28ac1f8d50c68 > > Thank you for the review and advices! Nice to finally see this patch integrated! But, it looks like you didn't include all the test case I posted 1.5y ago in this post, that are still problematic and not formatting correctly with your patch: For example, this simple case do not format correctly: paramBeforeAndAfterLambda(8,[](int x){call();},5); The output is: paramBeforeAndAfterLambda( 8, [](int x) { call(); }, 5); although I would expect to see paramBeforeAndAfterLambda(8, [](int x) { call(); }, 5); See my proposed fix in the discussion, but note that I don't think it's clean enough to be accepted :) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D44609/new/ https://reviews.llvm.org/D44609 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
