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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
