On Wed, 4 Dec 2024 13:13:11 GMT, Alan Bateman <al...@openjdk.org> wrote:
>> src/java.management/share/classes/javax/management/MBeanServerFactory.java >> line 368: >> >>> 366: >>> 367: // No context class loader? Try with Class.forName() >>> 368: return Class.forName(builderClassName); >> >> Maybe we should keep that call to ReflectUtil here for the moment? > > The java.management module is mapped to the boot loader so it's okay to > replace the use of ReflectUtil.forName with Class.forName here (no behavior > change). OK - I withdraw my objection then ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/22100#discussion_r1869478007
