- User Since
- Oct 4 2018, 12:11 AM (154 w, 1 d)
Jun 16 2021
If you did The only difference (with this current patch) would be the positioning of the first arguments on paramBeforeAndAfterLambda and doubleLambda (I'm not sure the reason for why that is)
Jun 15 2021
Due to the pandemic, I wasn't able to look at https://reviews.llvm.org/D44609 which got broken since it was pushed to LLVM. Nevertheless I commented in that post with a rather complete unit tests that should pass if any changes are to be pushed to that part of the code.
Feb 26 2020
Oct 4 2018
I fixed it like this (not sure it's 100% correct though!!)
Ok I found the issue with noexcept, it's not related to your patch.
There is a missing
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).
consider the following sample code (formatted without your patch)