Java printer: arrow-switch case labels print static constants unqualified
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 68/100
Línea de trabajo
Start with SwitchEntryImpl.print and trace how it chooses qualification for arrow-switch case conditions; compare this with the surrounding qualification used for old-style switch labels. Check the printer tests for switch statements and expressions, then add coverage for a static constant from another class and an enum constant. Done means both arrow-switch forms preserve compilable qualification while enum labels remain valid.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
SwitchEntryImpl.print prints every case condition with QualificationImpl.SIMPLE_ONLY. That is right for an enum constant (Java requires the simple name there before Java 21), but wrong for a static constant of another type:
class Consts { static final int ONE = 1; }
class User {
int kind(int v) {
switch (v) {
case Consts.ONE -> { return 10; }
default -> { return 20; }
}
}
}
prints as case ONE -> ..., which does not compile in User (no ONE in scope). The same holds for a switch expression, which prints its entries the same way. Old-style switches print their labels with the surrounding qualification and are not affected.
Found when printing a method body into another class: the original text was case CodeConstants.GROUP_JUMP ->, the printed copy case GROUP_JUMP ->.
Suggested fix: use SIMPLE_ONLY only when the condition is a field reference to an enum constant of the selector's enum type, and print the other conditions with the given qualification.
- Lenguaje dominante
- Java
- Estrellas
- 1
- Forks
- 1
- Merge medio
- 1 h 58 min
- PR fusionados (30 d)
- 5
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de CodeLaser/maddi
-
build/ci good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
Los mantenedores suelen responder en 1 día
-
C# printer: a self-bounded interface's wildcard interface and companion class share a name (CS0102)Abiertofront-end:csharp
Dificultad 3/5 1-2 días Aptitud para principiantes 64/100
CodeLaser/maddi#160 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
bug front-end:javac
Dificultad 4/5 3-5 días Aptitud para principiantes 42/100
CodeLaser/maddi#145 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
C# printer: an inner class of a generic class needs the outer type's arguments (Outer<A>.Inner)Abiertofront-end:csharp
Dificultad 4/5 3-5 días Aptitud para principiantes 30/100
Los mantenedores suelen responder en 1 día
Todos los issues de CodeLaser/maddi
Issues similares
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 68/100
quarkusio/quarkus-agent-mcp#296 ·
Los mantenedores suelen responder en 1 día
-
enhancement web-ui
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100
jenkinsci/jenkins#27583 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
apache/rocketmq-dashboard#7302 ·
Los mantenedores suelen responder en 3 días
-
area: android good first issue help wanted testing
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Gautam-aman/ResQNet#56 ·
Los mantenedores suelen responder en 1 día