> On Oct 19, 2016, at 3:50 PM, David DeHaven <david.deha...@oracle.com> wrote:
> 
> 
> Please review the new patch version:
> http://cr.openjdk.java.net/~ddehaven/8165271/jdk.1/
> 

Looks okay but I think the new interface should be named 
“JavaNetURLClassLoaderAccess” (matching “URLClassLoader")

> This was updated for the recent changes due to JDK-8168073. Note that the 
> patch is against jdk9-client but I can apply and push to 9-dev. Test runs 
> with both client and jake were successful.

Pushing to jdk9/dev would avoid any merge if any change to related area is made 
before the next jdk9/client integration to jdk9/dev.

Mandy

Reply via email to