Skip to content

Comments

Add Rect::area#309

Merged
bors-servo merged 1 commit intoservo:masterfrom
nical:rect-area
Nov 16, 2018
Merged

Add Rect::area#309
bors-servo merged 1 commit intoservo:masterfrom
nical:rect-area

Conversation

@nical
Copy link
Contributor

@nical nical commented Nov 16, 2018

Just a convenience function that calls into Size2D::area.


This change is Reviewable

@nical
Copy link
Contributor Author

nical commented Nov 16, 2018

r? anyone

}

impl<T: Copy + Clone + Mul<T, Output = T>, U> TypedRect<T, U> {
pub fn area(&self) -> T {
Copy link
Member

Choose a reason for hiding this comment

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

Maybe #[inline]? But not a big deal in any case.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good point. Done.

@emilio
Copy link
Member

emilio commented Nov 16, 2018

@bors-servo delegate+

@bors-servo
Copy link
Contributor

✌️ @nical can now approve this pull request

@nical
Copy link
Contributor Author

nical commented Nov 16, 2018

@bors-servo r=emilio

@bors-servo
Copy link
Contributor

📌 Commit 65e4369 has been approved by emilio

bors-servo pushed a commit that referenced this pull request Nov 16, 2018
Add Rect::area

Just a convenience function that calls into `Size2D::area`.

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

⌛ Testing commit 65e4369 with merge 0b810d5...

@bors-servo
Copy link
Contributor

☀️ Test successful - status-travis
Approved by: emilio
Pushing 0b810d5 to master...

@bors-servo bors-servo merged commit 65e4369 into servo:master Nov 16, 2018
@nical nical deleted the rect-area branch November 16, 2018 18:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants