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

Correct greater than or equal logic in offset base #8210

Merged
merged 3 commits into from
Mar 19, 2019

Conversation

jonahwilliams
Copy link
Contributor

Not sure what bugs this has been hiding.

Copy link
Contributor

@dnfield dnfield left a comment

Choose a reason for hiding this comment

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

LGTM - I hesitate to ask but is this a breaking change?

@jonahwilliams
Copy link
Contributor Author

Lets not ask the questions we don't want answers to

@Hixie
Copy link
Contributor

Hixie commented Mar 19, 2019

this definitely needs tests. ideally comprehensive tests of every operator...

also, oops. good catch.

Copy link
Member

@goderbauer goderbauer left a comment

Choose a reason for hiding this comment

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

LGTM

Nice catch!

@jonahwilliams jonahwilliams merged commit 146646d into flutter:master Mar 19, 2019
@jonahwilliams jonahwilliams deleted the ordered branch March 19, 2019 20:41
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 19, 2019
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 19, 2019
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 19, 2019
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 19, 2019
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 20, 2019
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 20, 2019
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 20, 2019
RBogie pushed a commit to RBogie/flutter-engine that referenced this pull request Apr 8, 2019
RBogie added a commit to RBogie/flutter-engine that referenced this pull request Apr 8, 2019
RBogie added a commit to RBogie/flutter-engine that referenced this pull request Apr 8, 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.

5 participants