Skip to content

Implement PCZT support #307

Implement PCZT support

Implement PCZT support #307

Triggered via push December 3, 2024 11:46
Status Success
Total duration 40s
Artifacts

lints-beta.yml

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

Annotations

24 warnings
/home/runner/work/orchard/orchard/src/lib.rs#L298
using `clone` on type `RandomSeed` which implements the `Copy` trait
/home/runner/work/orchard/orchard/src/lib.rs#L388
using `clone` on type `RandomSeed` which implements the `Copy` trait
/home/runner/work/orchard/orchard/src/lib.rs#L625
usage of `Iterator::fold` on a type that implements `Try`
/home/runner/work/orchard/orchard/src/lib.rs#L797
explicit call to `.into_iter()` in function argument accepting `IntoIterator`
/home/runner/work/orchard/orchard/src/lib.rs#L19
doc list item without indentation
/home/runner/work/orchard/orchard/src/lib.rs#L21
doc list item without indentation
/home/runner/work/orchard/orchard/src/lib.rs#L23
doc list item without indentation
/home/runner/work/orchard/orchard/src/lib.rs#L24
doc list item without indentation
/home/runner/work/orchard/orchard/src/lib.rs#L171
the following explicit lifetimes could be elided: 'a
/home/runner/work/orchard/orchard/src/lib.rs#L244
explicit call to `.into_iter()` in function argument accepting `IntoIterator`
/home/runner/work/orchard/orchard/src/lib.rs#L90
this expression creates a reference which is immediately dereferenced by the compiler
/home/runner/work/orchard/orchard/src/lib.rs#L214
usage of `Iterator::fold` on a type that implements `Try`
/home/runner/work/orchard/orchard/src/lib.rs#L220
usage of `Iterator::fold` on a type that implements `Try`
Clippy (beta)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
using `clone` on type `RandomSeed` which implements the `Copy` trait: src/lib.rs#L298
warning: using `clone` on type `RandomSeed` which implements the `Copy` trait --> src/builder.rs:298:25 | 298 | rseed: Some(self.note.rseed().clone()), | ^^^^^^^^^^^^^^^^^^^^^^^^^ help: try dereferencing it: `*self.note.rseed()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#clone_on_copy = note: `#[warn(clippy::clone_on_copy)]` on by default
using `clone` on type `RandomSeed` which implements the `Copy` trait: src/lib.rs#L388
warning: using `clone` on type `RandomSeed` which implements the `Copy` trait --> src/builder.rs:388:25 | 388 | rseed: Some(note.rseed().clone()), | ^^^^^^^^^^^^^^^^^^^^ help: try dereferencing it: `*note.rseed()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#clone_on_copy
usage of `Iterator::fold` on a type that implements `Try`: src/lib.rs#L625
warning: usage of `Iterator::fold` on a type that implements `Try` --> src/builder.rs:625:14 | 625 | .fold(Some(ValueSum::zero()), |acc, note_value| acc? + note_value) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: use `try_fold` instead: `try_fold(ValueSum::zero(), |acc, note_value| ...)` | = 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
explicit call to `.into_iter()` in function argument accepting `IntoIterator`: src/lib.rs#L797
warning: explicit call to `.into_iter()` in function argument accepting `IntoIterator` --> src/builder.rs:797:18 | 797 | .zip(indexed_outputs.into_iter()) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider removing the `.into_iter()`: `indexed_outputs` | note: this parameter accepts any `IntoIterator`, so you don't need to call `.into_iter()` --> /rustc/ea5e5ca0553095786f968671ab1c222e5b1df5c3/library/core/src/iter/traits/iterator.rs:589:12 = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_conversion = note: `#[warn(clippy::useless_conversion)]` on by default
usage of `Iterator::fold` on a type that implements `Try`: src/lib.rs#L820
warning: usage of `Iterator::fold` on a type that implements `Try` --> src/builder.rs:820:10 | 820 | .fold(Some(ValueSum::zero()), |acc, action| { | __________^ 821 | | acc? + action.value_sum() 822 | | }) | |__________^ 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
doc list item without indentation: src/lib.rs#L19
warning: doc list item without indentation --> src/bundle/commitments.rs:19:5 | 19 | /// 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 | 19 | /// with ZCASH_ORCHARD_ACTIONS_COMPACT_HASH_PERSONALIZATION | +
doc list item without indentation: src/lib.rs#L21
warning: doc list item without indentation --> src/bundle/commitments.rs:21:5 | 21 | /// 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 | 21 | /// with ZCASH_ORCHARD_ACTIONS_MEMOS_HASH_PERSONALIZATION | +
doc list item without indentation: src/lib.rs#L23
warning: doc list item without indentation --> src/bundle/commitments.rs:23:5 | 23 | /// 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 | 23 | /// with ZCASH_ORCHARD_ACTIONS_NONCOMPACT_HASH_PERSONALIZATION | +
doc list item without indentation: src/lib.rs#L24
warning: doc list item without indentation --> src/bundle/commitments.rs:24:5 | 24 | /// 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 | 24 | /// as defined in [ZIP-244: Transaction Identifier Non-Malleability][zip244] | +++
the following explicit lifetimes could be elided: 'a: src/lib.rs#L171
warning: the following explicit lifetimes could be elided: 'a --> src/bundle.rs:171:14 | 171 | impl<'a, T: fmt::Debug> fmt::Debug for Actions<'a, T> { | ^^ ^^ | = 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 | 171 - impl<'a, T: fmt::Debug> fmt::Debug for Actions<'a, T> { 171 + impl<T: fmt::Debug> fmt::Debug for Actions<'_, T> { |