Indentation wrong in default case in a switch statement.

[hildrum]

If I have:

      case 3:
        return whatever;
      default:
        throw new IllegalArgumentException("oops");

and I run the google-java-format, it adds two spaces to the default and throw,

      case 3:
        return whatever;
        default:
          throw new IllegalArgumentException("oops");

This switch statement is inside a function defined in an enum.

[cushon]

Can you share a full example? Any chance it's something like this?

class T {
  void f(int x) {
    switch (x) {
    case 3:
      return whatever;
    default:
      throw new IllegalArgumentException("oops");
    }
  }
}

google-java-format -lines 6 T.java
class T {
  void f(int x) {
    switch (x) {
    case 3:
      return whatever;
      default:
        throw new IllegalArgumentException("oops");
    }
  }
}

The formatter assumes the surrounding code is correctly indented when doing partial formatting, it doesn't indent relative to the existing indentation.

[hildrum]

Yup, that was exactly the problem! Thanks.