isn't that function just `fun <K, V> Map<K, V?>.filterValuesNotNull() = filterValues { i...

# announcements

[arekolek]

isn't that function just fun <K, V> Map<K, V?>.filterValuesNotNull() = filterValues { it != null } as Map<K, V> ? I'm not sure I would even define it at all

[pdegand]

the cast is unchecked

[pdegand]

i defined mine like that :

fun <K, V> Map<out K, V?>.filterValuesNotNull(): Map<K, V> {
  val result = LinkedHasMap<K, V>()
  for (entry in this) {
    entry.value?.let { result[entry.key] = it }
  }
}

mostly copied from filterValues of stdlib

[arekolek]

even though it's unchecked, I don't see how it could fail

[arekolek]

here's an issue you can track though https://youtrack.jetbrains.com/issue/KT-4734

[karelpeeters]

This is one of the things-to-think about in the contracts KEEP.

[karelpeeters]

https://github.com/Kotlin/KEEP/blob/bea9c861b5ae7e803a81626fdb3a1925d1a4862c/proposals/kotlin-contracts.md#open-questions