improves the backward compatibility of LoaderClassPath.

LoaderClassPath is modified to show the same behavior in both Java 8 and 9.

build.xml

@@ -83,7 +83,7 @@
       <classpath refid="classpath"/>
     </javac>
   </target>
-	
+
   <target name="test-compile" depends="compile">
     <javac srcdir="${test.src.dir}"
            destdir="${test.build.dir}"

src/main/javassist/ClassPoolTail.java

@@ -245,8 +245,7 @@ public ClassPath appendSystemPath() {
         }
         else {
             ClassLoader cl = Thread.currentThread().getContextClassLoader();
-            appendClassPath(new LoaderClassPath(cl));
-            return appendClassPath(new ModuleClassPath());
+            return appendClassPath(new LoaderClassPath(cl, true));
         }
     }
 

src/main/javassist/CtClass.java

@@ -69,7 +69,7 @@ public abstract class CtClass {
     /**
      * The version number of this release.
      */
-    public static final String version = "3.21.0-java9";
+    public static final String version = "3.21.0-java9beta2";
 
     /**
      * Prints the version number and the copyright notice.

src/main/javassist/LoaderClassPath.java

@@ -47,11 +47,40 @@
 public class LoaderClassPath implements ClassPath {
     private WeakReference clref;
 
+    /**
+     * If true, this search path implicitly includes
+     * a {@code ModuleClassPath} as a fallback.
+     * For backward compatibility, this field is set to true
+     * if the JVM is Java 9 or later.  It can be false in
+     * Java 9 but the behavior of {@code LoadClassPath} will
+     * be different from its behavior in Java 8 or older.
+     *
+     * <p>This field must be false if the JVM is Java 8 or older.
+     *
+     * @since 3.21
+     */
+    public static boolean fallbackOnModuleClassPath
+        = javassist.bytecode.ClassFile.MAJOR_VERSION >= javassist.bytecode.ClassFile.JAVA_9;
+
+    private static ModuleClassPath moduleClassPath = null;
+
+    private boolean doFallback;
+
     /**
      * Creates a search path representing a class loader.
      */
     public LoaderClassPath(ClassLoader cl) {
+        this(cl, fallbackOnModuleClassPath);
+    }
+
+    LoaderClassPath(ClassLoader cl, boolean fallback) {
         clref = new WeakReference(cl);
+        doFallback = fallback;
+        if (fallback)
+            synchronized (LoaderClassPath.class) {
+                if (moduleClassPath == null)
+                    moduleClassPath = new ModuleClassPath();
+            }
     }
 
     public String toString() {
@@ -67,13 +96,18 @@ public String toString() {
      * This method calls <code>getResourceAsStream(String)</code>
      * on the class loader.
      */
-    public InputStream openClassfile(String classname) {
+    public InputStream openClassfile(String classname) throws NotFoundException {
         String cname = classname.replace('.', '/') + ".class";
         ClassLoader cl = (ClassLoader)clref.get();
         if (cl == null)
             return null;        // not found
-        else
-            return cl.getResourceAsStream(cname);
+        else {
+            InputStream is = cl.getResourceAsStream(cname);
+            if (is == null && doFallback)
+                return moduleClassPath.openClassfile(classname);
+            else
+                return is;
+        }
     }
 
     /**
@@ -88,8 +122,13 @@ public URL find(String classname) {
         ClassLoader cl = (ClassLoader)clref.get();
         if (cl == null)
             return null;        // not found
-        else
-            return cl.getResource(cname);
+        else {
+            URL url = cl.getResource(cname);
+            if (url == null && doFallback)
+                return moduleClassPath.find(classname);
+            else
+                return url;
+        }
     }
 
     /**

src/main/javassist/ModuleClassPath.java

@@ -32,6 +32,7 @@
  * @see ClassPool#appendClassPath(ClassPath)
  * @see LoaderClassPath
  * @see ClassClassPath
+ * @see 3.21
  */
 public class ModuleClassPath implements ClassPath {
     private HashMap packages = new HashMap();

src/test/javassist/JvstTest5.java

@@ -235,4 +235,11 @@ public void test83StackmapWithArrayType() throws Exception {
         Object obj = make(ctClass.getName());
         assertEquals(1, invoke(obj, "run"));
     }
+
+    public void testLoaderClassPath() throws Exception {
+        ClassPool cp = new ClassPool();
+        cp.appendClassPath(new LoaderClassPath(new Loader()));
+        assertNotNull(cp.get(Object.class.getName()));
+        assertNotNull(cp.get(this.getClass().getName()));
+    }
 }