[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