[PATCH] D115972: [clang-format] Fix AlignConsecutiveAssignments breaking lambda formatting.

Marek Kurdej via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 31 01:28:51 PST 2022


curdeius added a comment.

In D115972#3283229 <https://reviews.llvm.org/D115972#3283229>, @MyDeveloperDay wrote:

>> This patch, for the moment, basically disables alignment in the lambdas which is not what I want.
>
> I sort of feel that's ok for now.

Ok.

>> This is still a WIP, but I'd like to have your opinion on how to tackle this problem.
>
> What are we trying to achieve? what is the expected output if not this? From what I can tell what you've put in your lists it looks ok
>
>   auto aaaaaaaaaaaaaaaaaaaaa = {};
>   auto b                     = g([] {
>     return;
>   });

The expected output is ok indeed, I'm rather not sure about the way I solve it here.

But yeah, it's better than what we have now.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115972



More information about the cfe-commits mailing list