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

Revert Rect/RRect 64 bit #8690

Merged
merged 2 commits into from
Apr 22, 2019
Merged

Revert Rect/RRect 64 bit #8690

merged 2 commits into from
Apr 22, 2019

Conversation

dnfield
Copy link
Contributor

@dnfield dnfield commented Apr 22, 2019

This broke RRect.contains - needs further refinement.

Will land TBR to unblock the roll. /cc @cbracken @jason-simmons

@dnfield dnfield merged commit 4f2fd84 into flutter:master Apr 22, 2019
@dnfield dnfield deleted the const_rect_64 branch April 22, 2019 22:46
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 22, 2019
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 23, 2019
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Apr 23, 2019
flutter/engine@934772d...0523870

git log 934772d..0523870 --no-merges --oneline
0523870 Add tests from framework (flutter/engine#8692)
ed1f3fd Change Vertices.indices to use a Uint16 list to more accurately reflect Skia's API (flutter/engine#8657)
4f2fd84 Revert Rect/RRect 64 bit (flutter/engine#8690)
6dc5dca Revert "Remove unused Settings::ToString. (#8642)" (flutter/engine#8689)
9fa7336 fix toString (flutter/engine#8688)
c123152 Make Rect and RRect use 64 bit doubles, and make them const-able (flutter/engine#8565)

The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff (liyuqian@google.com), and stop
the roller if necessary.
dnfield added a commit to dnfield/engine that referenced this pull request Apr 23, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants