[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