RFR: JDK-8210009: Source Launcher classloader should support getResource and getResourceAsStream

mandy chung mandy.chung at oracle.com
Thu Sep 6 20:10:00 UTC 2018



On 9/5/18 12:08 PM, Jonathan Gibbons wrote:
> Updated webrev, addressing comments from Peter.
>
> 1. Fixed protocol string, and updated test to verify that URLs are OK
> 2. Changed init of the stream handler
>
> Webrev: http://cr.openjdk.java.net/~jjg/8210009/webrev.02/
>

The updated webrev looks okay.

The test now verifies URL.toURI() - that's good (I mixed that in my 
review). The formatting in src/CLTest.java seems to need some clean up 
(the identation) - no need for a new webrev.

Mandy
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/compiler-dev/attachments/20180906/2a1e330b/attachment.html>


More information about the compiler-dev mailing list