Re: RFR 8174151: URLClassLoader no longer uses custom URLStreamHandler for jar URLs

2017-02-14 Thread Chris Hegarty
> On 14 Feb 2017, at 18:37, Paul Sandoz wrote: > >> >> On 14 Feb 2017, at 05:58, Chris Hegarty wrote: >> >> >>> On 13 Feb 2017, at 22:52, Paul Sandoz wrote: >>> >>> Hi, >>> >>> Please review. >>> >>>

Re: RFR 8174151: URLClassLoader no longer uses custom URLStreamHandler for jar URLs

2017-02-14 Thread Paul Sandoz
> On 14 Feb 2017, at 05:58, Chris Hegarty wrote: > > >> On 13 Feb 2017, at 22:52, Paul Sandoz wrote: >> >> Hi, >> >> Please review. >> >> http://cr.openjdk.java.net/~psandoz/jdk9/JDK-8174151-url-class-path-loader/webrev/index.html > > If I

Re: RFR 8174151: URLClassLoader no longer uses custom URLStreamHandler for jar URLs

2017-02-14 Thread Chris Hegarty
> On 13 Feb 2017, at 22:52, Paul Sandoz wrote: > > Hi, > > Please review. > > > http://cr.openjdk.java.net/~psandoz/jdk9/JDK-8174151-url-class-path-loader/webrev/index.html If I understand this correctly then the only optimisation ( by-pass of openConnection ),

Re: RFR 8174151: URLClassLoader no longer uses custom URLStreamHandler for jar URLs

2017-02-14 Thread Alan Bateman
On 13/02/2017 22:52, Paul Sandoz wrote: Hi, Please review. http://cr.openjdk.java.net/~psandoz/jdk9/JDK-8174151-url-class-path-loader/webrev/index.html This is a nasty edge case when a URLStreamHandler is registered (via the system property) to process jar: URLs. The support for

RFR 8174151: URLClassLoader no longer uses custom URLStreamHandler for jar URLs

2017-02-13 Thread Paul Sandoz
Hi, Please review. http://cr.openjdk.java.net/~psandoz/jdk9/JDK-8174151-url-class-path-loader/webrev/index.html This is a nasty edge case when a URLStreamHandler is registered (via the system property) to process jar: URLs. The support for multi-release JAR files changed the implementation