Use Cow<str> instead of &str in Crop deserialization

serde-rs/serde#1413 (comment)
lumeohq · May 28, 2021 · 609fc18 · 609fc18
1 parent ad003ca
commit 609fc18
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/pipeline/src/node_properties/transform_properties.rs b/pipeline/src/node_properties/transform_properties.rs
@@ -1,6 +1,6 @@
 use serde::{Deserialize, Serialize};
 use serde_with::skip_serializing_none;
-use std::str::FromStr;
+use std::{borrow::Cow, str::FromStr};
 
 #[derive(Debug, Clone, PartialEq, Serialize, Deserialize)]
 pub enum FlipDirection {
@@ -60,9 +60,9 @@ impl<'de> Deserialize<'de> for Crop {
     where
         D: serde::de::Deserializer<'de>,
     {
-        let s = <&str>::deserialize(deserializer)?;
+        let s = <Cow<str>>::deserialize(deserializer)?;
 
-        Crop::from_str(s).map_err(serde::de::Error::custom)
+        Crop::from_str(&s).map_err(serde::de::Error::custom)
     }
 }