fix: IllegalStateException thrown when removing a 'null' element from…

… the Path cache
Collektive · Nov 13, 2024 · 813c477 · 813c477
1 parent b970ca1
commit 813c477
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/dsl/src/commonMain/kotlin/it/unibo/collektive/path/impl/PathImpl.kt b/dsl/src/commonMain/kotlin/it/unibo/collektive/path/impl/PathImpl.kt
@@ -19,12 +19,13 @@ internal data class PathImpl(private val path: List<Any?>) : Path {
     companion object {
         private val cache = mutableMapOf<List<Any?>, Path>()
         private const val MAX_CACHE_SIZE = 10_000
-        private const val CACHE_CLEANUP_SIZE = 500
+        private const val CACHE_CLEANUP_SIZE = 1000
 
         fun of(path: List<Any?>): Path = cache.getOrPut(path) { PathImpl(path) }.also {
             if (cache.size >= MAX_CACHE_SIZE) {
                 val iterator = cache.iterator()
                 repeat(CACHE_CLEANUP_SIZE) {
+                    iterator.next()
                     iterator.remove()
                 }
             }