RFR: 8344891: Remove uses of sun.misc.ReflectUtil in java.desktop

Phil Race prr at openjdk.org
Fri Nov 22 22:33:47 UTC 2024


On Fri, 22 Nov 2024 22:27:32 GMT, Phil Race <prr at openjdk.org> wrote:

> Remove uses of sun.misc.ReflectUtils which checked obsolete permissions

src/java.desktop/share/classes/com/sun/beans/finder/MethodFinder.java line 81:

> 79:         try {
> 80:             Method method = CACHE.get(signature);
> 81:             return (method == null) ? method : CACHE.create(signature);

This code looks wrong, meaning it was wrong already.
It is out of scope here to evaluate and fix that, so I submitted https://bugs.openjdk.org/browse/JDK-8344892

src/java.desktop/share/classes/java/beans/DefaultPersistenceDelegate.java line 30:

> 28: import java.lang.reflect.*;
> 29: import java.util.Objects;
> 30: import sun.reflect.misc.MethodUtil;

Now we only need this one class.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/22335#discussion_r1854801172
PR Review Comment: https://git.openjdk.org/jdk/pull/22335#discussion_r1854802121


More information about the client-libs-dev mailing list