[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 00:23:49 PST 2022


curdeius added a comment.

This is still a WIP, but I'd like to have your opinion on how to tackle this problem.
What I'd like to do basically is to avoid shifting right the body of the lambda but still align things inside the lambda body.
This patch, for the moment, basically disables alignment in the lambdas which is not what I want.
Also, this approach is ugly.

All these problems were introduced somewhere between release 12 and 13. Does anyone know which commit could provoke this?



================
Comment at: clang/unittests/Format/FormatTest.cpp:16605
+               Alignment);
+  verifyFormat("auto aaaaaaaaaaaa = f();\n"
+               "auto b            = f(aaaaaaaaaaaaaaaaaaaaaaaaa,\n"
----------------
This test case still fails, but I'm inclined to keep it commented for the moment and fix it later.


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