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

Revert "Remove unused Settings::ToString. (#8642)"#8689

Merged
chinmaygarde merged 2 commits into
flutter-team-archive:masterfrom
chinmaygarde:revert_settings_to_string
Apr 22, 2019
Merged

Revert "Remove unused Settings::ToString. (#8642)"#8689
chinmaygarde merged 2 commits into
flutter-team-archive:masterfrom
chinmaygarde:revert_settings_to_string

Conversation

@chinmaygarde

Copy link
Copy Markdown
Contributor

This reverts commit 4ead9c8.

Reland tracked in flutter/flutter#31433.

@dnfield dnfield left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@chinmaygarde

Copy link
Copy Markdown
Contributor Author

Failing presubmit addressed by pending manual roll.

@chinmaygarde
chinmaygarde merged commit 6dc5dca into flutter-team-archive:master Apr 22, 2019
@chinmaygarde
chinmaygarde deleted the revert_settings_to_string branch April 22, 2019 21:31
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Apr 23, 2019
flutter-team-archive/engine@934772d...0523870

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

eggfly commented Mar 15, 2022

Copy link
Copy Markdown
Contributor

I also found this ToString function. Why can't we delete it?

@zanderso

Copy link
Copy Markdown
Contributor

Since the Fuchsia runner is in-tree, I'm guessing the ToString can now be deleted more easily.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Development

Successfully merging this pull request may close these issues.

5 participants