On Wed, 4 Dec 2024 10:16:53 GMT, Kevin Walls <kev...@openjdk.org> wrote:
>> Remove redundant SecurityManager, AccessController references >> (following on from JDK-8338411: Implement JEP 486: Permanently Disable the >> Security Manager). > > 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? ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/22100#discussion_r1869463223
