Kamel-Media / Kamel

Kotlin asynchronous media loading and caching library for Compose.
Apache License 2.0
592 stars 23 forks source link

Replaced Stately ConcurrentMutableMap with Ktor ConcurrentMap #84

Closed kamiox closed 6 months ago

kamiox commented 6 months ago

The Stately's implementation does not work properly on JVM targets causing ConcurrentModificationException

https://github.com/touchlab/Stately/issues/105 https://github.com/Kamel-Media/Kamel/issues/75

I've only tested it on the JVM targets and I was not able to reproduce the problem.

luca992 commented 6 months ago

Yeah we could go with that for now (even though ktor asks that people do not use it haha). Thanks for the pr!