Use ConcurrentHashMap instead of ArrayMap to avoid thread safe problem.

alibaba · Dec 26, 2018 · fb1c7d8 · fb1c7d8
1 parent 2893007
commit fb1c7d8
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/gradle.properties b/gradle.properties
@@ -46,7 +46,7 @@ org.gradle.jvmargs=-Xmx1536M
 GROUP=com.alibaba.android
 ARTIFACT=tangram
 VERSION=1
-VERSION_NAME=3.1.9
+VERSION_NAME=3.1.9.1
 PACKAGING_TYPE=aar
 systemProp.compileSdkVersion=26
 systemProp.targetSdkVersion=26

diff --git a/tangram/build.gradle b/tangram/build.gradle
@@ -143,7 +143,7 @@ dependencies {
             exclude group: 'com.android.support', module: 'support-annotations'
         }
     } else {
-        compile('com.alibaba.android:virtualview:1.3.8@aar') {
+        compile('com.alibaba.android:virtualview:1.3.8.3@aar') {
             transitive true
             exclude group: 'com.android.support', module: 'appcompat'
             exclude group: 'com.android.support', module: 'appcompat-v7'

diff --git a/tangram/src/main/java/com/tmall/wireless/tangram/BaseTangramEngine.java b/tangram/src/main/java/com/tmall/wireless/tangram/BaseTangramEngine.java
@@ -80,6 +80,7 @@
 import java.util.LinkedList;
 import java.util.List;
 import java.util.Map;
+import java.util.concurrent.ConcurrentHashMap;
 
 /**
  * {@link O} is the type of data, {@link T} is the array type of data, {@link C} is the class of Group, {@link L} is the class of Component
@@ -91,7 +92,7 @@
 public class BaseTangramEngine<O, T, C, L> implements ServiceManager {
 
 
-    private Map<Class<?>, Object> mServices = new ArrayMap<>();
+    private ConcurrentHashMap<Class<?>, Object> mServices = new ConcurrentHashMap<>();
 
     @NonNull
     private final Context mContext;