What the developer wrote
Authored by soralit
style: run cargo fmt
What changed, and why it matters
This commit is purely a code-formatting cleanup. It removes one debug-print logging statement and adjusts brace style to match Rust's standard formatter. There is no functional change, no bug fix, and no security-relevant behavior change.
No security action needed. Treat as routine style maintenance.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff shows changes produced by cargo fmt. In rust/apps/zcash/src/pczt/check.rs, a find closure is collapsed and a rust_tools::debug_print! call is removed. In rust/rust_c/src/wallet/multi_coins_wallet/keystone_connect.rs, match arms are reformatted (e.g., 0 => { None } becomes 0 => None). No logic, control flow, or data handling changes are introduced.
Changed components
rust/apps/zcash/src/pczt/check.rsrust/rust_c/src/wallet/multi_coins_wallet/keystone_connect.rsInspect captured patch +5 / −8
diff --git a/rust/apps/zcash/src/pczt/check.rs b/rust/apps/zcash/src/pczt/check.rs
index f123b82..2fc656f 100644
--- a/rust/apps/zcash/src/pczt/check.rs
+++ b/rust/apps/zcash/src/pczt/check.rs
@@ -104,10 +104,7 @@ fn check_transparent_input<P: consensus::Parameters>(
let my_derivation = input
.bip32_derivation()
.iter()
- .find(|(_pubkey, derivation)| {
- rust_tools::debug_print!("Checking transparent input bip32 derivation seed fingerprint: {} vs {}", hex::encode(seed_fingerprint), hex::encode(derivation.seed_fingerprint()));
- return seed_fingerprint == derivation.seed_fingerprint();
- });
+ .find(|(_pubkey, derivation)| seed_fingerprint == derivation.seed_fingerprint());
match my_derivation {
None => {
//not my input, pass
diff --git a/rust/rust_c/src/wallet/multi_coins_wallet/keystone_connect.rs b/rust/rust_c/src/wallet/multi_coins_wallet/keystone_connect.rs
index 22cc3c1..bf14c01 100644
--- a/rust/rust_c/src/wallet/multi_coins_wallet/keystone_connect.rs
+++ b/rust/rust_c/src/wallet/multi_coins_wallet/keystone_connect.rs
@@ -35,14 +35,14 @@ pub unsafe extern "C" fn get_keystone_connect_wallet_ur(
};
let sfp = match zcash_sfp_length {
- 0 => {
- None
- }
+ 0 => None,
32 => {
let _sfp = extract_array!(zcash_sfp, u8, zcash_sfp_length);
match <[u8; 32]>::try_from(_sfp) {
Ok(sfp) => Some(sfp),
- Err(e) => return UREncodeResult::from(URError::UrEncodeError(e.to_string())).c_ptr(),
+ Err(e) => {
+ return UREncodeResult::from(URError::UrEncodeError(e.to_string())).c_ptr()
+ }
}
}
_ => {
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.