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

Conversation

dnfield
Copy link
Contributor

@dnfield dnfield commented Sep 24, 2020

Follow-up on #21057

The framework wants to compare images to avoid potentially expensive logic when setting an image. What it really wants to know is if the underlying image reference is the same. This makes that possible.

@auto-assign auto-assign bot requested a review from gw280 September 24, 2020 00:23
@dnfield dnfield requested review from chinmaygarde and removed request for gw280 September 24, 2020 00:25
}

/// Returns true if `other` shares the same underlying image memory as this,
/// even if this or `other` is [dispose]d.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

might be worth referring to clone in these docs

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

Copy link
Contributor

@Hixie Hixie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dnfield dnfield 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 Sep 24, 2020
@zanderso
Copy link
Member

The tree looks green. Landing this.

@zanderso zanderso merged commit 2ea7e52 into flutter:master Sep 24, 2020
@dnfield dnfield deleted the image_wrap branch September 24, 2020 04:41
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Sep 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes 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.

4 participants