fix concurrent modification

jensstein · Feb 5, 2023 · 893b033 · 893b033
1 parent ace6eae
commit 893b033
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/app/src/main/java/com/machiav3lli/backup/ui/compose/item/Components.kt b/app/src/main/java/com/machiav3lli/backup/ui/compose/item/Components.kt
@@ -245,11 +245,12 @@ object IconCache {
     }
 
     fun dropAllButUsed(pkgs: List<Package>) {
+        val used = pkgs.map { it.iconData }
         beginNanoTimer("limitIconCache")
         val keys = synchronized(painterCache) { painterCache.keys }
-        (keys - pkgs.map { it.iconData }).forEach {
+        (keys - used).forEach {
             if (it !is Int) {
-                IconCache.removeIcon(it)
+                removeIcon(it)
             }
         }
         endNanoTimer("limitIconCache")

diff --git a/app/src/main/java/com/machiav3lli/backup/viewmodels/MainViewModel.kt b/app/src/main/java/com/machiav3lli/backup/viewmodels/MainViewModel.kt
@@ -159,7 +159,7 @@ class MainViewModel(
             //val pkgs = p.toPackageList(appContext, emptyList(), b)
             val pkgs = p.toPackageList(appContext, emptyList(), getBackups())
 
-            IconCache.dropAllButUsed(pkgs)
+            IconCache.dropAllButUsed(pkgs.drop(0))
 
             traceFlows { "***** packages ->> ${pkgs.size}" }
             pkgs