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.
