RFR: 8344575: Examine usage of ReflectUtil.forName() in java.sql.rowset - XmlReaderContentHandler [v2]

[Lance Andersen]

On Fri, 6 Dec 2024 17:38:22 GMT, Brent Christian <bchristi at openjdk.org> wrote:

>> `XmlReaderContentHandler.endElement()` routes a `Class.forName()` call through `ReflectUtil.forName()`. When `sun.reflect.misc.ReflectUtil.forName()` calls the 1-arg `Class.forName()`, it is doing so from `java.base`, and so using the boot loader.
>> 
>> Changing `XmlReaderContentHandler` to use the 3-arg `Class.forName()` method with a null classloader should yield equivalent behavior.
>
> Brent Christian has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Remove ReflectUtil.forName; update module info

Looks OK Brent, you will need to add a label to the JBS issue to mark the issue as noreg-cleanup noreg-trivial  etc..., otherwise, at some point someone will be looking for a test with this change

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

Marked as reviewed by lancea (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/22585#pullrequestreview-2485562778