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

Don't require the Zero trait for most Rect methods #112

Merged
merged 1 commit into from Oct 28, 2015

Conversation

@mrobinson
Copy link
Member

mrobinson commented Oct 27, 2015

Only require it for Rect::union. This makes it easier to call these
methods in Servo.

Review on Reviewable

Only require it for Rect::union. This makes it easier to call these
methods in Servo.
@metajack
Copy link
Contributor

metajack commented Oct 28, 2015

@mbrubeck
Copy link
Contributor

mbrubeck commented Oct 28, 2015

@bors-servo
Copy link
Contributor

bors-servo commented Oct 28, 2015

📌 Commit e7a22b7 has been approved by mbrubeck

@bors-servo
Copy link
Contributor

bors-servo commented Oct 28, 2015

Testing commit e7a22b7 with merge d577618...

bors-servo added a commit that referenced this pull request Oct 28, 2015
Don't require the Zero trait for most Rect methods

Only require it for Rect::union. This makes it easier to call these
methods in Servo.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/euclid/112)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Oct 28, 2015

☀️ Test successful - travis

@bors-servo bors-servo merged commit e7a22b7 into servo:master Oct 28, 2015
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
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

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