Mirrors/servo
1
0
Fork 0
mirror of https://github.com/servo/servo.git synced 2025-08-05 21:50:18 +01:00
Code Issues Projects Releases Packages Wiki Activity

Fix dimensionality of Au

Browse source 
Previously, we implemented:

Au * Au -> Au
Au / Au -> Au
Au % Au -> Au

... which are inconsistent. It should be:

Au * Au -> SquaredAu
Au / Au -> i32
Au % Au -> i32

or:

Au * i32 -> Au
Au / i32 -> Au
Au % i32 -> Au

I picked the latter.

Also, the multiplicative identity does not make sense
when multiplication take two different types.
This commit is contained in:
Simon Sapin 2014-09-19 13:54:57 +01:00
parent 2652d223f5
commit 415bbaeb2e
2 changed files with 11 additions and 21 deletions
Download patch file Download diff file Expand all files Collapse all files

30
components/util/geometry.rs
View file

@ -9,7 +9,7 @@ use geom::size::Size2D;
use serialize::{Encodable, Encoder};
use std::default::Default;
use std::num::{NumCast, One, Zero};
use std::num::{NumCast, Zero};
use std::fmt;
// Units for use with geom::length and geom::scale_factor.
@ -102,30 +102,27 @@ impl Sub<Au,Au> for Au {
}
impl Mul<Au,Au> for Au {
impl Mul<i32, Au> for Au {
#[inline]
fn mul(&self, other: &Au) -> Au {
fn mul(&self, other: &i32) -> Au {
let Au(s) = *self;
let Au(o) = *other;
Au(s * o)
Au(s * *other)
}
}
impl Div<Au,Au> for Au {
impl Div<i32, Au> for Au {
#[inline]
fn div(&self, other: &Au) -> Au {
fn div(&self, other: &i32) -> Au {
let Au(s) = *self;
let Au(o) = *other;
Au(s / o)
Au(s / *other)
}
}
impl Rem<Au,Au> for Au {
impl Rem<i32, Au> for Au {
#[inline]
fn rem(&self, other: &Au) -> Au {
fn rem(&self, other: &i32) -> Au {
let Au(s) = *self;
let Au(o) = *other;
Au(s % o)
Au(s % *other)
}
}
@ -137,13 +134,6 @@ impl Neg<Au> for Au {
}
}
impl One for Au {
#[inline]
fn one() -> Au { Au(1) }
}
impl Num for Au {}
#[inline]
pub fn min(x: Au, y: Au) -> Au { if x < y { x } else { y } }
#[inline]