[PATCH] D151761: clang-format: Add AlignConsecutiveShortCaseStatements

[Galen Elias via Phabricator via cfe-commits]

galenelias marked 6 inline comments as done.
galenelias added a comment.

In D151761#4524653 <https://reviews.llvm.org/D151761#4524653>, @owenpan wrote:

> FWIW, I think we can use a shorter name `AlignConsecutiveCaseStatements` instead of `AlignConsecutiveShortCaseStatements`.

My only hesitation with that name is that it might seem like something like there should be some alignment being applied to 'normal' consecutive case statements, which there isn't.  Maybe it's fine because the documentation makes it clear?  I'm definitely not picky about the name, whatever sounds idiomatic.  @HazardyKnusperkeks, thoughts on just `AlignConsecutiveCaseStatements`?

  switch (level) {
  case 0:
  case 100:
     return "error";
  }


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

https://reviews.llvm.org/D151761