Skip to content

Add methods for validating aspects of PCZT bundles #320

Add methods for validating aspects of PCZT bundles

Add methods for validating aspects of PCZT bundles #320

Triggered via push December 12, 2024 13:49
Status Success
Total duration 45s
Artifacts

lints-beta.yml

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

Annotations

23 warnings
/home/runner/work/orchard/orchard/src/lib.rs#L626
usage of `Iterator::fold` on a type that implements `Try`
/home/runner/work/orchard/orchard/src/lib.rs#L798
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#L17
the following explicit lifetimes could be elided: 'a
/home/runner/work/orchard/orchard/src/lib.rs#L42
the following explicit lifetimes could be elided: 'a
/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
usage of `Iterator::fold` on a type that implements `Try`: src/lib.rs#L626
warning: usage of `Iterator::fold` on a type that implements `Try` --> src/builder.rs:626:14 | 626 | .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#L798
warning: explicit call to `.into_iter()` in function argument accepting `IntoIterator` --> src/builder.rs:798:18 | 798 | .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/202008a1b8de96d2e5b6bc02d379db03a877d34d/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#L821
warning: usage of `Iterator::fold` on a type that implements `Try` --> src/builder.rs:821:10 | 821 | .fold(Some(ValueSum::zero()), |acc, action| { | __________^ 822 | | acc? + action.value_sum() 823 | | }) | |__________^ 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> { |
large array defined as const: src/lib.rs#L30
warning: large array defined as const --> src/constants/fixed_bases/commit_ivk_r.rs:30:1 | 30 | pub const U: [[[u8; 32]; super::H]; super::NUM_WINDOWS] = [ | ^ ----- help: make this a static item: `static` | _| | | 31 | | [ 32 | | [ 33 | | 38, 43, 235, 53, 200, 116, 153, 194, 47, 92, 18, 54, 194, 221, 170, 155, 205, 143, 92, ... | 2920 | | ], 2921 | | ]; | |__^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#large_const_arrays = note: `#[warn(clippy::large_const_arrays)]` on by default
large array defined as const: src/lib.rs#L30
warning: large array defined as const --> src/constants/fixed_bases/note_commit_r.rs:30:1 | 30 | pub const U: [[[u8; 32]; super::H]; super::NUM_WINDOWS] = [ | ^ ----- help: make this a static item: `static` | _| | | 31 | | [ 32 | | [ 33 | | 114, 245, 9, 0, 5, 43, 7, 118, 244, 43, 15, 200, 116, 210, 0, 24, 161, 28, 185, 152, ... | 2920 | | ], 2921 | | ]; | |__^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#large_const_arrays