Implement AsRef<[u8]> for Binary and HexBinary

Author: webmaster128

CHANGELOG.md

@@ -19,11 +19,13 @@ and this project adheres to
   [#1533]).
 - cosmwasm-std: Upgrade `serde-json-wasm` dependency to 0.5.0 which adds map
   support to `to_vec`/`to_binary` and friends.
+- cosmwasm-std: Implement `AsRef<[u8]>` for `Binary` and `HexBinary` ([#1550]).
 
 [#1437]: https://github.com/CosmWasm/cosmwasm/issues/1437
 [#1481]: https://github.com/CosmWasm/cosmwasm/pull/1481
 [#1478]: https://github.com/CosmWasm/cosmwasm/pull/1478
 [#1533]: https://github.com/CosmWasm/cosmwasm/pull/1533
+[#1550]: https://github.com/CosmWasm/cosmwasm/issues/1550
 
 ### Changed
 

packages/std/src/binary.rs

@@ -96,6 +96,12 @@ impl Deref for Binary {
     }
 }
 
+impl AsRef<[u8]> for Binary {
+    fn as_ref(&self) -> &[u8] {
+        self.as_slice()
+    }
+}
+
 // Slice
 impl From<&[u8]> for Binary {
     fn from(binary: &[u8]) -> Self {
@@ -497,6 +503,34 @@ mod tests {
         assert_eq!(binary_slice, &[7u8, 35, 49, 101, 0, 255]);
     }
 
+    #[test]
+    fn binary_implements_as_ref() {
+        // Can use as_ref (this we already get via the Deref implementation)
+        let data = Binary(vec![7u8, 35, 49, 101, 0, 255]);
+        assert_eq!(data.as_ref(), &[7u8, 35, 49, 101, 0, 255]);
+
+        let data = Binary(vec![7u8, 35, 49, 101, 0, 255]);
+        let data_ref = &data;
+        assert_eq!(data_ref.as_ref(), &[7u8, 35, 49, 101, 0, 255]);
+
+        // Implements as ref
+
+        // This is a dummy function to mimic the signature of
+        // https://docs.rs/sha2/0.10.6/sha2/trait.Digest.html#tymethod.digest
+        fn hash(data: impl AsRef<[u8]>) -> u64 {
+            let mut hasher = DefaultHasher::new();
+            data.as_ref().hash(&mut hasher);
+            hasher.finish()
+        }
+
+        let data = Binary(vec![7u8, 35, 49, 101, 0, 255]);
+        hash(data);
+
+        let data = Binary(vec![7u8, 35, 49, 101, 0, 255]);
+        let data_ref = &data;
+        hash(data_ref);
+    }
+
     #[test]
     fn binary_implements_hash() {
         let a1 = Binary::from([0, 187, 61, 11, 250, 0]);

packages/std/src/hex_binary.rs

@@ -92,6 +92,12 @@ impl Deref for HexBinary {
     }
 }
 
+impl AsRef<[u8]> for HexBinary {
+    fn as_ref(&self) -> &[u8] {
+        self.as_slice()
+    }
+}
+
 // Slice
 impl From<&[u8]> for HexBinary {
     fn from(binary: &[u8]) -> Self {
@@ -561,6 +567,34 @@ mod tests {
         assert_eq!(data_slice, &[7u8, 35, 49, 101, 0, 255]);
     }
 
+    #[test]
+    fn hex_binary_implements_as_ref() {
+        // Can use as_ref (this we already get via the Deref implementation)
+        let data = HexBinary(vec![7u8, 35, 49, 101, 0, 255]);
+        assert_eq!(data.as_ref(), &[7u8, 35, 49, 101, 0, 255]);
+
+        let data = HexBinary(vec![7u8, 35, 49, 101, 0, 255]);
+        let data_ref = &data;
+        assert_eq!(data_ref.as_ref(), &[7u8, 35, 49, 101, 0, 255]);
+
+        // Implements as ref
+
+        // This is a dummy function to mimic the signature of
+        // https://docs.rs/sha2/0.10.6/sha2/trait.Digest.html#tymethod.digest
+        fn hash(data: impl AsRef<[u8]>) -> u64 {
+            let mut hasher = DefaultHasher::new();
+            data.as_ref().hash(&mut hasher);
+            hasher.finish()
+        }
+
+        let data = HexBinary(vec![7u8, 35, 49, 101, 0, 255]);
+        hash(data);
+
+        let data = HexBinary(vec![7u8, 35, 49, 101, 0, 255]);
+        let data_ref = &data;
+        hash(data_ref);
+    }
+
     #[test]
     fn hex_binary_implements_hash() {
         let a1 = HexBinary::from([0, 187, 61, 11, 250, 0]);