Re: RFR: 8344231: SecurityManager cleanup in java.lang.module and jdk.internal.module [v2]

2024-11-14 Thread Alan Bateman
On Fri, 15 Nov 2024 07:01:39 GMT, Eirik Bjørsnøs wrote: >> Please review this PR which removes code depending on SecurityManager from >> the `java.lang.module` and `jdk.internal.module` packages: >> >> * `ModuleFinder::ofSystem` is updated to not check the >> `"accessSystemModules"` RuntimePer

Re: RFR: 8344231: SecurityManager cleanup in java.lang.module and jdk.internal.module [v2]

2024-11-14 Thread Eirik Bjørsnøs
On Fri, 15 Nov 2024 06:33:30 GMT, Alan Bateman wrote: >> Eirik Bjørsnøs has updated the pull request incrementally with two >> additional commits since the last revision: >> >> - Remove empty line in addProvides >> - Add full stop after class description for SystemImage > > src/java.base/shar

Re: RFR: 8344231: SecurityManager cleanup in java.lang.module and jdk.internal.module [v2]

2024-11-14 Thread Eirik Bjørsnøs
> Please review this PR which removes code depending on SecurityManager from > the `java.lang.module` and `jdk.internal.module` packages: > > * `ModuleFinder::ofSystem` is updated to not check the > `"accessSystemModules"` RuntimePermission > * The `ModuleReferences.ExplodedModuleReader` constru

Re: RFR: 8344231: SecurityManager cleanup in java.lang.module and jdk.internal.module

2024-11-14 Thread Alan Bateman
On Thu, 14 Nov 2024 17:32:40 GMT, Eirik Bjørsnøs wrote: > Please review this PR which removes code depending on SecurityManager from > the `java.lang.module` and `jdk.internal.module` packages: > > * `ModuleFinder::ofSystem` is updated to not check the > `"accessSystemModules"` RuntimePermissi

RFR: 8344231: SecurityManager cleanup in java.lang.module and jdk.internal.module

2024-11-14 Thread Eirik Bjørsnøs
Please review this PR which removes code depending on SecurityManager from the `java.lang.module` and `jdk.internal.module` packages: * `ModuleFinder::ofSystem` is updated to not check the `"accessSystemModules"` RuntimePermission * The `ModuleReferences.ExplodedModuleReader` constructor is upda