From 4a90c790bb753ee6284764d973b2796d905741b0 Mon Sep 17 00:00:00 2001
From: Martin Habovstiak <martin.habovstiak@gmail.com>
Date: Wed, 9 Feb 2022 20:35:06 +0100
Subject: [PATCH] Breaking: changed `Parity` serialization to `u8`

Serializing the value as `u8` is more compact but this is a breaking
change.

`Visitor` was renamed to avoid hungarian notation and maybe allow other
integers in the future.
---
 src/key.rs | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/src/key.rs b/src/key.rs
index 9ce636141..a5449ed0e 100644
--- a/src/key.rs
+++ b/src/key.rs
@@ -1271,41 +1271,41 @@ impl BitXor for Parity {
     }
 }
 
-/// The parity is serialized as `i32` - `0` for even, `1` for odd.
+/// The parity is serialized as `u8` - `0` for even, `1` for odd.
 #[cfg(feature = "serde")]
 #[cfg_attr(docsrs, doc(cfg(feature = "serde")))]
 impl ::serde::Serialize for Parity {
     fn serialize<S: ::serde::Serializer>(&self, s: S) -> Result<S::Ok, S::Error> {
-        s.serialize_i32(self.to_i32())
+        s.serialize_u8(self.to_u8())
     }
 }
 
-/// The parity is deserialized as `i32` - `0` for even, `1` for odd.
+/// The parity is deserialized as `u8` - `0` for even, `1` for odd.
 #[cfg(feature = "serde")]
 #[cfg_attr(docsrs, doc(cfg(feature = "serde")))]
 impl<'de> ::serde::Deserialize<'de> for Parity {
     fn deserialize<D: ::serde::Deserializer<'de>>(d: D) -> Result<Self, D::Error> {
-        struct I32Visitor;
+        struct Visitor;
 
-        impl<'de> ::serde::de::Visitor<'de> for I32Visitor
+        impl<'de> ::serde::de::Visitor<'de> for Visitor
         {
             type Value = Parity;
 
             fn expecting(&self, formatter: &mut fmt::Formatter) -> fmt::Result {
-                formatter.write_str("32-bit integer with value 0 or 1")
+                formatter.write_str("8-bit integer (byte) with value 0 or 1")
             }
 
-            fn visit_i32<E>(self, v: i32) -> Result<Self::Value, E>
+            fn visit_u8<E>(self, v: u8) -> Result<Self::Value, E>
                 where E: ::serde::de::Error
             {
                 use serde::de::Unexpected;
 
-                Parity::from_i32(v)
-                    .map_err(|error| E::unexpected_value(Unexpected::Signed(v.into()), "0 or 1"))
+                Parity::from_u8(v)
+                    .map_err(|error| E::unexpected_value(Unexpected::Unsigned(v.into()), "0 or 1"))
             }
         }
 
-        d.deserialize_i32(I32Visitor)
+        d.deserialize_u8(Visitor)
     }
 }