xml: Differentiate between ValueSerialize and ValueDeserialize to prevent unwraps

crates/xml/derive/src/field.rs

@@ -232,7 +232,7 @@ impl Field {
         let builder_field_ident = self.builder_field_ident();
         let value = wrap_option_if_no_default(
             quote! {
-                rustical_xml::Value::deserialize(text.as_ref())?
+                ::rustical_xml::ValueDeserialize::deserialize(text.as_ref())?
             },
             self.attrs.default.is_some(),
         );
@@ -250,8 +250,8 @@ impl Field {
 
         let value = wrap_option_if_no_default(
             quote! {
-            rustical_xml::Value::deserialize(attr.unescape_value()?.as_ref())?
-                    },
+            ::rustical_xml::ValueDeserialize::deserialize(attr.unescape_value()?.as_ref())?
+                },
             self.attrs.default.is_some(),
         );
 
@@ -270,7 +270,7 @@ impl Field {
 
         let value = wrap_option_if_no_default(
             quote! {
-            rustical_xml::Value::deserialize(&String::from_utf8_lossy(name.as_ref()))?
+            rustical_xml::ValueDeserialize::deserialize(&String::from_utf8_lossy(name.as_ref()))?
                     },
             self.attrs.default.is_some(),
         );

crates/xml/derive/src/xml_struct.rs

@@ -239,7 +239,7 @@ impl NamedStruct {
                 quote! {
                     ::quick_xml::events::attributes::Attribute {
                         key: ::quick_xml::name::QName(#field_name),
-                        value: ::std::borrow::Cow::from(::rustical_xml::Value::serialize(&self.#field_index).into_bytes())
+                        value: ::std::borrow::Cow::from(::rustical_xml::ValueSerialize::serialize(&self.#field_index).into_bytes())
                     }
                 }
             });

crates/xml/src/lib.rs

@@ -13,7 +13,7 @@ pub use de::XmlDocument;
 pub use error::XmlDeError;
 pub use se::XmlSerialize;
 pub use se::XmlSerializeRoot;
-pub use value::Value;
+pub use value::{ParseValueError, ValueDeserialize, ValueSerialize};
 pub use xml_derive::XmlRootTag;
 
 impl XmlDeserialize for () {

crates/xml/src/value.rs

@@ -17,18 +17,23 @@ pub enum ParseValueError {
     ParseFloatError(#[from] ParseFloatError),
 }
 
-pub trait Value: Sized {
+pub trait ValueSerialize: Sized {
     fn serialize(&self) -> String;
+}
+
+pub trait ValueDeserialize: Sized {
     fn deserialize(val: &str) -> Result<Self, XmlDeError>;
 }
 
 macro_rules! impl_value_parse {
     ($t:ty) => {
-        impl Value for $t {
+        impl ValueSerialize for $t {
             fn serialize(&self) -> String {
                 self.to_string()
             }
+        }
 
+        impl ValueDeserialize for $t {
             fn deserialize(val: &str) -> Result<Self, XmlDeError> {
                 val.parse()
                     .map_err(ParseValueError::from)
@@ -52,17 +57,13 @@ impl_value_parse!(u64);
 impl_value_parse!(isize);
 impl_value_parse!(usize);
 
-impl Value for &str {
+impl ValueSerialize for &str {
     fn serialize(&self) -> String {
         self.to_string()
     }
-
-    fn deserialize(_val: &str) -> Result<Self, XmlDeError> {
-        Err(XmlDeError::Other("TODO: Handle this error".to_owned()))
-    }
 }
 
-impl<T: Value> XmlDeserialize for T {
+impl<T: ValueDeserialize> XmlDeserialize for T {
     fn deserialize<R: BufRead>(
         reader: &mut quick_xml::NsReader<R>,
         _start: &BytesStart,
@@ -88,11 +89,11 @@ impl<T: Value> XmlDeserialize for T {
             }
         }
 
-        Value::deserialize(&string)
+        ValueDeserialize::deserialize(&string)
     }
 }
 
-impl<T: Value> XmlSerialize for T {
+impl<T: ValueSerialize> XmlSerialize for T {
     fn serialize<W: std::io::Write>(
         &self,
         ns: Option<Namespace>,