Does anyone know if there is a place to submit feature requests for ktx. I have been thinking about

Parcel.readSparseArray

. It returns

SparseArray

without any generic parameter and I am not aware of a way to read it with the generic parameter. It should be fairly easy to reify that one to do a cast for you. I know that this is not something that is super hard to do yourself but it feels like a hack every time i need to suppress the unchecked cast warning.

Report it here https://issuetracker.google.com/issues/new?component=396204

Yeah sure I can. I was thinking that the goal of ktx was to reduce boilerplate and this is something that I find myself doing in nearly every project

Yeah, sure, you may report

Yeah maybe slap on an

@Trows

annotation. Or return a

SparseArray?

No, you didn’t get what I mean

True. But do you mean that the code can succeed the cast but crash when accessing the items?

see:

val input = SparseArray<Any>()
input.put(0, "String!") // Put string
a.writeSparseArray(input)
val output = a.readSparseArray(javaClass.classLoader) as SparseArray<Int> // No exception here!
val item: Int = output.get(0) // Exception will happen here!

Oh this was news to me but I guess it make sense since type erasure.

Yes

Yeah then maybe I will just have to continue adding an extension and name it something like

readUncheckedSparseArray

You can try to write something like that:

inline fun <reified T> Parcel.readAndCastSparseArray(): SparseArray<T>? {
    val sparseArray = readSparseArray(javaClass.classLoader) ?: return null
    for (index in 0 until sparseArray.size()) {
        val value = sparseArray.valueAt(index)
        if (value != null && value !is T) {
            throw ClassCastException("Value $value on index $index cast failed to ${T::class}")
        }
    }
    @Suppress("UNCHECKED_CAST")
    return sparseArray as SparseArray<T>
}

Looks cool! Will for sure give it a try!