diff --git a/src/common.rs b/src/common.rs index 1987356..ee94712 100644 --- a/src/common.rs +++ b/src/common.rs @@ -36,12 +36,7 @@ pub(crate) fn ceil_f64(v: f64) -> f64 { pub(crate) fn ceil_f64(v: f64) -> f64 { debug_assert!(v >= 0.0); - // `Decimal::from_f64` returns `None` for non-finite values, so fall back to - // the input and mirror `f64::ceil` instead of panicking. - match Decimal::from_f64(v) { - Some(d) => d.ceil().to_f64().unwrap_or(v), - None => v, - } + Decimal::from_f64(v).unwrap().ceil().to_f64().unwrap() } #[cfg(any(feature = "byte", feature = "bit"))] @@ -59,12 +54,7 @@ pub(crate) fn ceil_f32(v: f32) -> f32 { pub(crate) fn ceil_f32(v: f32) -> f32 { debug_assert!(v >= 0.0); - // `Decimal::from_f32` returns `None` for non-finite values, so fall back to - // the input and mirror `f32::ceil` instead of panicking. - match Decimal::from_f32(v) { - Some(d) => d.ceil().to_f32().unwrap_or(v), - None => v, - } + Decimal::from_f32(v).unwrap().ceil().to_f32().unwrap() } #[cfg(any(feature = "byte", feature = "bit"))] diff --git a/tests/byte.rs b/tests/byte.rs index d0502c9..8687ce1 100644 --- a/tests/byte.rs +++ b/tests/byte.rs @@ -170,12 +170,3 @@ fn tests() { assert_eq!(byte, serde_json::from_str::(case.0).unwrap(), "{i}"); } } - -#[test] -fn from_non_finite_returns_none() { - // Non-finite inputs must return None instead of panicking, in both builds. - assert_eq!(Byte::from_f64(f64::INFINITY), None); - assert_eq!(Byte::from_f64(f64::NAN), None); - assert_eq!(Byte::from_f32(f32::INFINITY), None); - assert_eq!(Byte::from_f32(f32::NAN), None); -}