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

Conversation

@mkustermann
Copy link
Member

The old native "<name>" syntax is being replaced with external functions that
have a @pragma("external-name", "<name>").

See dart-lang/sdk#28791

Copy link
Contributor

@dcharkes dcharkes left a comment

Choose a reason for hiding this comment

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

lgtm

bool operator ==(Object other) {
return other is Handle
&& other.handle == handle;
return other is Handle &&
Copy link
Contributor

Choose a reason for hiding this comment

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

spurious formatting?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah, I changed the code, it autoformatted, then I had to undo the autoformatting by-hand, ...

(Flutter doesn't use dart format for reasons beyond my comprehension)

Copy link
Contributor

Choose a reason for hiding this comment

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

(Flutter doesn't use dart format for reasons beyond my comprehension)

++

@mkustermann mkustermann merged commit ff7bdf8 into flutter:main Oct 28, 2022
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 28, 2022
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 28, 2022
schwa423 pushed a commit to schwa423/engine that referenced this pull request Nov 16, 2022
)

The old native "<name>" syntax is being replaced with external functions that
have a @pragma("external-name", "<name>").

See dart-lang/sdk#28791
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants