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