Remove the lax_deserialize feature

Author: jplatte

Cargo.toml

@@ -16,7 +16,6 @@ default-features = false
 
 [features]
 float_deserialize = ["serde"]
-lax_deserialize = ["float_deserialize"]
 default = ["std"]
 std = []
 

src/int.rs

@@ -590,11 +590,8 @@ impl<'de> Deserialize<'de> for Int {
 
         #[cfg(feature = "float_deserialize")]
         {
-            #[cfg(not(feature = "lax_deserialize"))]
             const EXPECTING: &str =
                 "a number between -2^53 + 1 and 2^53 - 1 without fractional component";
-            #[cfg(feature = "lax_deserialize")]
-            const EXPECTING: &str = "a number between -2^53 + 1 and 2^53 - 1";
 
             let val = f64::deserialize(deserializer)?;
 

src/lib.rs

@@ -57,12 +57,5 @@ pub use self::{
 #[cfg(feature = "float_deserialize")]
 #[inline(always)]
 pub(crate) fn is_acceptable_float(float: f64) -> bool {
-    #[cfg(not(feature = "lax_deserialize"))]
-    {
-        if float.fract() != 0.0 {
-            return false;
-        }
-    }
-
-    !float.is_nan()
+    !float.is_nan() && float.fract() == 0.0
 }

src/uint.rs

@@ -585,10 +585,7 @@ impl<'de> Deserialize<'de> for UInt {
 
         #[cfg(feature = "float_deserialize")]
         {
-            #[cfg(not(feature = "lax_deserialize"))]
             const EXPECTING: &str = "a number between 0 and 2^53 - 1 without fractional component";
-            #[cfg(feature = "lax_deserialize")]
-            const EXPECTING: &str = "a number between 0 and 2^53 - 1";
 
             let val = f64::deserialize(deserializer)?;
 

tests/int.rs

@@ -41,7 +41,6 @@ fn dont_deserialize_integral_float() {
 }
 
 #[test]
-#[cfg_attr(feature = "lax_deserialize", ignore)]
 fn dont_deserialize_fractional_float() {
     assert!(deserialize_from(0.5).is_err());
     assert!(deserialize_from(42.1337).is_err());
@@ -67,14 +66,6 @@ fn deserialize_integral_float() {
     assert!(deserialize_from(f64::NEG_INFINITY).is_err());
 }
 
-#[test]
-#[cfg_attr(not(feature = "lax_deserialize"), ignore)]
-fn deserialize_fractional_float() {
-    assert_eq!(deserialize_from(0.5).unwrap(), int!(0));
-    assert_eq!(deserialize_from(42.1337).unwrap(), int!(42));
-    assert_eq!(deserialize_from(-42.1337).unwrap(), int!(-42));
-}
-
 fn deserialize_from<'de, Value: IntoDeserializer<'de>>(
     value: Value,
 ) -> Result<Int, serde::de::value::Error> {

tests/uint.rs

@@ -35,7 +35,6 @@ fn dont_deserialize_integral_float() {
 }
 
 #[test]
-#[cfg_attr(feature = "lax_deserialize", ignore)]
 fn dont_deserialize_fractional_float() {
     assert!(deserialize_from(0.5).is_err());
     assert!(deserialize_from(42.1337).is_err());
@@ -58,13 +57,6 @@ fn deserialize_integral_float() {
     assert!(deserialize_from(f64::NEG_INFINITY).is_err());
 }
 
-#[test]
-#[cfg_attr(not(feature = "lax_deserialize"), ignore)]
-fn deserialize_fractional_float() {
-    assert_eq!(deserialize_from(0.5).unwrap(), uint!(0));
-    assert_eq!(deserialize_from(42.1337).unwrap(), uint!(42));
-}
-
 fn deserialize_from<'de, Value: IntoDeserializer<'de>>(
     value: Value,
 ) -> Result<UInt, serde::de::value::Error> {