Skip to content

Commit 26a05a6

Browse files
Merge pull request #9296 from matthiasblaesing/gh9294
Java: Additional flag to name conversion for SYNCHRONIZED_OR_ACC_SUPER and VOLATILE_OR_ACC_BRIDGE
2 parents 72a99d2 + 9932eb4 commit 26a05a6

1 file changed

Lines changed: 3 additions & 12 deletions

File tree

  • java/java.source.base/src/org/netbeans/modules/java/source/pretty

java/java.source.base/src/org/netbeans/modules/java/source/pretty/VeryPretty.java

Lines changed: 3 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -2901,17 +2901,7 @@ else if (addSpace)
29012901
needSpace();
29022902
}
29032903
}
2904-
2905-
private static final String[] flagLowerCaseNames = new String[FlagsEnum.values().length];
2906-
2907-
static {
2908-
for (FlagsEnum flag : FlagsEnum.values()) {
2909-
flagLowerCaseNames[flag.ordinal()] = flag.name().toLowerCase(Locale.ENGLISH);
2910-
}
2911-
flagLowerCaseNames[FlagsEnum.NON_SEALED.ordinal()] = "non-sealed";
2912-
flagLowerCaseNames[FlagsEnum.TRANSIENT_OR_ACC_VARARGS.ordinal()] = "transient"; //TODO: there should be a test in java.source.base for this
2913-
}
2914-
2904+
29152905
/**
29162906
* Workaround for defect #239258. Prints flag names converted to lowercase in ENGLISH locale to
29172907
* avoid weird Turkish I > i-without-dot-above conversion.
@@ -2925,7 +2915,8 @@ public static String flagNames(long flags) {
29252915
String sep = ""; // NOI18N
29262916
for (FlagsEnum flag : Flags.asFlagSet(flags)) {
29272917
buf.append(sep);
2928-
String fname = flagLowerCaseNames[flag.ordinal()];
2918+
// Since JDK26 javac FlagsEnum#toString is usable for printing
2919+
String fname = flag.toString();
29292920
buf.append(fname);
29302921
sep = " "; // NOI18N
29312922
}

0 commit comments

Comments
 (0)