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

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