use ConcurrentHashMap

hoc081098 · Aug 5, 2022 · b3f1894 · b3f1894
1 parent e6d04c6
commit b3f1894
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 9 deletions.
diff --git a/.editorconfig b/.editorconfig
@@ -9,4 +9,6 @@ insert_final_newline=true
 ij_kotlin_imports_layout=*
 disabled_rules=filename
 [*.xml]
-indent_size=4
+indent_size=4
+[*.gradle]
+indent_size=4
diff --git a/library/src/main/java/com/hoc081098/viewbindingdelegate/internal/cache.kt b/library/src/main/java/com/hoc081098/viewbindingdelegate/internal/cache.kt
@@ -35,16 +35,11 @@ internal sealed interface MethodCache {
 }
 
 private abstract class AbstractMethodCache : MethodCache {
-  private val cache: MutableMap<Class<out ViewBinding>, Method> = ConcurrentHashMap()
+  private val cache = ConcurrentHashMap<Class<out ViewBinding>, Method>()
 
-  override fun <T : ViewBinding> getOrPut(clazz: Class<T>) =
+  override fun <T : ViewBinding> getOrPut(clazz: Class<T>): Method =
     measureNanoTime("[${this::class.java.simpleName}-getOrPut] class=$clazz") {
-      cache[clazz]?.let { return@measureNanoTime it }
-
-      clazz.findMethod().also {
-        // Cache update.
-        cache[clazz] = it
-      }
+      cache.getOrPut(clazz) { clazz.findMethod() }
     }
 
   abstract fun <T : ViewBinding> Class<T>.findMethod(): Method