djasper added a comment.
New options for this would not be acceptable IMO. Too much cost for too little
benefit.
I'd suggest to first make the change to fall back to the style with a regular
block when there are statements other than break after the closing brace. That
is always bad, no matter the indentation of the case labels:
switch (x) {
case 1: {
doSomething();
}
doSomethingElse();
break;
}
Fixing this is good no matter what.
And then the second question is whether this style should be used or not (with
IndentCaseLabels: false):
switch (x) {
case 1: {
doSomething();
}
}
Pro: Saves horizontal and vertical space.
Con: It's weird to have to braces in the same column.
I don't personally have an opinion here, but I'll check with a few LLVM
developers who work with LLVM style.
Repository:
rC Clang
https://reviews.llvm.org/D43183
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
