Is there a better (more correct?) way to write thi...

# coroutines

[Dominaezzz]

Is there a better (more correct?) way to write this operator?

@ExperimentalCoroutinesApi
internal fun <T> Flow<T?>.coalesce(other: Flow<T>): Flow<T> {
    return distinctUntilChanged { old, new -> old == null && new == null }
        .transformLatest {
            if (it != null) {
                // TODO: Will "Latest" wrongly cancel this branch?
                emit(it)
            } else {
                emitAll(other)
            }
        }
}

[ephemient]

I would have gone for

.flatMapLatest {
    flowOf(it ?: return@flatMapLatest other)
}

but it's functionally the same