From 96dfcdf3b7b183f72419980c36de636febd3dae4 Mon Sep 17 00:00:00 2001
From: Noah <34389972+nlanson@users.noreply.github.com>
Date: Fri, 16 Sep 2022 22:29:05 +1000
Subject: [PATCH] Implement From for hash types

---
 bitcoin/src/blockdata/block.rs       | 24 ++++++++++++++++++++++++
 bitcoin/src/blockdata/script.rs      | 24 ++++++++++++++++++++++++
 bitcoin/src/blockdata/transaction.rs | 24 ++++++++++++++++++++++++
 bitcoin/src/util/bip32.rs            | 12 ++++++++++++
 bitcoin/src/util/taproot.rs          | 24 ++++++++++++++++++++++++
 5 files changed, 108 insertions(+)

diff --git a/bitcoin/src/blockdata/block.rs b/bitcoin/src/blockdata/block.rs
index 2990ee86..b09dd6a7 100644
--- a/bitcoin/src/blockdata/block.rs
+++ b/bitcoin/src/blockdata/block.rs
@@ -425,6 +425,30 @@ impl std::error::Error for Bip34Error {
     }
 }
 
+impl From<BlockHeader> for BlockHash {
+    fn from(header: BlockHeader) -> BlockHash {
+        header.block_hash()
+    }
+}
+
+impl From<&BlockHeader> for BlockHash {
+    fn from(header: &BlockHeader) -> BlockHash {
+        header.block_hash()
+    }
+}
+
+impl From<Block> for BlockHash {
+    fn from(block: Block) -> BlockHash {
+        block.block_hash()
+    }
+}
+
+impl From<&Block> for BlockHash {
+    fn from(block: &Block) -> BlockHash {
+        block.block_hash()
+    }
+}
+
 #[cfg(test)]
 mod tests {
     use crate::hashes::hex::FromHex;
diff --git a/bitcoin/src/blockdata/script.rs b/bitcoin/src/blockdata/script.rs
index 43cb6e83..fc536f23 100644
--- a/bitcoin/src/blockdata/script.rs
+++ b/bitcoin/src/blockdata/script.rs
@@ -760,6 +760,30 @@ impl From<Vec<u8>> for Script {
     fn from(v: Vec<u8>) -> Script { Script(v.into_boxed_slice()) }
 }
 
+impl From<Script> for ScriptHash {
+    fn from(script: Script) -> ScriptHash {
+        script.script_hash()
+    }
+}
+
+impl From<&Script> for ScriptHash {
+    fn from(script: &Script) -> ScriptHash {
+        script.script_hash()
+    }
+}
+
+impl From<Script> for WScriptHash {
+    fn from(script: Script) -> WScriptHash {
+        script.wscript_hash()
+    }
+}
+
+impl From<&Script> for WScriptHash {
+    fn from(script: &Script) -> WScriptHash {
+        script.wscript_hash()
+    }
+}
+
 /// A "parsed opcode" which allows iterating over a [`Script`] in a more sensible way.
 #[derive(Debug, PartialEq, Eq, Clone)]
 pub enum Instruction<'a> {
diff --git a/bitcoin/src/blockdata/transaction.rs b/bitcoin/src/blockdata/transaction.rs
index 4cf61aea..d08d3083 100644
--- a/bitcoin/src/blockdata/transaction.rs
+++ b/bitcoin/src/blockdata/transaction.rs
@@ -990,6 +990,30 @@ impl Decodable for Transaction {
     }
 }
 
+impl From<Transaction> for Txid {
+    fn from(tx: Transaction) -> Txid {
+        tx.txid()
+    }
+}
+
+impl From<&Transaction> for Txid {
+    fn from(tx: &Transaction) -> Txid {
+        tx.txid()
+    }
+}
+
+impl From<Transaction> for Wtxid {
+    fn from(tx: Transaction) -> Wtxid {
+        tx.wtxid()
+    }
+}
+
+impl From<&Transaction> for Wtxid {
+    fn from(tx: &Transaction) -> Wtxid {
+        tx.wtxid()
+    }
+}
+
 #[deprecated(since = "0.30.0", note = "use crate::NonStandardSighashType instead")]
 pub use crate::util::sighash::NonStandardSighashType;
 #[deprecated(since = "0.30.0", note = "use crate::EcdsaSighashType instead")]
diff --git a/bitcoin/src/util/bip32.rs b/bitcoin/src/util/bip32.rs
index e5e317b5..1c91b3fa 100644
--- a/bitcoin/src/util/bip32.rs
+++ b/bitcoin/src/util/bip32.rs
@@ -831,6 +831,18 @@ impl FromStr for ExtendedPubKey {
     }
 }
 
+impl From<ExtendedPubKey> for XpubIdentifier {
+    fn from(key: ExtendedPubKey) -> XpubIdentifier {
+        key.identifier()
+    }
+}
+
+impl From<&ExtendedPubKey> for XpubIdentifier {
+    fn from(key: &ExtendedPubKey) -> XpubIdentifier {
+        key.identifier()
+    }
+}
+
 #[cfg(test)]
 mod tests {
     use super::*;
diff --git a/bitcoin/src/util/taproot.rs b/bitcoin/src/util/taproot.rs
index 7e37e2e3..2560b5e7 100644
--- a/bitcoin/src/util/taproot.rs
+++ b/bitcoin/src/util/taproot.rs
@@ -107,6 +107,18 @@ impl TapLeafHash {
     }
 }
 
+impl From<ScriptLeaf> for TapLeafHash {
+    fn from(leaf: ScriptLeaf) -> TapLeafHash {
+        leaf.leaf_hash()
+    }
+}
+
+impl From<&ScriptLeaf> for TapLeafHash {
+    fn from(leaf: &ScriptLeaf) -> TapLeafHash {
+        leaf.leaf_hash()
+    }
+}
+
 impl TapBranchHash {
     /// Computes branch hash given two hashes of the nodes underneath it.
     pub fn from_node_hashes(a: sha256::Hash, b: sha256::Hash) -> TapBranchHash {
@@ -311,6 +323,18 @@ impl TaprootSpendInfo {
     }
 }
 
+impl From<TaprootSpendInfo> for TapTweakHash {
+    fn from(spend_info: TaprootSpendInfo) -> TapTweakHash {
+        spend_info.tap_tweak()
+    }
+}
+
+impl From<&TaprootSpendInfo> for TapTweakHash {
+    fn from(spend_info: &TaprootSpendInfo) -> TapTweakHash {
+        spend_info.tap_tweak()
+    }
+}
+
 /// Builder for building taproot iteratively. Users can specify tap leaf or omitted/hidden branches
 /// in a depth-first search (DFS) walk order to construct this tree.
 ///