[PATCH] D100727: [clang-format] Correctly apply AllowShortIfStatementsOnASingleLine: Always to else branch.

Björn Schäpers via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 19 01:18:37 PDT 2021


HazardyKnusperkeks added a comment.

In D100727#2697815 <https://reviews.llvm.org/D100727#2697815>, @curdeius wrote:

> Oh, you're right. Should we go for a separate option then? AllowShortElseStatementsOnASingleLine? What would control else if statements then?

I think I would prefer to expand this enum and deprecate `WithoutElse`, give them better names and documentation. But a different option would also be possible.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100727



More information about the cfe-commits mailing list