Skip to content

Commit

Permalink
Auto merge of #8248 - Ms2ger:finite-zero, r=jdm
Browse files Browse the repository at this point in the history
Remove the Zeroable definition for Finite<T>.

I have no idea why it was added, and it appears to be unused.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8248)
<!-- Reviewable:end -->
  • Loading branch information
bors-servo committed Oct 30, 2015
2 parents 00b60b9 + 04e32c7 commit 851e709
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions components/script/dom/bindings/num.rs
Original file line number Diff line number Diff line change
Expand Up @@ -4,16 +4,13 @@

//! The `Finite<T>` struct.

use core::nonzero::Zeroable;
use num::Float;
use std::ops::Deref;

/// Encapsulates the IDL restricted float type.
#[derive(JSTraceable, Clone, Copy, Eq, PartialEq)]
pub struct Finite<T: Float>(T);

unsafe impl<T: Float> Zeroable for Finite<T> {}

impl<T: Float> Finite<T> {
/// Create a new `Finite<T: Float>` safely.
pub fn new(value: T) -> Option<Finite<T>> {
Expand Down

0 comments on commit 851e709

Please sign in to comment.