[PATCH] D50078: clang-format: support aligned nested conditionals formatting

Francois Ferrand via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu May 28 09:50:32 PDT 2020


Typz added a comment.

Regarding `?:`, this is definitely not considered, and should be relatively easy to handle.

Regarding case `D`, this looks like a bug... The break in the first operand is kind of unexpected (as in "not the usual case"), and I had many such corner cases while making and testing the change. I thought I finally had a proper solution, seems there was still something, though I am not sure it is related to `?:` or could happen in other cases (on second thought, it may indeed be related, as there is no operand to align with...)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D50078





More information about the cfe-commits mailing list