Typz added a comment. > I don't agree that that's the same thing. The closing brace is still neatly > aligned with the line of the opening brace (which happens to be just the > opening brace).
This invariant is not really applicable to switch statements, where code of each "branch" is already indented with no opening brace. :-) I can try to change the mode so that we get either "google" style: switch (x) { case 0: { foo(): } break; } or "regular indentation" mode (e.g. what would look like no special indentation rule): switch (x) { case 0: { foo(): } break; } Generally, would a flag for `CaseBlockIndent` be acceptable for this purpose ? Or should the behavior simply be selected depending on `IndentCaseLabels` ? Repository: rC Clang https://reviews.llvm.org/D43183 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits