RFR: 8344299: SM cleanup in javax.naming modules

Thu Nov 28 13:58:58 UTC 2024

On Thu, 28 Nov 2024 07:52:14 GMT, Alan Bateman <alanb at openjdk.org> wrote:

> Good cleanup. Maybe for a follow on PR but I assume VersionHelper can be removed. There's been several post JEP 486 removals that mean that createThread and loadClass shouldn't be needed now. Also isSerialDataAllowed could move to Obj, which I think is the only place where we need to know if javaXXX attributes can be deserialised.

I think we can do it as part of this PR: I have removed the LDAP `VersionHelper` (`com.sun.jndi.ldap.VersionHelper`) - it involved to put a replacement for `VersionHelper.loadClass` into a couple of LDAP-related classes. Pushed as 3c8687c17eb46ffae73c108f14377439e36ec518

`tier1` to `tier3` tests are still good with the latest change

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

PR Comment: https://git.openjdk.org/jdk/pull/22421#issuecomment-2506186106