Re: RFR: 8344565: SM cleanup in jdk/internal and java/lang package private classes

[Chen Liang]

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