-
Notifications
You must be signed in to change notification settings - Fork 59
Conversation
|
This should be in a good state now, r? @jdm |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2 requests, 1 question. Otherwise r=me.
src/azure_hl.rs
Outdated
@@ -1781,6 +1793,13 @@ impl AzPoint { | |||
y: point.y, | |||
} | |||
} | |||
|
|||
fn from_vector_2d(point: &Vector2D<AzFloat>) -> Self { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why aren't the from_
methods From
impls?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No idea, I really just followed whatever was already in place (but asked myself the same question), there's also something very similar in the servo repo in the code that interacts with azure (something that should just use From/Into).
src/azure_hl.rs
Outdated
// Azure does not have a separation between points and vectors, AzPoint is used | ||
// to represent both | ||
impl AsAzurePoint for Vector2D<AzFloat> { | ||
fn as_azure_point(&self) -> AzPoint { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#[inline]
src/azure_hl.rs
Outdated
@@ -1781,6 +1793,13 @@ impl AzPoint { | |||
y: point.y, | |||
} | |||
} | |||
|
|||
fn from_vector_2d(point: &Vector2D<AzFloat>) -> Self { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#[inline]
@bors-servo delegate+ |
✌️ @nical can now approve this pull request |
@bors-servo r=nox |
📌 Commit cdb257c has been approved by |
Bump euclid to 0.14 In order for this to build we need servo-skia to be published on crates.io. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/rust-azure/264) <!-- Reviewable:end -->
☀️ Test successful - status-travis |
In order for this to build we need servo-skia to be published on crates.io.
This change is