fix: not panic when inf or nan to value

Author: liuq19

Cargo.toml

@@ -10,7 +10,7 @@ license       = "Apache-2.0"
 name          = "sonic-rs"
 readme        = "README.md"
 repository    = "https://github.com/cloudwego/sonic-rs"
-version       = "0.5.4"
+version       = "0.5.5"
 
 # See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
 

examples/iterator.rs

@@ -26,8 +26,8 @@ fn main() {
     let iter = unsafe { to_object_iter_unchecked(&json) };
     for ret in iter {
         // deal with errors
-        if ret.is_err() {
-            println!("{}", ret.unwrap_err());
+        if let Err(e) = ret {
+            println!("{}", e);
             return;
         }
 

src/serde/mod.rs

@@ -354,11 +354,6 @@ mod test {
             "Invalid UTF-8 characters in json at line 1 column 5\n\n\t\"\0\0\0��\"\n\t....^..\n"
         );
 
-        #[derive(Debug, Deserialize, Serialize, PartialEq)]
-        struct TestStruct {
-            char_: char,
-        }
-
         // char's deserialize will iterator on the `str`
         let data = [34, 255, 34];
         let value: crate::Result<char> = from_slice(&data);

src/value/ser.rs

@@ -154,9 +154,7 @@ impl serde::Serializer for Serializer {
         if value.is_finite() {
             Ok(unsafe { Value::new_f64_unchecked(value as f64) })
         } else {
-            Err(key_must_be_str_or_num(Unexpected::Other(
-                "NaN or Infinite f32",
-            )))
+            Ok(Value::new_null())
         }
     }
 
@@ -165,9 +163,7 @@ impl serde::Serializer for Serializer {
         if value.is_finite() {
             Ok(unsafe { Value::new_f64_unchecked(value) })
         } else {
-            Err(key_must_be_str_or_num(Unexpected::Other(
-                "NaN or Infinite f64",
-            )))
+            Ok(Value::new_null())
         }
     }
 
@@ -947,4 +943,12 @@ mod test {
 
         assert_eq!(value["arr"][2]["app_name"].as_str(), Some("test"));
     }
+
+    #[test]
+    fn test_inf_or_nan_to_value() {
+        assert_eq!(to_value(&f64::INFINITY).unwrap(), Value::new_null());
+        assert_eq!(to_value(&f64::NAN).unwrap(), Value::new_null());
+        assert_eq!(to_value(&f32::INFINITY).unwrap(), Value::new_null());
+        assert_eq!(to_value(&f32::NAN).unwrap(), Value::new_null());
+    }
 }