Add AndroidX collections dependency

chrisbanes · Aug 23, 2019 · a524b53 · a524b53
1 parent 650051c
commit a524b53
Show file tree

Hide file tree

Showing 5 changed files with 15 additions and 9 deletions.
diff --git a/base-android/build.gradle b/base-android/build.gradle
@@ -46,6 +46,7 @@ dependencies {
 
     implementation Libs.AndroidX.Fragment.fragment
     implementation Libs.AndroidX.coreKtx
+    implementation Libs.AndroidX.collection
 
     implementation Libs.timber
     implementation Libs.threeTenAbp

diff --git a/base-android/src/main/java/app/tivi/SharedElementHelper.kt b/base-android/src/main/java/app/tivi/SharedElementHelper.kt
@@ -17,15 +17,17 @@
 package app.tivi
 
 import android.view.View
+import androidx.collection.ArrayMap
 
-data class SharedElementHelper(
+class SharedElementHelper {
-    private val sharedElementViews: MutableMap<View, String?> = mutableMapOf()
+    private val _sharedElements = ArrayMap<View, String?>()
-) {
-    val sharedElements: Map<View, String?> = sharedElementViews
 
-    fun addSharedElement(view: View, name: String? = null) {
+    val sharedElements: Map<View, String?>
-        sharedElementViews[view] = name ?: view.transitionName
+        get() = _sharedElements
+
+    fun addSharedElement(view: View, transitionName: String? = view.transitionName) {
+        _sharedElements[view] = transitionName ?: view.transitionName
     }
 
-    fun isEmpty(): Boolean = sharedElementViews.isEmpty()
+    fun isEmpty(): Boolean = _sharedElements.isEmpty
 }
diff --git a/buildSrc/src/main/java/app/tivi/buildsrc/dependencies.kt b/buildSrc/src/main/java/app/tivi/buildsrc/dependencies.kt
@@ -73,6 +73,7 @@ object Libs {
     object AndroidX {
         const val appcompat = "androidx.appcompat:appcompat:1.1.0-rc01"
         const val browser = "androidx.browser:browser:1.0.0"
+        const val collection = "androidx.collection:collection-ktx:1.1.0"
         const val palette = "androidx.palette:palette:1.0.0"
         const val recyclerview = "androidx.recyclerview:recyclerview:1.1.0-beta03"
         const val emoji = "androidx.emoji:emoji:1.0.0"

diff --git a/data/build.gradle b/data/build.gradle
@@ -32,6 +32,7 @@ dependencies {
 
     api Libs.AndroidX.Room.common
     api Libs.AndroidX.Room.ktx
+    implementation Libs.AndroidX.collection
 
     api Libs.AndroidX.Paging.common
 

diff --git a/data/src/main/java/app/tivi/data/repositories/search/SearchStore.kt b/data/src/main/java/app/tivi/data/repositories/search/SearchStore.kt
@@ -16,14 +16,15 @@
 
 package app.tivi.data.repositories.search
 
+import androidx.collection.LruCache
 import javax.inject.Inject
 
 class SearchStore @Inject constructor() {
-    private val cache = HashMap<String, LongArray>()
+    private val cache = LruCache<String, LongArray>(32)
 
     fun getResults(query: String) = cache[query]
 
     fun setResults(query: String, results: LongArray) {
-        cache[query] = results
+        cache.put(query, results)
     }
 }