Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Be less strict when detecting 2D translations #2457

Merged
merged 1 commit into from Feb 22, 2018

Conversation

@mrobinson
Copy link
Member

mrobinson commented Feb 22, 2018

We can avoid comparing directly against 1 here and use an epsilon to
account for floating point error.


This change is Reviewable

We can avoid comparing directly against 1 here and use an epsilon to
account for floating point error.
@mrobinson mrobinson requested a review from glennw Feb 22, 2018
@mrobinson
Copy link
Member Author

mrobinson commented Feb 22, 2018

@glennw
Copy link
Member

glennw commented Feb 22, 2018

@bors-servo
Copy link
Contributor

bors-servo commented Feb 22, 2018

📌 Commit 4ee249f has been approved by glennw

@bors-servo
Copy link
Contributor

bors-servo commented Feb 22, 2018

Testing commit 4ee249f with merge 0b711f2...

bors-servo added a commit that referenced this pull request Feb 22, 2018
Be less strict when detecting 2D translations

We can avoid comparing directly against 1 here and use an epsilon to
account for floating point error.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/webrender/2457)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Feb 22, 2018

☀️ Test successful - status-appveyor, status-taskcluster, status-travis
Approved by: glennw
Pushing 0b711f2 to master...

@bors-servo bors-servo merged commit 4ee249f into servo:master Feb 22, 2018
3 checks passed
3 checks passed
Taskcluster (pull_request) TaskGroup: success
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
homu Test successful
Details
@mrobinson mrobinson deleted the mrobinson:less-strict-2d-detection branch Feb 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.