Add support for valuable to fixed hashes

ethereum-types/Cargo.toml

@@ -29,3 +29,4 @@ arbitrary = ["ethbloom/arbitrary", "fixed-hash/arbitrary", "uint-crate/arbitrary
 rlp = ["impl-rlp", "ethbloom/rlp", "primitive-types/rlp"]
 codec = ["impl-codec", "ethbloom/codec", "scale-info", "primitive-types/scale-info"]
 num-traits = ["primitive-types/num-traits"]
+valuable = ["fixed-hash/valuable"]

fixed-hash/Cargo.toml

@@ -21,6 +21,7 @@ rand = { version = "0.8.0", optional = true, default-features = false }
 rustc-hex = { version = "2.0.1", optional = true, default-features = false }
 static_assertions = "1.0.0"
 arbitrary = { version = "1.0", optional = true }
+valuable = { version = "0.1.0", optional = true }
 
 [dev-dependencies]
 rand_xorshift = "0.3.0"

fixed-hash/src/hash.rs

@@ -316,6 +316,7 @@ macro_rules! construct_fixed_hash {
 		impl_rustc_hex_for_fixed_hash!($name);
 		impl_quickcheck_for_fixed_hash!($name);
 		impl_arbitrary_for_fixed_hash!($name);
+		impl_valuable_for_fixed_hash!($name);
 	}
 }
 
@@ -638,6 +639,44 @@ macro_rules! impl_quickcheck_for_fixed_hash {
 	};
 }
 
+// Implementation for disabled valuable crate support.
+//
+// # Note
+//
+// Feature guarded macro definitions instead of feature guarded impl blocks
+// to work around the problems of introducing `valuable` crate feature in
+// a user crate.
+#[cfg(not(feature = "valuable"))]
+#[macro_export]
+#[doc(hidden)]
+macro_rules! impl_valuable_for_fixed_hash {
+	( $name:ident ) => {};
+}
+
+// Implementation for enabled valuable crate support.
+//
+// # Note
+//
+// Feature guarded macro definitions instead of feature guarded impl blocks
+// to work around the problems of introducing `valuable` crate feature in
+// a user crate.
+#[cfg(feature = "valuable")]
+#[macro_export]
+#[doc(hidden)]
+macro_rules! impl_valuable_for_fixed_hash {
+	( $name:ident ) => {
+		impl $crate::valuable::Valuable for $name {
+			fn as_value(&self) -> $crate::valuable::Value<'_> {
+				$crate::valuable::Value::Listable(&self.0)
+			}
+
+			fn visit(&self, visit: &mut dyn $crate::valuable::Visit) {
+				visit.visit_unnamed_fields(&[$crate::valuable::Value::String(&format!("{:?}", self))])
+			}
+		}
+	};
+}
+
 // When the `arbitrary` feature is disabled.
 //
 // # Note

fixed-hash/src/lib.rs

@@ -50,6 +50,10 @@ pub use quickcheck;
 #[doc(hidden)]
 pub use arbitrary;
 
+#[cfg(feature = "valuable")]
+#[doc(hidden)]
+pub use valuable;
+
 #[macro_use]
 mod hash;
 

primitive-types/Cargo.toml

@@ -30,6 +30,7 @@ rlp = ["impl-rlp"]
 arbitrary = ["fixed-hash/arbitrary", "uint/arbitrary"]
 fp-conversion = ["std"]
 num-traits = ["impl-num-traits"]
+valuable = ["fixed-hash/valuable"]
 
 [[test]]
 name = "scale_info"