# HG changeset patch # User Tom Rodriguez # Date 1461966449 25200 # Node ID acae988d96ad5328414e768f1a9175567ded4873 # Parent 802821d82a92a4756780e16c4e73ba9f101ce1d2 Remove ResolvedJavaType.getClassFilePath diff -r 802821d82a92 -r acae988d96ad jvmci/jdk.vm.ci.hotspot/src/jdk/vm/ci/hotspot/HotSpotResolvedObjectTypeImpl.java --- a/jvmci/jdk.vm.ci.hotspot/src/jdk/vm/ci/hotspot/HotSpotResolvedObjectTypeImpl.java Tue May 03 08:45:04 2016 -0700 +++ b/jvmci/jdk.vm.ci.hotspot/src/jdk/vm/ci/hotspot/HotSpotResolvedObjectTypeImpl.java Fri Apr 29 14:47:29 2016 -0700 @@ -33,7 +33,6 @@ import java.lang.reflect.Constructor; import java.lang.reflect.Method; import java.lang.reflect.Modifier; -import java.net.URL; import java.nio.ByteOrder; import java.util.ArrayList; import java.util.Arrays; @@ -49,7 +48,6 @@ import jdk.vm.ci.meta.JavaConstant; import jdk.vm.ci.meta.JavaKind; import jdk.vm.ci.meta.JavaType; -import jdk.vm.ci.meta.MetaUtil; import jdk.vm.ci.meta.ModifiersProvider; import jdk.vm.ci.meta.ResolvedJavaField; import jdk.vm.ci.meta.ResolvedJavaMethod; @@ -852,12 +850,6 @@ } @Override - public URL getClassFilePath() { - Class cls = mirror(); - return cls.getResource(MetaUtil.getSimpleName(cls, true).replace('.', '$') + ".class"); - } - - @Override public boolean isLocal() { return mirror().isLocalClass(); } diff -r 802821d82a92 -r acae988d96ad jvmci/jdk.vm.ci.hotspot/src/jdk/vm/ci/hotspot/HotSpotResolvedPrimitiveType.java --- a/jvmci/jdk.vm.ci.hotspot/src/jdk/vm/ci/hotspot/HotSpotResolvedPrimitiveType.java Tue May 03 08:45:04 2016 -0700 +++ b/jvmci/jdk.vm.ci.hotspot/src/jdk/vm/ci/hotspot/HotSpotResolvedPrimitiveType.java Fri Apr 29 14:47:29 2016 -0700 @@ -27,7 +27,6 @@ import java.lang.annotation.Annotation; import java.lang.reflect.Array; import java.lang.reflect.Modifier; -import java.net.URL; import jdk.vm.ci.common.JVMCIError; import jdk.vm.ci.meta.Assumptions.AssumptionResult; @@ -230,11 +229,6 @@ } @Override - public URL getClassFilePath() { - return null; - } - - @Override public boolean isLocal() { return false; } diff -r 802821d82a92 -r acae988d96ad jvmci/jdk.vm.ci.meta/src/jdk/vm/ci/meta/ResolvedJavaType.java --- a/jvmci/jdk.vm.ci.meta/src/jdk/vm/ci/meta/ResolvedJavaType.java Tue May 03 08:45:04 2016 -0700 +++ b/jvmci/jdk.vm.ci.meta/src/jdk/vm/ci/meta/ResolvedJavaType.java Fri Apr 29 14:47:29 2016 -0700 @@ -23,7 +23,6 @@ package jdk.vm.ci.meta; import java.lang.annotation.Annotation; -import java.net.URL; import jdk.vm.ci.meta.Assumptions.AssumptionResult; @@ -307,11 +306,6 @@ String getSourceFileName(); /** - * Returns the class file path - if available - of this type, or {@code null}. - */ - URL getClassFilePath(); - - /** * Returns {@code true} if the type is a local type. */ boolean isLocal(); diff -r 802821d82a92 -r acae988d96ad jvmci/jdk.vm.ci.runtime.test/src/jdk/vm/ci/runtime/test/TestResolvedJavaType.java --- a/jvmci/jdk.vm.ci.runtime.test/src/jdk/vm/ci/runtime/test/TestResolvedJavaType.java Tue May 03 08:45:04 2016 -0700 +++ b/jvmci/jdk.vm.ci.runtime.test/src/jdk/vm/ci/runtime/test/TestResolvedJavaType.java Fri Apr 29 14:47:29 2016 -0700 @@ -46,7 +46,6 @@ import java.lang.reflect.Field; import java.lang.reflect.Method; import java.lang.reflect.Modifier; -import java.net.URL; import java.util.Arrays; import java.util.Collections; import java.util.HashMap; @@ -809,23 +808,6 @@ } @Test - public void classFilePathTest() { - for (Class c : classes) { - ResolvedJavaType type = metaAccess.lookupJavaType(c); - URL path = type.getClassFilePath(); - if (type.isPrimitive() || type.isArray()) { - assertEquals(null, path); - } else { - assertNotNull(path); - String pathString = path.getPath(); - if (type.isLocal() || type.isMember()) { - assertTrue(pathString.indexOf('$') > 0); - } - } - } - } - - @Test public void isLeafTest() { for (Class c : classes) { ResolvedJavaType type = metaAccess.lookupJavaType(c); @@ -913,7 +895,6 @@ "hasFinalizableSubclass", "hasFinalizer", "getSourceFileName", - "getClassFilePath", "isLocal", "isJavaLangObject", "isMember",