[PATCH] D101033: [clang-format] fix indent in alignChainedConditionals
Sam McCall via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Apr 23 05:37:27 PDT 2021
sammccall accepted this revision.
sammccall added a comment.
This revision is now accepted and ready to land.
Can't say I really understand this (either the before/after state).
It seems like we're trying to vertically-align a bunch of operands to a ?: chain, and this patch stops aligning the ones that follow a ? in certain cases. But I don't understand why this is correct, and what the point is of aligning (only) the ones following `:`.
Is there an intuitive explanation?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D101033/new/
https://reviews.llvm.org/D101033
More information about the cfe-commits
mailing list