Skip to content

Memory Wallet Implementation #12

Memory Wallet Implementation

Memory Wallet Implementation #12

Triggered via pull request August 21, 2024 21:59
@ec2ec2
synchronize #1
ec2/memwallet
Status Success
Total duration 2m 33s
Artifacts

audits.yml

on: pull_request
Vet Rust dependencies
2m 25s
Vet Rust dependencies
Check licenses
48s
Check licenses
Fit to window
Zoom out
Zoom in

Annotations

4 errors
unnecessary closure used to substitute value for `Option::None`: zcash_client_memory/src/mem_wallet/wallet_write.rs#L58
error: unnecessary closure used to substitute value for `Option::None` --> zcash_client_memory/src/mem_wallet/wallet_write.rs:58:22 | 58 | .map(|a| a.next().ok_or_else(|| Self::Error::AccountOutOfRange)) | ^^^^^^^^^--------------------------------------------- | | | help: use `ok_or(..)` instead: `ok_or(Self::Error::AccountOutOfRange)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_lazy_evaluations
unnecessary closure used to substitute value for `Option::None`: zcash_client_memory/src/mem_wallet/wallet_write.rs#L53
error: unnecessary closure used to substitute value for `Option::None` --> zcash_client_memory/src/mem_wallet/wallet_write.rs:53:32 | 53 | let seed_fingerprint = SeedFingerprint::from_seed(seed.expose_secret()) | ________________________________^ 54 | | .ok_or_else(|| Self::Error::InvalidSeedLength)?; | |______________--------------------------------------------^ | | | help: use `ok_or(..)` instead: `ok_or(Self::Error::InvalidSeedLength)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_lazy_evaluations = note: `-D clippy::unnecessary-lazy-evaluations` implied by `-D warnings`
using `clone` on type `TransparentAddress` which implements the `Copy` trait: zcash_client_memory/src/mem_wallet/wallet_read.rs#L295
error: using `clone` on type `TransparentAddress` which implements the `Copy` trait --> zcash_client_memory/src/mem_wallet/wallet_read.rs:295:32 | 295 | let addr = output.recipient_address().clone(); | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try dereferencing it: `*output.recipient_address()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#clone_on_copy = note: `-D clippy::clone-on-copy` implied by `-D warnings`
this expression creates a reference which is immediately dereferenced by the compiler: zcash_client_memory/src/mem_wallet/wallet_read.rs#L290
error: this expression creates a reference which is immediately dereferenced by the compiler --> zcash_client_memory/src/mem_wallet/wallet_read.rs:290:35 | 290 | .contains_key(&outpoint) | ^^^^^^^^^ help: change this to: `outpoint` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow = note: `-D clippy::needless-borrow` implied by `-D warnings`