Page MenuHomePhabricator

christophe-calmejane (Christophe Calmejane)
User

Projects

User does not belong to any projects.

User Details

User Since
Oct 4 2018, 12:11 AM (37 w, 4 d)

Recent Activity

Oct 4 2018

christophe-calmejane added a comment to D44609: [Clang-Format] New option BeforeLambdaBody to manage lambda line break inside function parameter call (in Allman style).

I fixed it like this (not sure it's 100% correct though!!)

Oct 4 2018, 9:39 AM · Restricted Project
christophe-calmejane added a comment to D44609: [Clang-Format] New option BeforeLambdaBody to manage lambda line break inside function parameter call (in Allman style).

Ok I found the issue with noexcept, it's not related to your patch.
There is a missing

Oct 4 2018, 2:41 AM · Restricted Project
christophe-calmejane added a comment to D44609: [Clang-Format] New option BeforeLambdaBody to manage lambda line break inside function parameter call (in Allman style).

Actually, without your change to HasMultipleNestedBlocks, I'm almost getting the expected result: Lambda body is correctly indented (and not by function name's length).
The only thing not working (and it's not either way, with or without your change to HasMultipleNestedBlocks), is nested lambdas. The body is not properly indented (but it's not that bad).

Oct 4 2018, 1:48 AM · Restricted Project
christophe-calmejane added a comment to D44609: [Clang-Format] New option BeforeLambdaBody to manage lambda line break inside function parameter call (in Allman style).

consider the following sample code (formatted without your patch)

Oct 4 2018, 12:20 AM · Restricted Project