Extract option check

oracle · Apr 8, 2024 · b48b52b · b48b52b
1 parent 71687b9
commit b48b52b
Showing 1 changed file with 23 additions and 22 deletions.
diff --git a/substratevm/src/com.oracle.svm.hosted/src/com/oracle/svm/hosted/EmbeddedResourcesInfo.java b/substratevm/src/com.oracle.svm.hosted/src/com/oracle/svm/hosted/EmbeddedResourcesInfo.java
@@ -48,28 +48,29 @@ public static EmbeddedResourcesInfo singleton() {
     }
 
     public void declareResourceAsRegistered(Module module, String resource, String source) {
-        if (ResourcesFeature.Options.GenerateEmbeddedResourcesFile.getValue()) {
-            Resources.ModuleResourceKey key = Resources.createStorageKey(module, resource);
-            registeredResources.compute(key, (k, v) -> {
-                if (v == null) {
-                    ArrayList<String> newValue = new ArrayList<>();
-                    newValue.add(source);
-                    return newValue;
-                }
-
-                /*
-                 * We have to avoid duplicated sources here. In case when declaring resource that
-                 * comes from module as registered, we don't have information whether the resource
-                 * is already registered or not. That check is performed later in {@link
-                 * Resources.java#addEntry}, so we have to perform same check here, to avoid
-                 * duplicates when collecting information about resource.
-                 */
-                if (!v.contains(source)) {
-                    v.add(source);
-                }
-                return v;
-            });
+        if (!ResourcesFeature.Options.GenerateEmbeddedResourcesFile.getValue()) {
+            return;
         }
-    }
 
+        Resources.ModuleResourceKey key = Resources.createStorageKey(module, resource);
+        registeredResources.compute(key, (k, v) -> {
+            if (v == null) {
+                ArrayList<String> newValue = new ArrayList<>();
+                newValue.add(source);
+                return newValue;
+            }
+
+            /*
+             * We have to avoid duplicated sources here. In case when declaring resource that comes
+             * from module as registered, we don't have information whether the resource is already
+             * registered or not. That check is performed later in {@link Resources.java#addEntry},
+             * so we have to perform same check here, to avoid duplicates when collecting
+             * information about resource.
+             */
+            if (!v.contains(source)) {
+                v.add(source);
+            }
+            return v;
+        });
+    }
 }