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

Revert "Extract WideToUTF16String/UTF16StringToWide to FML" #39019

Merged
merged 1 commit into from
Jan 20, 2023

Conversation

godofredoc
Copy link
Contributor

Reverts #39006

@godofredoc godofredoc added revert Label used to revert changes in a closed and merged pull request. and removed revert Label used to revert changes in a closed and merged pull request. labels Jan 20, 2023
@auto-submit auto-submit bot removed the revert Label used to revert changes in a closed and merged pull request. label Jan 20, 2023
@auto-submit
Copy link
Contributor

auto-submit bot commented Jan 20, 2023

This pull request cannot be merged due to conflicts. Please resolve conflicts and re-add the revert label.

@godofredoc godofredoc added the revert Label used to revert changes in a closed and merged pull request. label Jan 20, 2023
@auto-submit auto-submit bot merged commit acca56c into main Jan 20, 2023
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Jan 20, 2023
…118869)

* 7bbe79e10 Extract WideToUTF16String/UTF16StringToWide to FML (flutter/engine#39006)

* acca56ce0 Revert "Extract WideToUTF16String/UTF16StringToWide to FML (#39006)" (flutter/engine#39019)

* 730e88fb6 [Impeller] Check the correct stencil coverage when deciding whether to elide a restore (flutter/engine#39023)
@cbracken cbracken deleted the revert-39006-WideToUTF16 branch January 25, 2023 00:11
@cbracken
Copy link
Member

LGTM thanks for reverting. This was caused by a race between two PRs: the one being reverted which removed the old function and another one that landed which added a new use of it. I've relanded this patch with the fix.

ricardoamador pushed a commit to ricardoamador/engine that referenced this pull request Jan 25, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
revert Label used to revert changes in a closed and merged pull request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants