On Wed, 4 Dec 2024 13:08:05 GMT, Daniel Fuchs <dfu...@openjdk.org> wrote:
>> Kevin Walls has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains 18 commits: >> >> - Merge >> - Merge remote-tracking branch 'upstream/master' into >> java_management_post_sm >> - Exception handling >> - tidyup >> - Suppression tidyup >> - unnecessary variables >> - remove redundant method >> - Merge remote-tracking branch 'upstream/master' into >> java_management_post_sm >> - ReflectUtil removals >> - Remove checkPackageAccess calls >> - ... and 8 more: https://git.openjdk.org/jdk/compare/994504c3...1437532f > > 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). ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/22100#discussion_r1869470860
