From 2a7055a8fff6ec5f16e960b803a414e5d83e9e0f Mon Sep 17 00:00:00 2001 From: Connor Pearson Date: Thu, 10 Nov 2022 21:35:00 +0000 Subject: [PATCH] style: Remove unused std::ops::Rem implementations Differential Revision: https://phabricator.services.mozilla.com/D161844 --- components/style/values/computed/length.rs | 11 +---------- components/style/values/specified/length.rs | 20 +------------------- 2 files changed, 2 insertions(+), 29 deletions(-) diff --git a/components/style/values/computed/length.rs b/components/style/values/computed/length.rs index 4222f2924ed..29bed80d926 100644 --- a/components/style/values/computed/length.rs +++ b/components/style/values/computed/length.rs @@ -17,7 +17,7 @@ use crate::values::{specified, CSSFloat}; use crate::Zero; use app_units::Au; use std::fmt::{self, Write}; -use std::ops::{Add, AddAssign, Div, Mul, MulAssign, Neg, Rem, Sub, SubAssign}; +use std::ops::{Add, AddAssign, Div, Mul, MulAssign, Neg, Sub, SubAssign}; use style_traits::{CSSPixel, CssWriter, ToCss}; pub use super::image::Image; @@ -402,15 +402,6 @@ impl Neg for CSSPixelLength { } } -impl Rem for CSSPixelLength { - type Output = Self; - - #[inline] - fn rem(self, other: Self) -> Self { - CSSPixelLength::new(self.0 % other.0) - } -} - impl Sub for CSSPixelLength { type Output = Self; diff --git a/components/style/values/specified/length.rs b/components/style/values/specified/length.rs index cc436a91c41..979a910509f 100644 --- a/components/style/values/specified/length.rs +++ b/components/style/values/specified/length.rs @@ -22,7 +22,7 @@ use crate::{Zero, ZeroNoPercent}; use app_units::Au; use cssparser::{Parser, Token}; use std::cmp; -use std::ops::{Add, Mul, Rem, Sub}; +use std::ops::{Add, Mul, Sub}; use style_traits::values::specified::AllowedNumericType; use style_traits::{ParseError, SpecifiedValueInfo, StyleParseErrorKind}; @@ -760,24 +760,6 @@ impl Sub for AbsoluteLength { } } -impl Rem for AbsoluteLength { - type Output = Self; - - #[inline] - fn rem(self, rhs: Self) -> Self { - match (self, rhs) { - (AbsoluteLength::Px(x), AbsoluteLength::Px(y)) => AbsoluteLength::Px(x % y), - (AbsoluteLength::In(x), AbsoluteLength::In(y)) => AbsoluteLength::In(x % y), - (AbsoluteLength::Cm(x), AbsoluteLength::Cm(y)) => AbsoluteLength::Cm(x % y), - (AbsoluteLength::Mm(x), AbsoluteLength::Mm(y)) => AbsoluteLength::Mm(x % y), - (AbsoluteLength::Q(x), AbsoluteLength::Q(y)) => AbsoluteLength::Q(x % y), - (AbsoluteLength::Pt(x), AbsoluteLength::Pt(y)) => AbsoluteLength::Pt(x % y), - (AbsoluteLength::Pc(x), AbsoluteLength::Pc(y)) => AbsoluteLength::Pc(x % y), - _ => AbsoluteLength::Px(self.to_px() % rhs.to_px()), - } - } -} - /// A `` without taking `calc` expressions into account /// ///