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

Fix an infinite loop in ellipse arc length calculation #2713

Merged

Conversation

@mrobinson
Copy link
Member

mrobinson commented May 1, 2018

It's possible to encounter this situation in very, very lopsided
ellipses like the kind you might encounter on an evil, or at least very
misguided, web page.

Fixes #2469.


This change is Reviewable

It's possible to encounter this situation in very, very lopsided
ellipses like the kind you might encounter on an evil, or at least very
misguided, web page.

Fixes #2469.
@nical
Copy link
Collaborator

nical commented May 1, 2018

@bors-servo
Copy link
Contributor

bors-servo commented May 1, 2018

📌 Commit cd0f80c has been approved by nical

@bors-servo
Copy link
Contributor

bors-servo commented May 1, 2018

Testing commit cd0f80c with merge cc67736...

bors-servo added a commit that referenced this pull request May 1, 2018
…ical

Fix an infinite loop in ellipse arc length calculation

It's possible to encounter this situation in very, very lopsided
ellipses like the kind you might encounter on an evil, or at least very
misguided, web page.

Fixes #2469.

<!-- 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/2713)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented May 1, 2018

☀️ Test successful - status-appveyor, status-taskcluster
Approved by: nical
Pushing cc67736 to master...

@bors-servo bors-servo merged commit cd0f80c into servo:master May 1, 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
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.