-
Notifications
You must be signed in to change notification settings - Fork 6k
Remove usages of deprecated old native "<name>" syntax.
#37104
Conversation
…:external-anme', '<name>') instead See dart-lang/sdk#28791
dcharkes
left a comment
There was a problem hiding this 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 && |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
spurious formatting?
There was a problem hiding this comment.
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)
There was a problem hiding this comment.
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 formatfor reasons beyond my comprehension)
++
) The old native "<name>" syntax is being replaced with external functions that have a @pragma("external-name", "<name>"). See dart-lang/sdk#28791
The old
native "<name>"syntax is being replaced with external functions thathave a
@pragma("external-name", "<name>").See dart-lang/sdk#28791