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

Reply via email to