Skip to content

Commit eb692a7

Browse files
authored
fix(core)!: Change SerializeSeq trait to return () for each individual element. (#115)
1 parent cf150b5 commit eb692a7

File tree

4 files changed

+7
-7
lines changed

4 files changed

+7
-7
lines changed

xmlity-quick-xml/src/ser.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -385,7 +385,7 @@ impl<'a> ser::Serializer for &'a mut TextSerializer {
385385

386386
type SerializeSeq = &'a mut TextSerializer;
387387

388-
fn serialize_text<S: AsRef<str>>(self, text: S) -> Result<Self::Ok, Self::Error> {
388+
fn serialize_text<S: AsRef<str>>(self, text: S) -> Result<(), Self::Error> {
389389
if self.value.is_some() {
390390
return Err(Error::unexpected_serialize(Unexpected::Text));
391391
}
@@ -663,7 +663,7 @@ impl<W: Write> ser::SerializeSeq for ChildrenSerializeSeq<'_, W> {
663663
type Ok = ();
664664
type Error = Error;
665665

666-
fn serialize_element<V: Serialize>(&mut self, value: &V) -> Result<Self::Ok, Self::Error> {
666+
fn serialize_element<V: Serialize>(&mut self, value: &V) -> Result<(), Self::Error> {
667667
value.serialize(self.serializer.deref_mut())
668668
}
669669

@@ -701,7 +701,7 @@ impl<W: Write> ser::SerializeSeq for SerializeSeq<'_, W> {
701701
type Ok = ();
702702
type Error = Error;
703703

704-
fn serialize_element<V: Serialize>(&mut self, v: &V) -> Result<Self::Ok, Self::Error> {
704+
fn serialize_element<V: Serialize>(&mut self, v: &V) -> Result<(), Self::Error> {
705705
v.serialize(self.serializer.deref_mut())
706706
}
707707

xmlity/src/noop.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ impl<Ok, Err: ser::Error> crate::ser::SerializeSeq for NoopDeSerializer<Ok, Err>
1313

1414
type Error = Err;
1515

16-
fn serialize_element<V: ser::Serialize>(&mut self, _: &V) -> Result<Self::Ok, Self::Error> {
16+
fn serialize_element<V: ser::Serialize>(&mut self, _: &V) -> Result<(), Self::Error> {
1717
unreachable!("Self has infallible child - cannot be constructed and thus cannot be used.")
1818
}
1919

xmlity/src/ser.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -133,7 +133,7 @@ pub trait SerializeSeq {
133133
type Error: Error;
134134

135135
/// Serialize an element in the sequence.
136-
fn serialize_element<V: Serialize>(&mut self, v: &V) -> Result<Self::Ok, Self::Error>;
136+
fn serialize_element<V: Serialize>(&mut self, v: &V) -> Result<(), Self::Error>;
137137

138138
/// End the serialization of the sequence.
139139
fn end(self) -> Result<Self::Ok, Self::Error>;

xmlity/src/value/serializer.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -338,7 +338,7 @@ impl crate::ser::SerializeSeq for &mut XmlSeq<XmlValue> {
338338

339339
type Error = XmlValueSerializerError;
340340

341-
fn serialize_element<V: Serialize>(&mut self, v: &V) -> Result<Self::Ok, Self::Error> {
341+
fn serialize_element<V: Serialize>(&mut self, v: &V) -> Result<(), Self::Error> {
342342
v.serialize(self)
343343
}
344344

@@ -352,7 +352,7 @@ impl crate::ser::SerializeSeq for &mut XmlSeq<XmlChild> {
352352

353353
type Error = XmlValueSerializerError;
354354

355-
fn serialize_element<V: Serialize>(&mut self, v: &V) -> Result<Self::Ok, Self::Error> {
355+
fn serialize_element<V: Serialize>(&mut self, v: &V) -> Result<(), Self::Error> {
356356
v.serialize(self)?;
357357
Ok(())
358358
}

0 commit comments

Comments
 (0)