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