feat(schema): for HashMap<K, V> -> HashMap<K, V, S>, for `HashSet…

…<T>` -> `HashSet<T, S>` (#294)

* assert variant too suprising

* may be this right?

* and set

* clean

* why

borsh/src/schema.rs

@@ -696,7 +696,11 @@ pub mod hashes {
     #[cfg(not(feature = "std"))]
     use alloc::format;
 
-    impl<K, V> BorshSchema for HashMap<K, V>
+    // S is not serialized, so we ignore it in schema too
+    // forcing S to be BorshSchema forces to define Definition
+    // which must be empty, but if not - it will fail
+    // so better to ignore it
+    impl<K, V, S> BorshSchema for HashMap<K, V, S>
     where
         K: BorshSchema,
         V: BorshSchema,
@@ -715,7 +719,8 @@ pub mod hashes {
             format!(r#"HashMap<{}, {}>"#, K::declaration(), V::declaration())
         }
     }
-    impl<T> BorshSchema for HashSet<T>
+
+    impl<T, S> BorshSchema for HashSet<T, S>
     where
         T: BorshSchema,
     {

borsh/src/ser/mod.rs

@@ -384,9 +384,8 @@ pub mod hashes {
             u32::try_from(vec.len())
                 .map_err(|_| ErrorKind::InvalidData)?
                 .serialize(writer)?;
-            for (key, value) in vec {
-                key.serialize(writer)?;
-                value.serialize(writer)?;
+            for kv in vec {
+                kv.serialize(writer)?;
             }
             Ok(())
         }