Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Conversation

@blasten
Copy link

@blasten blasten commented Apr 27, 2022

Fixes b/230038195

This was a mistake. SparseArray#get takes a key, not an index.
Alternatively, valueAt(index) can be used to get the value at the given index.

@blasten blasten added the waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land. label Apr 27, 2022
@fluttergithubbot fluttergithubbot merged commit 133108c into flutter:main Apr 27, 2022
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 27, 2022
muditatandon pushed a commit to muditatandon/engine that referenced this pull request May 10, 2022
CaseyHillers pushed a commit that referenced this pull request May 11, 2022
* [flutter_releases] Apply flutter-2.13 candidate.7 CPs to latest roll (#33169)

* [flutter_releases] Fix condition in OpacityLayer where it inherits opacity twice (#33085) (#33105)

Co-authored-by: Jim Graham <[email protected]>

* [flutter_releases] Remove reference to FX_LOG_ALL (#33027) (#33147)

It was partially removed in
https://fuchsia-review.googlesource.com/c/fuchsia/+/674049; this commit
allows remaining references to be removed.

Co-authored-by: Tamir Duberstein <[email protected]>

Co-authored-by: Jim Graham <[email protected]>
Co-authored-by: Tamir Duberstein <[email protected]>

* Fix SparseArray lookup in PlatformViewsController (#32947)

Co-authored-by: Casey Hillers <[email protected]>
Co-authored-by: Jim Graham <[email protected]>
Co-authored-by: Tamir Duberstein <[email protected]>
Co-authored-by: Emmanuel Garcia <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

platform-android waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants