google-java-format icon indicating copy to clipboard operation
google-java-format copied to clipboard

Incorrect comment indentation inside switch statements on case that is not first

Open romani opened this issue 1 year ago • 1 comments

continuation of https://github.com/google/google-java-format/issues/1147

$ javac Test.java 
$ cat Test.java 
public class Test {
  public void check1(int x) {
    switch (x) {
      case 1:
        x = x + 1;
      /* fall-through*/ case 2:
        ;
      default:
        return;
    }
  }
}

$ java -jar google-java-format-1.23.0-all-deps.jar Test.java > TestUpdated.java
$ diff -u Test.java TestUpdated.java 
--- Test.java	2024-09-01 07:31:37.268757862 -0700
+++ TestUpdated.java	2024-09-01 07:31:54.329333795 -0700
@@ -3,7 +3,7 @@
     switch (x) {
       case 1:
         x = x + 1;
-      /* fall-through*/ case 2:
+        /* fall-through*/ case 2:
         ;
       default:
         return;

$ cat TestUpdated.java 
public class Test {
  public void check1(int x) {
    switch (x) {
      case 1:
        x = x + 1;
        /* fall-through*/ case 2:
        ;
      default:
        return;
    }
  }
}

romani avatar Sep 01 '24 14:09 romani

I agree that ident is a bug, although I'd also encourage code following Google Style to use the 'typical' fall through comment the style guide suggests which is the line comment // fall through.

cushon avatar Sep 10 '24 15:09 cushon