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

Reply via email to