Review Request JDK-8169909 java agent fails to add to class path when the initial module is a named module

Alan Bateman Alan.Bateman at oracle.com
Thu Nov 17 21:22:07 UTC 2016



On 17/11/2016 20:03, Mandy Chung wrote:
> http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8169909/webrev.00/
>
> This fixes a regression that causes java agent fails to
> append to class path when the initial module on the module
> path but no class path.  The AppClassLoader should have a
> URLClassPath instance set for the support of Java agent,
> Instrumentation::appendToSystemClassLoaderSearch.
>
> This patch also updates JavaClassPathTest to cover more test
> cases for JAR file and CLASS-PATH attribute.
>
This looks okay, I just wondering if the source for jdk.test in 
java/lang/instrument/src should be moved down one directly level so that 
it's clear which test this code is for.

-Alan


More information about the core-libs-dev mailing list