RFR: 8151542: URL resources for multi-release jar files have a #runtime fragment appended to them
Alan Bateman
Alan.Bateman at oracle.com
Sun May 1 07:45:06 UTC 2016
On 30/04/2016 01:10, Steve Drach wrote:
> Hopefully the last one ;-) This webrev removes the lowercase of
> protocol, and incorporates better (in my mind) seperation of choices
> for choosing the loader, similar to what Paul suggested. Everything
> else remains the same. Only URLClassPath changed from previous webrev.
>
> http://cr.openjdk.java.net/~sdrach/8151542/webrev.04/index.html
> <http://cr.openjdk.java.net/%7Esdrach/8151542/webrev.04/index.html>
>
This version looks okay but a comment to explain why the protocol is
checked without regard to case would be useful. Also would it be
possible to fix the overly long lines (L377-378) as that is very
inconsistent with the existing code and annoying when looking at
side-by-side diffs.
-Alan.
More information about the core-libs-dev
mailing list