On Thu, 14 Nov 2024 10:39:33 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).
src/java.management/share/classes/javax/management/remote/JMXConnectorFactory.java
line 448:
> 446: { return super.loadClass(name,
> resolve); }
> 447: }
> 448: : null;
I suspect we could just return `parent` here but that might require some deeper
investigation.
src/java.management/share/classes/sun/management/ManagementFactoryHelper.java
line 454:
> 452: * otherwise, just return.
> 453: */
> 454: @SuppressWarnings("removal")
Do you still need the `@SuppressWarnings` annotation here?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22100#discussion_r1846739026
PR Review Comment: https://git.openjdk.org/jdk/pull/22100#discussion_r1846741416
