Skip to content

Commit f509af8

Browse files
Simplify NativeImageClassLoader.initRemotePackageMap
1 parent 399be80 commit f509af8

File tree

1 file changed

+4
-12
lines changed

1 file changed

+4
-12
lines changed

substratevm/src/com.oracle.svm.hosted/src/com/oracle/svm/hosted/NativeImageClassLoader.java

Lines changed: 4 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -159,8 +159,8 @@ CodeSource codeSource() {
159159
localNameToModule = Collections.unmodifiableMap(nameToModule);
160160
localPackageToModule = Collections.unmodifiableMap(packageToModule);
161161
/*
162-
* Other than in {@code jdk.internal.loader.Loader} we initialize remotePackageToLoader here
163-
* which allows us to use an unmodifiable map instead of a ConcurrentHashMap.
162+
* Unlike {@code jdk.internal.loader.Loader}, we initialize remotePackageToLoader here which
163+
* allows us to use an unmodifiable map instead of a ConcurrentHashMap.
164164
*/
165165
remotePackageToLoader = initRemotePackageMap(configuration, List.of(ModuleLayer.boot()));
166166

@@ -185,7 +185,7 @@ public static URL toURL(URI uri) {
185185
}
186186

187187
/**
188-
* See {@code jdk.internal.loader.Loader#initRemotePackageMap}.
188+
* See {@link jdk.internal.loader.Loader#initRemotePackageMap}.
189189
*/
190190
private Map<String, ClassLoader> initRemotePackageMap(Configuration cf, List<ModuleLayer> parentModuleLayers) {
191191
Map<String, ClassLoader> remotePackageMap = new HashMap<>();
@@ -220,16 +220,8 @@ private Map<String, ClassLoader> initRemotePackageMap(Configuration cf, List<Mod
220220
ClassLoader l = loader;
221221
descriptor.packages().forEach(pn -> remotePackage(remotePackageMap, pn, l));
222222
} else {
223-
String target = resolvedModule.name();
224223
for (ModuleDescriptor.Exports e : descriptor.exports()) {
225-
boolean delegate;
226-
if (e.isQualified()) {
227-
delegate = (other.configuration() == cf) && e.targets().contains(target);
228-
} else {
229-
delegate = true;
230-
}
231-
232-
if (delegate) {
224+
if (!e.isQualified()) {
233225
remotePackage(remotePackageMap, e.source(), loader);
234226
}
235227
}

0 commit comments

Comments
 (0)