Bundle/PersistableBundle/ContentValues creation with Pairs about android-ktx HOT 2 CLOSED

bundleOf() does not make Bundle().apply{} irrelevant. The extra allocations created by the Pair should not have a meaningful impact on apps unless they store thousands of items in bundles (which would indicate other issues). Short-lived allocations like this are also fairly well optimized out in N and O.

from android-ktx.

(Whoops I had a reply to this in a tab and forgot to hit enter.)

The full transformation with all syntactic niceties becomes:

bundleOf(
    KEY_ANSWER_ONE to answerOne?.text?.toString(),
    KEY_ANSWER_TWO to answerTwo?.text?.toString())

This syntax matches that of the various *Of factories in the stdlib which take pairs (e.g., mapOf, mutableMapOf, and our provided arrayMapOf).

One advantage to the allocation here is that the underlying collection can be pre-sized to exactly fit. This avoids it always starting empty, going through an allocation to 4 slots, and then having to double (based on how many elements you add).

Allocations like this wind up in many APIs. Hopefully someday the compiler will be able to see through the varargs and pair allocations someday. Or we could try getting the optimizations into D8, R8, or even ART.

The runtime check is the most unfortunate part, but as of yet there's no way to optimize it. We can bundle lint checks for it, though.

from android-ktx.