On Fri, 22 Nov 2024 19:19:37 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 incrementally with one additional > commit since the last revision: > > Remove last import sun.reflect.misc.ReflectUtil Thanks @alexmenkov - I'm making the callAs calls handle and unwrap the CompletionException, and pass it on to existing exception handling. Yes, the existing fetchNotifications() doesn't use the doPrivOperation style. Seems relevant that it is part of RMIConnection interface, but not part of MBeanServerConnection. RMIConnector defines an RMINotifClient to loop, calling fetchNotifications, i.e. it's not what user code normally calls. ------------- PR Comment: https://git.openjdk.org/jdk/pull/22270#issuecomment-2498210052
