Merge pull request #61 from mina86/c

chipshort · web-flow · commit a34f5f229e98 · 2023-08-04T16:07:29.000+02:00
ser: don’t unnecessarily allocate a String when serialising a character
diff --git a/src/ser/map.rs b/src/ser/map.rs
@@ -166,7 +166,8 @@ impl<'a> ser::Serializer for MapKeySerializer<'a> {
     }
 
     fn serialize_char(self, value: char) -> Result<()> {
-        self.ser.serialize_str(&value.to_string())
+        let mut buf = [0u8; 4];
+        self.ser.serialize_str(value.encode_utf8(&mut buf))
     }
 
     fn serialize_bytes(self, _value: &[u8]) -> Result<()> {

Original file line number	Diff line number	Diff line change
`@@ -166,7 +166,8 @@ impl<'a> ser::Serializer for MapKeySerializer<'a> {`
`166`	`166`	`}`
`167`	`167`
`168`	`168`	`fn serialize_char(self, value: char) -> Result<()> {`
`169`		`- self.ser.serialize_str(&value.to_string())`
	`169`	`+ let mut buf = [0u8; 4];`
	`170`	`+ self.ser.serialize_str(value.encode_utf8(&mut buf))`
`170`	`171`	`}`
`171`	`172`
`172`	`173`	`fn serialize_bytes(self, _value: &[u8]) -> Result<()> {`