On Wed, 20 Nov 2024 10:14:40 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.rmi/share/classes/javax/management/remote/rmi/RMIConnectionImpl.java
line 32:
> 30: import java.rmi.UnmarshalException;
> 31: import java.rmi.server.Unreferenced;
> 32: import java.security.PrivilegedActionException;
should this be removed too?
src/java.management.rmi/share/classes/javax/management/remote/rmi/RMIConnectionImpl.java
line 1286:
> 1284:
> 1285: /** @throws UnsupportedOperationException {@inheritDoc} */
> 1286: private Object doPrivilegedOperation(final int operation,
Maybe rename "doPrivilegedOperation" into an overload of "doOperation"
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22270#discussion_r1854188609
PR Review Comment: https://git.openjdk.org/jdk/pull/22270#discussion_r1854196303
