From e36debbacf070bf9944f3cac33d40fc33e751869 Mon Sep 17 00:00:00 2001 From: Ryan Summers Date: Wed, 24 Jul 2024 16:32:24 +0200 Subject: [PATCH] Fixing clippy --- rust-toolchain | 1 - src/de/mod.rs | 2 +- src/ser/map.rs | 8 ++++---- src/ser/mod.rs | 26 +++++++++++++------------- src/ser/seq.rs | 12 ++++++------ src/ser/struct_.rs | 8 ++++---- 6 files changed, 28 insertions(+), 29 deletions(-) delete mode 100644 rust-toolchain diff --git a/rust-toolchain b/rust-toolchain deleted file mode 100644 index 2bf5ad044..000000000 --- a/rust-toolchain +++ /dev/null @@ -1 +0,0 @@ -stable diff --git a/src/de/mod.rs b/src/de/mod.rs index d1c5048a7..f2ccbe2ba 100644 --- a/src/de/mod.rs +++ b/src/de/mod.rs @@ -758,7 +758,7 @@ where #[cfg(test)] mod tests { - use core::str::FromStr; + use serde_derive::Deserialize; #[derive(Debug, Deserialize, PartialEq)] diff --git a/src/ser/map.rs b/src/ser/map.rs index 8e010c75a..fe17a1f2e 100644 --- a/src/ser/map.rs +++ b/src/ser/map.rs @@ -22,9 +22,9 @@ impl<'a, 'b: 'a> ser::SerializeMap for SerializeMap<'a, 'b> { Ok(()) } - fn serialize_key(&mut self, key: &T) -> Result<()> + fn serialize_key(&mut self, key: &T) -> Result<()> where - T: ser::Serialize, + T: ser::Serialize + ?Sized, { if !self.first { self.ser.push(b',')?; @@ -35,9 +35,9 @@ impl<'a, 'b: 'a> ser::SerializeMap for SerializeMap<'a, 'b> { Ok(()) } - fn serialize_value(&mut self, value: &T) -> Result<()> + fn serialize_value(&mut self, value: &T) -> Result<()> where - T: ser::Serialize, + T: ser::Serialize + ?Sized, { value.serialize(&mut *self.ser)?; Ok(()) diff --git a/src/ser/mod.rs b/src/ser/mod.rs index 746b86bea..977509d7f 100644 --- a/src/ser/mod.rs +++ b/src/ser/mod.rs @@ -186,8 +186,8 @@ macro_rules! serialize_unsigned { macro_rules! serialize_signed { ($self:ident, $N:expr, $v:expr, $ixx:ident, $uxx:ident) => {{ let v = $v; - let (signed, mut v) = if v == $ixx::min_value() { - (true, $ixx::max_value() as $uxx + 1) + let (signed, mut v) = if v == $ixx::MIN { + (true, $ixx::MAX as $uxx + 1) } else if v < 0 { (true, -v as $uxx) } else { @@ -339,9 +339,9 @@ impl<'a, 'b: 'a> ser::Serializer for &'a mut Serializer<'b> { self.extend_from_slice(b"null") } - fn serialize_some(self, value: &T) -> Result + fn serialize_some(self, value: &T) -> Result where - T: ser::Serialize, + T: ser::Serialize + ?Sized, { value.serialize(self) } @@ -363,14 +363,14 @@ impl<'a, 'b: 'a> ser::Serializer for &'a mut Serializer<'b> { self.serialize_str(variant) } - fn serialize_newtype_struct(self, _name: &'static str, value: &T) -> Result + fn serialize_newtype_struct(self, _name: &'static str, value: &T) -> Result where - T: ser::Serialize, + T: ser::Serialize + ?Sized, { value.serialize(self) } - fn serialize_newtype_variant( + fn serialize_newtype_variant( self, _name: &'static str, _variant_index: u32, @@ -378,7 +378,7 @@ impl<'a, 'b: 'a> ser::Serializer for &'a mut Serializer<'b> { value: &T, ) -> Result where - T: ser::Serialize, + T: ser::Serialize + ?Sized, { self.push(b'{')?; let mut s = SerializeStruct::new(self); @@ -441,9 +441,9 @@ impl<'a, 'b: 'a> ser::Serializer for &'a mut Serializer<'b> { Ok(SerializeStructVariant::new(self)) } - fn collect_str(self, value: &T) -> Result + fn collect_str(self, value: &T) -> Result where - T: fmt::Display, + T: fmt::Display + ?Sized, { self.push(b'"')?; @@ -715,7 +715,7 @@ mod tests { assert_eq!( &*crate::to_string::<_, N>(&Temperature { - temperature: -2.3456789012345e-23 + temperature: -2.345_678_8e-23 }) .unwrap(), r#"{"temperature":-2.3456788e-23}"# @@ -871,7 +871,7 @@ mod tests { { let mut aux: String<{ N }> = String::new(); write!(aux, "{:.2}", self.0).unwrap(); - serializer.serialize_bytes(&aux.as_bytes()) + serializer.serialize_bytes(aux.as_bytes()) } } @@ -881,7 +881,7 @@ mod tests { let sd2 = SimpleDecimal(0.000); assert_eq!(&*crate::to_string::<_, N>(&sd2).unwrap(), r#"0.00"#); - let sd3 = SimpleDecimal(22222.777777); + let sd3 = SimpleDecimal(22_222.777); assert_eq!(&*crate::to_string::<_, N>(&sd3).unwrap(), r#"22222.78"#); } } diff --git a/src/ser/seq.rs b/src/ser/seq.rs index 7a4c4943d..f4f799c13 100644 --- a/src/ser/seq.rs +++ b/src/ser/seq.rs @@ -17,9 +17,9 @@ impl<'a, 'b: 'a> ser::SerializeSeq for SerializeSeq<'a, 'b> { type Ok = (); type Error = Error; - fn serialize_element(&mut self, value: &T) -> Result<()> + fn serialize_element(&mut self, value: &T) -> Result<()> where - T: ser::Serialize, + T: ser::Serialize + ?Sized, { if !self.first { self.de.push(b',')?; @@ -40,9 +40,9 @@ impl<'a, 'b: 'a> ser::SerializeTuple for SerializeSeq<'a, 'b> { type Ok = (); type Error = Error; - fn serialize_element(&mut self, value: &T) -> Result<()> + fn serialize_element(&mut self, value: &T) -> Result<()> where - T: ser::Serialize, + T: ser::Serialize + ?Sized, { ser::SerializeSeq::serialize_element(self, value) } @@ -56,9 +56,9 @@ impl<'a, 'b: 'a> ser::SerializeTupleStruct for SerializeSeq<'a, 'b> { type Ok = (); type Error = Error; - fn serialize_field(&mut self, value: &T) -> Result<()> + fn serialize_field(&mut self, value: &T) -> Result<()> where - T: ser::Serialize, + T: ser::Serialize + ?Sized, { ser::SerializeSeq::serialize_element(self, value) } diff --git a/src/ser/struct_.rs b/src/ser/struct_.rs index a6c31a8ca..6a35d347e 100644 --- a/src/ser/struct_.rs +++ b/src/ser/struct_.rs @@ -17,9 +17,9 @@ impl<'a, 'b: 'a> ser::SerializeStruct for SerializeStruct<'a, 'b> { type Ok = (); type Error = Error; - fn serialize_field(&mut self, key: &'static str, value: &T) -> Result<()> + fn serialize_field(&mut self, key: &'static str, value: &T) -> Result<()> where - T: ser::Serialize, + T: ser::Serialize + ?Sized, { // XXX if `value` is `None` we not produce any output for this field if !self.first { @@ -57,9 +57,9 @@ impl<'a, 'b: 'a> ser::SerializeStructVariant for SerializeStructVariant<'a, 'b> type Ok = (); type Error = Error; - fn serialize_field(&mut self, key: &'static str, value: &T) -> Result<()> + fn serialize_field(&mut self, key: &'static str, value: &T) -> Result<()> where - T: ser::Serialize, + T: ser::Serialize + ?Sized, { // XXX if `value` is `None` we not produce any output for this field if !self.first {