RFR: 8148626: URI.toURL needs to use protocol Handler to parse file URIs

joe darcy joe.darcy at oracle.com
Sat Jan 30 18:35:24 UTC 2016


Hello,

The change looks okay in that the new code is limited to the jrt 
protocol. I assume the failing test

     java/net/URL/B5086147.java

passes again with this change.

If a fix for this issue is not pushed soon, I want to have the failing 
test problem listed 
(http://mail.openjdk.java.net/pipermail/core-libs-dev/2016-January/038459.html).

Thanks,

-Joe

On 1/30/2016 6:30 AM, Claes Redestad wrote:
> Hi,
>
> it turns out trying to optimize URI to URL conversion for file URLs 
> don't work out very well due to special treatment of backward slashes.
>
> Instead of trying to deal with such file URLs, this patch partially 
> backs out 8147462 and only leave the optimization in place for the jrt 
> protocol. This still helps
> reducing footprint in jigsaw while keeping things performance neutral 
> for other kinds of URLs.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8148626
> Webrev: http://cr.openjdk.java.net/~redestad/8148626/webrev.01/
>
> /Claes




More information about the core-libs-dev mailing list