Avoid manually calling to_owned and rely on the visitor contract instead.

adamreichold · adamreichold · commit b43c682e518d · 2025-04-09T21:56:28.000+02:00
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/Cargo.toml b/Cargo.toml
@@ -7,7 +7,7 @@ authors = ["Adam Reichold <adam.reichold@t-online.de>"]
 license = "MIT OR Apache-2.0"
 repository = "https://github.com/adamreichold/serde-mincode"
 documentation = "https://docs.rs/serde-mincode"
-version = "0.1.3"
+version = "0.1.4"
 edition = "2021"
 
 [dependencies]
diff --git a/src/de.rs b/src/de.rs
@@ -147,7 +147,7 @@ impl<'de> serde::de::Deserializer<'de> for &mut Decoder<'de> {
     {
         let bytes = self.decode_bytes()?;
 
-        visitor.visit_byte_buf(bytes.to_owned())
+        visitor.visit_bytes(bytes)
     }
 
     fn deserialize_str<V>(self, visitor: V) -> Result<V::Value, Self::Error>
@@ -173,7 +173,7 @@ impl<'de> serde::de::Deserializer<'de> for &mut Decoder<'de> {
             return Error::InvalidStr.into();
         };
 
-        visitor.visit_string(value.to_owned())
+        visitor.visit_str(value)
     }
 
     fn deserialize_unit<V>(self, visitor: V) -> Result<V::Value, Self::Error>

Original file line number	Diff line number	Diff line change
`@@ -147,7 +147,7 @@ impl<'de> serde::de::Deserializer<'de> for &mut Decoder<'de> {`
`147`	`147`	`{`
`148`	`148`	`let bytes = self.decode_bytes()?;`
`149`	`149`
`150`		`- visitor.visit_byte_buf(bytes.to_owned())`
	`150`	`+ visitor.visit_bytes(bytes)`
`151`	`151`	`}`
`152`	`152`
`153`	`153`	`fn deserialize_str<V>(self, visitor: V) -> Result<V::Value, Self::Error>`
`@@ -173,7 +173,7 @@ impl<'de> serde::de::Deserializer<'de> for &mut Decoder<'de> {`
`173`	`173`	`return Error::InvalidStr.into();`
`174`	`174`	`};`
`175`	`175`
`176`		`- visitor.visit_string(value.to_owned())`
	`176`	`+ visitor.visit_str(value)`
`177`	`177`	`}`
`178`	`178`
`179`	`179`	`fn deserialize_unit<V>(self, visitor: V) -> Result<V::Value, Self::Error>`