Skip to content

Commit 17aa678

Browse files
committed
fix: switch loadedJavaScriptExtends to synchronized map to prevent null object issues
1 parent e0e92bf commit 17aa678

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

test-app/runtime/src/main/java/com/tns/Runtime.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -222,7 +222,7 @@ public Runtime(StaticConfiguration config, DynamicConfiguration dynamicConfigura
222222
this.weakInstances = new ConcurrentHashMap<>();
223223
// TODO: can't use a ConcurrentHashMap for loadedJavaScriptExtends because it loads null objects, which aren't supported
224224
// either leave it like this or create a separate set for null caches
225-
this.loadedJavaScriptExtends = new ConcurrentHashMap<>();
225+
this.loadedJavaScriptExtends = Collections.synchronizedMap(new HashMap<>());
226226
this.strongJavaObjectToID = Collections.synchronizedMap(new NativeScriptHashMap<>());
227227
this.weakJavaObjectToID = Collections.synchronizedMap(new NativeScriptWeakHashMap<>());
228228
}

0 commit comments

Comments
 (0)