RFR: 8344575: Examine usage of ReflectUtil.forName() in java.sql.rowset - XmlReaderContentHandler [v2]
Chen Liang
liach at openjdk.org
Fri Dec 6 18:07:40 UTC 2024
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
The change to ReflectUtil looks good.
-------------
Marked as reviewed by liach (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/22585#pullrequestreview-2485563087
More information about the core-libs-dev
mailing list