Feature Request: MutableVector should implement RandomAccess [170461551]

Fixed

Bug

Status Update

No update yet.

Description

[Deleted User]

created issue #1

Oct 9, 2020 07:20PM

Jetpack Compose release version: 1.0.0-alpha04

We have some internal utils that check for RandomAccess which isn't hit so we allocate an iterator instead. I don't see a reason why MutableVector should not implement `RandomAccess.

Comments

cl...@google.com <cl...@google.com> Oct 12, 2020 04:00PM

Assigned to mo...@google.com.

mo...@google.com <mo...@google.com> #2Oct 12, 2020 06:22PM

Hi Ed, Thank you so much for these suggestions. I've been reviewing them and merging them in. Hopefully it should be live. I've included a thank you note too in the article.

[Deleted User] <[Deleted User]> #3Oct 12, 2020 06:26PM

Great! Thanks a lot, I'll look for the live updates soon!

mo...@google.com <mo...@google.com> #4Oct 12, 2020 07:03PM

Accepted by mo...@google.com.

out for review: https://android-review.googlesource.com/c/platform/frameworks/support/+/1457518

I tested, just to be sure and it doesn't affect performance.

ap...@google.com <ap...@google.com> #5Oct 16, 2020 07:13PM

Marked as fixed.

Project: platform/frameworks/support
Branch: androidx-master-dev

commit de8d90deb96cce240c9eea7eae2779e22e2748b0
Author: George Mount <mount@google.com>
Date: Mon Oct 12 11:54:37 2020

Have MutableVector implement RandomAccess

Relnote: "MutableVector now implements RandomAccess"
Fixes: 170461551
Test: MutableVectorBenchmark
Change-Id: I85d73f55b19cce31ba70770dbd0dc98cb3a6957a

M compose/runtime/runtime/api/current.txt
M compose/runtime/runtime/api/public_plus_experimental_current.txt
M compose/runtime/runtime/api/restricted_current.txt
M compose/runtime/runtime/src/commonMain/kotlin/androidx/compose/runtime/collection/MutableVector.kt

https://android-review.googlesource.com/1457518

Issue 170461551