Re: RFR: 8260520: Avoid getting permissions in JarFileFactory when no SecurityManager installed [v2]

2021-01-28 Thread Claes Redestad
On Wed, 27 Jan 2021 19:03:38 GMT, Sean Mullan wrote: >> Claes Redestad has updated the pull request incrementally with two >> additional commits since the last revision: >> >> - Copyrights >> - Same for Windows > > Marked as reviewed by mullan (Reviewer). @seanjmullan @dfuch @Michael-Mc-Maho

Re: RFR: 8260520: Avoid getting permissions in JarFileFactory when no SecurityManager installed [v2]

2021-01-27 Thread Sean Mullan
On Wed, 27 Jan 2021 18:59:00 GMT, Claes Redestad wrote: >> 8260520: Avoid getting permissions in JarFileFactory when no SecurityManager >> installed > > Claes Redestad has updated the pull request incrementally with two additional > commits since the last revision: > > - Copyrights > - Same

Re: RFR: 8260520: Avoid getting permissions in JarFileFactory when no SecurityManager installed [v2]

2021-01-27 Thread Daniel Fuchs
On Wed, 27 Jan 2021 18:59:00 GMT, Claes Redestad wrote: >> 8260520: Avoid getting permissions in JarFileFactory when no SecurityManager >> installed > > Claes Redestad has updated the pull request incrementally with two additional > commits since the last revision: > > - Copyrights > - Same

Re: RFR: 8260520: Avoid getting permissions in JarFileFactory when no SecurityManager installed [v2]

2021-01-27 Thread Claes Redestad
> 8260520: Avoid getting permissions in JarFileFactory when no SecurityManager > installed Claes Redestad has updated the pull request incrementally with two additional commits since the last revision: - Copyrights - Same for Windows - Changes: - all: https://git.openjdk.java.

Re: RFR: 8260520: Avoid getting permissions in JarFileFactory when no SecurityManager installed [v2]

2021-01-27 Thread Claes Redestad
On Wed, 27 Jan 2021 16:57:25 GMT, Sean Mullan wrote: > Will you make the same change to > `src/java.base/windows/classes/sun/net/www/protocol/jar/JarFileFactory.java`? Good catch. Let's keep things consistent. :-) - PR: https://git.openjdk.java.net/jdk/pull/2263