Re: RFR: 8344394: Remove SecurityManager and related calls from java.management.rmi [v3]

2024-11-28 Thread Kevin Walls
On Fri, 22 Nov 2024 16:04:34 GMT, Daniel Fuchs wrote: >> Kevin Walls has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains eight additional >> commits

Re: RFR: 8344394: Remove SecurityManager and related calls from java.management.rmi [v3]

2024-11-26 Thread Kevin Walls
On Mon, 25 Nov 2024 14:47:00 GMT, Kevin Walls 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

Re: RFR: 8344394: Remove SecurityManager and related calls from java.management.rmi [v3]

2024-11-25 Thread Alex Menkov
On Mon, 25 Nov 2024 14:47:00 GMT, Kevin Walls 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

Re: RFR: 8344394: Remove SecurityManager and related calls from java.management.rmi [v2]

2024-11-25 Thread Kevin Walls
On Fri, 22 Nov 2024 19:19:37 GMT, Kevin Walls 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

Re: RFR: 8344394: Remove SecurityManager and related calls from java.management.rmi [v3]

2024-11-25 Thread Kevin Walls
> 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 incremental webrev excludes the unrela

Re: RFR: 8344394: Remove SecurityManager and related calls from java.management.rmi [v2]

2024-11-25 Thread Kevin Walls
On Fri, 22 Nov 2024 22:08:28 GMT, Alex Menkov wrote: >> Kevin Walls has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Remove last import sun.reflect.misc.ReflectUtil > > src/java.management.rmi/share/classes/javax/management/remote/rmi/RMI

Re: RFR: 8344394: Remove SecurityManager and related calls from java.management.rmi [v2]

2024-11-25 Thread Kevin Walls
On Fri, 22 Nov 2024 21:50:12 GMT, Alex Menkov wrote: >> Kevin Walls has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Remove last import sun.reflect.misc.ReflectUtil > > src/java.management.rmi/share/classes/javax/management/remote/rmi/RMI

Re: RFR: 8344394: Remove SecurityManager and related calls from java.management.rmi [v2]

2024-11-22 Thread Alex Menkov
On Fri, 22 Nov 2024 19:19:37 GMT, Kevin Walls 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

Re: RFR: 8344394: Remove SecurityManager and related calls from java.management.rmi

2024-11-22 Thread Kevin Walls
On Wed, 20 Nov 2024 10:14:40 GMT, Kevin Walls wrote: > Remove redundant SecurityManager, AccessController references > (following on from JDK-8338411: Implement JEP 486: Permanently Disable the > Security Manager). As mentioned in https://github.com/openjdk/jdk/pull/22100 java.management.rmi i

Re: RFR: 8344394: Remove SecurityManager and related calls from java.management.rmi [v2]

2024-11-22 Thread Kevin Walls
> 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.r

Re: RFR: 8344394: Remove SecurityManager and related calls from java.management.rmi

2024-11-22 Thread Kevin Walls
On Wed, 20 Nov 2024 10:14:40 GMT, Kevin Walls wrote: > Remove redundant SecurityManager, AccessController references > (following on from JDK-8338411: Implement JEP 486: Permanently Disable the > Security Manager). Thanks @dfuch - both comments relate to my note: There are still some Privileged

Re: RFR: 8344394: Remove SecurityManager and related calls from java.management.rmi

2024-11-22 Thread Daniel Fuchs
On Wed, 20 Nov 2024 10:14:40 GMT, Kevin Walls 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/RMIConnectionI

Re: RFR: 8344394: Remove SecurityManager and related calls from java.management.rmi

2024-11-21 Thread Kevin Walls
On Wed, 20 Nov 2024 10:14:40 GMT, Kevin Walls wrote: > Remove redundant SecurityManager, AccessController references > (following on from JDK-8338411: Implement JEP 486: Permanently Disable the > Security Manager). Notes: 1 RMIConnectionImpl.java In the method unwrap(..) at the old line 1565