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

Remove SkScalarIsFinite from Flutter Engine. #52270

Merged
merged 3 commits into from
Apr 20, 2024

Conversation

johnstiles-google
Copy link
Contributor

SkScalarIsFinite is deprecated. Use std::isfinite instead.

@jonahwilliams jonahwilliams requested a review from flar April 19, 2024 21:22
Copy link
Contributor

@flar flar left a comment

Choose a reason for hiding this comment

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

LGTM

@jonahwilliams jonahwilliams added the autosubmit Merge PR when tree becomes green via auto submit App label Apr 20, 2024
Copy link
Member

@jonahwilliams jonahwilliams left a comment

Choose a reason for hiding this comment

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

LGTM

@auto-submit auto-submit bot merged commit ca4d5c8 into flutter:main Apr 20, 2024
29 checks passed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 20, 2024
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Apr 20, 2024
…147105)

flutter/engine@4d34b6d...499edda

2024-04-20 [email protected] [et] Fix concurrent modification exception (flutter/engine#52247)
2024-04-20 [email protected] Remove SkScalarIsFinite from Flutter Engine. (flutter/engine#52270)
2024-04-20 [email protected] Roll Skia from 514d7fc627b3 to 7be6fe3150fc (1 revision) (flutter/engine#52280)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC [email protected],[email protected],[email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
gilnobrega pushed a commit to gilnobrega/flutter that referenced this pull request Apr 22, 2024
…lutter#147105)

flutter/engine@4d34b6d...499edda

2024-04-20 [email protected] [et] Fix concurrent modification exception (flutter/engine#52247)
2024-04-20 [email protected] Remove SkScalarIsFinite from Flutter Engine. (flutter/engine#52270)
2024-04-20 [email protected] Roll Skia from 514d7fc627b3 to 7be6fe3150fc (1 revision) (flutter/engine#52280)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC [email protected],[email protected],[email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
autosubmit Merge PR when tree becomes green via auto submit App
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants