On Mon, 25 Nov 2024 19:14:49 GMT, Roger Riggs <rri...@openjdk.org> wrote:
> SM removal for internal implementation classes: > java.lang.CharacterName > java.lang.ref.Finalizer > jdk.internal.icu.impl.ICUBinary > jdk.internal.misc.ExtendedMapMode src/java.base/share/classes/java/lang/CharacterName.java line 50: > 48: private CharacterName() { > 49: try (DataInputStream dis = new DataInputStream( > 50: new > InflaterInputStream(getClass().getResourceAsStream("uniName.dat")))) { The old `getClass()` gets the `PrivilegedAction` anonymous class. Should we just move this to `CharacterName.class`? src/java.base/share/classes/java/lang/ref/Finalizer.java line 117: > 115: * invokers of that method from a stalled or deadlocked finalizer > thread. > 116: */ > 117: @SuppressWarnings("removal") ThreadGroup doesn't seem deprecated for removal, so we can remove this suppression. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/22370#discussion_r1857229429 PR Review Comment: https://git.openjdk.org/jdk/pull/22370#discussion_r1857230768