What the developer wrote
Authored by Tobin C. Harding
Use re-export
Use type from `crate` not from `primitives` as is customary.
What changed, and why it matters
This is a tiny internal code cleanup change. It swaps one way of importing a type (Wtxid) for another equivalent way within the project's own test code. There is no functional change, no bug fix, and no security relevance.
No action needed. This is a non-security refactoring commit.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The commit changes a single import in bitcoin/src/blockdata/block.rs test module from primitives::Wtxid to crate::Wtxid, consolidating the import into an existing use crate::{...} statement. This is purely a stylistic refactor using the crate’s re-export. The diff is +1/-2 lines and affects only test code.
Changed components
bitcoin/src/blockdata/block.rs (test module only)Inspect captured patch +1 / −2
diff --git a/bitcoin/src/blockdata/block.rs b/bitcoin/src/blockdata/block.rs
index 1425c9a7..a22a3322 100644
--- a/bitcoin/src/blockdata/block.rs
+++ b/bitcoin/src/blockdata/block.rs
@@ -395,14 +395,13 @@ mod tests {
use hex_unstable::hex;
use internals::ToU64 as _;
- use primitives::Wtxid;
use super::*;
use crate::consensus::encode::{deserialize, serialize};
use crate::pow::test_utils::{u128_to_work, u64_to_work};
use crate::script::{ScriptPubKeyBuf, ScriptSigBuf};
use crate::transaction::{OutPoint, Transaction, TxIn, TxOut, Txid};
- use crate::{block, Amount, CompactTarget, Network, Sequence, TestnetVersion, Witness};
+ use crate::{block, Amount, CompactTarget, Network, Sequence, TestnetVersion, Witness, Wtxid};
#[test]
fn static_vector() {
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.