Skip to content

Add Clone to spendAuth #836

Add Clone to spendAuth

Add Clone to spendAuth #836

Triggered via push October 28, 2024 09:32
Status Success
Total duration 43s
Artifacts

lints-beta.yml

on: push
Clippy (beta)
31s
Clippy (beta)
Fit to window
Zoom out
Zoom in

Annotations

17 warnings
/home/runner/work/orchard/orchard/src/lib.rs#L52
doc list item without indentation
/home/runner/work/orchard/orchard/src/lib.rs#L54
doc list item without indentation
/home/runner/work/orchard/orchard/src/lib.rs#L56
doc list item without indentation
/home/runner/work/orchard/orchard/src/lib.rs#L57
doc list item without indentation
/home/runner/work/orchard/orchard/src/lib.rs#L218
the following explicit lifetimes could be elided: 'a
/home/runner/work/orchard/orchard/src/lib.rs#L270
use of a fallible conversion when an infallible one could be used
/home/runner/work/orchard/orchard/src/lib.rs#L337
use of a fallible conversion when an infallible one could be used
/home/runner/work/orchard/orchard/src/lib.rs#L114
unused return value of `std::hash::Hasher::finish` that must be used
usage of `Iterator::fold` on a type that implements `Try`: src/lib.rs#L847
warning: usage of `Iterator::fold` on a type that implements `Try` --> src/builder.rs:847:10 | 847 | .fold(Some(ValueSum::zero()), |acc, action| { | __________^ 848 | | acc? + action.value_sum() 849 | | }) | |__________^ help: use `try_fold` instead: `try_fold(ValueSum::zero(), |acc, action| ...)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_try_fold = note: `#[warn(clippy::manual_try_fold)]` on by default
doc list item without indentation: src/lib.rs#L52
warning: doc list item without indentation --> src/bundle/commitments.rs:52:5 | 52 | /// with ZCASH_ORCHARD_ACTIONS_COMPACT_HASH_PERSONALIZATION | ^^ | = help: if this is supposed to be its own paragraph, add a blank line = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#doc_lazy_continuation = note: `#[warn(clippy::doc_lazy_continuation)]` on by default help: indent this line | 52 | /// with ZCASH_ORCHARD_ACTIONS_COMPACT_HASH_PERSONALIZATION | +
doc list item without indentation: src/lib.rs#L54
warning: doc list item without indentation --> src/bundle/commitments.rs:54:5 | 54 | /// with ZCASH_ORCHARD_ACTIONS_MEMOS_HASH_PERSONALIZATION | ^^ | = help: if this is supposed to be its own paragraph, add a blank line = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#doc_lazy_continuation help: indent this line | 54 | /// with ZCASH_ORCHARD_ACTIONS_MEMOS_HASH_PERSONALIZATION | +
doc list item without indentation: src/lib.rs#L56
warning: doc list item without indentation --> src/bundle/commitments.rs:56:5 | 56 | /// with ZCASH_ORCHARD_ACTIONS_NONCOMPACT_HASH_PERSONALIZATION | ^^ | = help: if this is supposed to be its own paragraph, add a blank line = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#doc_lazy_continuation help: indent this line | 56 | /// with ZCASH_ORCHARD_ACTIONS_NONCOMPACT_HASH_PERSONALIZATION | +
doc list item without indentation: src/lib.rs#L57
warning: doc list item without indentation --> src/bundle/commitments.rs:57:5 | 57 | /// as defined in [ZIP-244: Transaction Identifier Non-Malleability][zip244] | ^ | = help: if this is supposed to be its own paragraph, add a blank line = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#doc_lazy_continuation help: indent this line | 57 | /// as defined in [ZIP-244: Transaction Identifier Non-Malleability][zip244] | +++
the following explicit lifetimes could be elided: 'a: src/lib.rs#L218
warning: the following explicit lifetimes could be elided: 'a --> src/bundle.rs:218:14 | 218 | impl<'a, A: fmt::Debug, D: OrchardDomainCommon> fmt::Debug for Actions<'a, A, D> { | ^^ ^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_lifetimes = note: `#[warn(clippy::needless_lifetimes)]` on by default help: elide the lifetimes | 218 - impl<'a, A: fmt::Debug, D: OrchardDomainCommon> fmt::Debug for Actions<'a, A, D> { 218 + impl<A: fmt::Debug, D: OrchardDomainCommon> fmt::Debug for Actions<'_, A, D> { |
use of a fallible conversion when an infallible one could be used: src/lib.rs#L270
warning: use of a fallible conversion when an infallible one could be used --> src/keys.rs:270:27 | 270 | self.0.to_bytes().try_into().unwrap() | ^^^^^^^^^^^^^^^^^^^ | = note: converting `GenericArray<u8, UInt<UInt<UInt<UInt<UInt<UInt<UTerm, B1>, B0>, B0>, B0>, B0>, B0>>` to `[u8; 32]` cannot fail = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_fallible_conversions = note: `#[warn(clippy::unnecessary_fallible_conversions)]` on by default help: use | 270 - self.0.to_bytes().try_into().unwrap() 270 + self.0.to_bytes().into() |
use of a fallible conversion when an infallible one could be used: src/lib.rs#L337
warning: use of a fallible conversion when an infallible one could be used --> src/keys.rs:337:27 | 337 | self.0.to_bytes().try_into().unwrap() | ^^^^^^^^^^^^^^^^^^^ | = note: converting `GenericArray<u8, UInt<UInt<UInt<UInt<UInt<UInt<UTerm, B1>, B0>, B0>, B0>, B0>, B0>>` to `[u8; 32]` cannot fail = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_fallible_conversions help: use | 337 - self.0.to_bytes().try_into().unwrap() 337 + self.0.to_bytes().into() |
unused return value of `std::hash::Hasher::finish` that must be used: src/lib.rs#L114
warning: unused return value of `std::hash::Hasher::finish` that must be used --> src/note/asset_base.rs:114:9 | 114 | h.finish(); | ^^^^^^^^^^ | = note: `#[warn(unused_must_use)]` on by default help: use `let _ = ...` to ignore the resulting value | 114 | let _ = h.finish(); | +++++++