Add and update comments #834
Annotations
10 warnings
usage of `Iterator::fold` on a type that implements `Try`:
src/lib.rs#L905
warning: usage of `Iterator::fold` on a type that implements `Try`
--> src/builder.rs:905:10
|
905 | .fold(Some(ValueSum::zero()), |acc, action| {
| __________^
906 | | acc? + action.value_sum()
907 | | })
| |__________^ 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#L54
warning: doc list item without indentation
--> src/bundle/commitments.rs:54:5
|
54 | /// 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
|
54 | /// with ZCASH_ORCHARD_ACTIONS_COMPACT_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_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
|
56 | /// with ZCASH_ORCHARD_ACTIONS_MEMOS_HASH_PERSONALIZATION
| +
|
doc list item without indentation:
src/lib.rs#L58
warning: doc list item without indentation
--> src/bundle/commitments.rs:58:5
|
58 | /// 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
|
58 | /// with ZCASH_ORCHARD_ACTIONS_NONCOMPACT_HASH_PERSONALIZATION
| +
|
doc list item without indentation:
src/lib.rs#L59
warning: doc list item without indentation
--> src/bundle/commitments.rs:59:5
|
59 | /// 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
|
59 | /// 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> {
|
|
first doc comment paragraph is too long:
src/lib.rs#L96
warning: first doc comment paragraph is too long
--> src/circuit.rs:96:1
|
96 | / /// The `OrchardCircuit` trait defines an interface for different implementations of the PLONK circuit
97 | | /// for the different Orchard protocol flavors (Vanilla and ZSA). It serves as a bridge between
98 | | /// plonk::Circuit interfaces and specific requirements of the Orchard protocol's variations.
| |_
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#too_long_first_doc_paragraph
= note: `#[warn(clippy::too_long_first_doc_paragraph)]` on by default
|
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();
| +++++++
|