On 29/04/2016 19:36, Steve Drach wrote:
I put a new webrev out with the change suggested by Paul, using
file.indexOf(“!/“) instead of file.endsWith(“!/“).
http://cr.openjdk.java.net/~sdrach/8151542/webrev.02/index.html
<http://cr.openjdk.java.net/%7Esdrach/8151542/webrev.02/index.html>
This still needs to be fixed to compare the URL protocol without regard
to case (yes, some of the old code isn't right but best not to make it
worss).
So you are planning to eventually change the URLClassLoader spec to
allow this or not?
I think we should push this changeset since URLClassLoader has always
accepted the jar:…!/ URL and all this changeset does is enable it to
support multi-release jars. And create an issue to track the change
to the URLClassLoader api.
There shouldn't be any urgency with this change going in but if you are
doing it this way then please create a high priority bug to get the spec
and implementation aligned.
-Alan
