Incorrect comment indentation inside switch statements on case that is not first
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Newbie friendliness
- 35/100
Research direction
Reproduce the report using the shown Test.java input and google-java-format 1.23.0, then compare the formatted output with the expected comment indentation. Trace the switch-statement formatting path and add regression coverage for a comment before a non-first case; done means the comment remains aligned as shown in the original input.
Written by the indexing model from the issue text.
Description
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;
}
}
}
- Dominant language
- Java
- Stars
- 6.2k
- Forks
- 936
- Avg merge
- 6m
- Merged PRs (30d)
- 3
Getting set up
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from google/google-java-format
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
google/google-java-format#1094 · 1 comment ·
Maintainers usually reply within 1 day
-
Difficulty 5/5 Over a week Newbie friendliness 15/100
google/google-java-format#1450 · 1 reaction ·
Maintainers usually reply within 1 day
-
Difficulty 4/5 3-5 days Newbie friendliness 55/100
google/google-java-format#1439 · 1 comment ·
Maintainers usually reply within 1 day
-
Difficulty 3/5 1-2 days Newbie friendliness 62/100
google/google-java-format#1436 · 2 comments ·
Maintainers usually reply within 1 day
-
Difficulty 3/5 1-2 days Newbie friendliness 48/100
google/google-java-format#1428 · 4 comments ·
Maintainers usually reply within 1 day
All issues in google/google-java-format
Similar issues
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
Maintainers usually reply within 1 day
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
Maintainers usually reply within 1 day
-
ci-failure-cause test-failure
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
Maintainers usually reply within 1 day
-
enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
nextcloud/notes-android#3367 ·
Maintainers usually reply within 1 day
-
:wave: team-triage a:chore in:isolated-projects in:kotlin-dsl
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
Maintainers usually reply within 1 day