Re: RFR: 8344562: Remove security manager dependency from module jdk.jdi

2024-11-20 Thread Brian Burkhalter
On Wed, 20 Nov 2024 13:33:27 GMT, Sean Mullan wrote: >> src/jdk.jdi/share/classes/com/sun/tools/jdi/VirtualMachineManagerImpl.java >> line 65: >> >>> 63: SecurityManager sm = System.getSecurityManager(); >>> 64: if (sm != null) { >>> 65: JDIPermission vmmPermission =

Re: RFR: 8344562: Remove security manager dependency from module jdk.jdi

2024-11-20 Thread Sean Mullan
On Wed, 20 Nov 2024 06:58:36 GMT, Andrey Turbanov wrote: >> Trivial removal of the use of the `SecurityManager` from a single class. > > src/jdk.jdi/share/classes/com/sun/tools/jdi/VirtualMachineManagerImpl.java > line 65: > >> 63: SecurityManager sm = System.getSecurityManager(); >> 64

Re: RFR: 8344562: Remove security manager dependency from module jdk.jdi

2024-11-19 Thread Andrey Turbanov
On Tue, 19 Nov 2024 18:29:33 GMT, Brian Burkhalter wrote: > Trivial removal of the use of the `SecurityManager` from a single class. src/jdk.jdi/share/classes/com/sun/tools/jdi/VirtualMachineManagerImpl.java line 65: > 63: SecurityManager sm = System.getSecurityManager(); > 64:

Re: RFR: 8344562: Remove security manager dependency from module jdk.jdi

2024-11-19 Thread Chris Plummer
On Tue, 19 Nov 2024 18:29:33 GMT, Brian Burkhalter wrote: > Trivial removal of the use of the `SecurityManager` from a single class. Marked as reviewed by cjplummer (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/22251#pullrequestreview-2447381122

Re: RFR: 8344562: Remove security manager dependency from module jdk.jdi

2024-11-19 Thread Leonid Mesnik
On Tue, 19 Nov 2024 18:29:33 GMT, Brian Burkhalter wrote: > Trivial removal of the use of the `SecurityManager` from a single class. Marked as reviewed by lmesnik (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/22251#pullrequestreview-2446814097

Re: RFR: 8344562: Remove security manager dependency from module jdk.jdi

2024-11-19 Thread Kevin Walls
On Tue, 19 Nov 2024 18:29:33 GMT, Brian Burkhalter wrote: > Trivial removal of the use of the `SecurityManager` from a single class. Marked as reviewed by kevinw (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/22251#pullrequestreview-2446342689

RFR: 8344562: Remove security manager dependency from module jdk.jdi

2024-11-19 Thread Brian Burkhalter
Trivial removal of the use of the `SecurityManager` from a single class. - Commit messages: - 8344562: Remove security manager dependency from module jdk.jdi Changes: https://git.openjdk.org/jdk/pull/22251/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=22251&range=00 Iss