Skip to content

Merge pull request #434 from nuttycom/release/orchard-0.9.0 #292

Merge pull request #434 from nuttycom/release/orchard-0.9.0

Merge pull request #434 from nuttycom/release/orchard-0.9.0 #292

Triggered via push August 12, 2024 19:25
Status Success
Total duration 39s
Artifacts

lints-beta.yml

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

Annotations

21 warnings
/home/runner/work/orchard/orchard/src/lib.rs#L534
usage of `Iterator::fold` on a type that implements `Try`
/home/runner/work/orchard/orchard/src/lib.rs#L615
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#L244
explicit call to `.into_iter()` in function argument accepting `IntoIterator`
/home/runner/work/orchard/orchard/src/lib.rs#L121
the borrowed expression implements the required traits
/home/runner/work/orchard/orchard/src/lib.rs#L195
usage of `Iterator::fold` on a type that implements `Try`
/home/runner/work/orchard/orchard/src/lib.rs#L201
usage of `Iterator::fold` on a type that implements `Try`
./file/that/probably/doesnt/exist.rs#L0
12 warnings emitted
usage of `Iterator::fold` on a type that implements `Try`: src/lib.rs#L534
warning: usage of `Iterator::fold` on a type that implements `Try` --> src/builder.rs:534:14 | 534 | .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#L615
warning: explicit call to `.into_iter()` in function argument accepting `IntoIterator` --> src/builder.rs:615:18 | 615 | .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/2059132d3c715514da0e4f9cd395a708e666e306/library/core/src/iter/traits/iterator.rs:603: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#L638
warning: usage of `Iterator::fold` on a type that implements `Try` --> src/builder.rs:638:10 | 638 | .fold(Some(ValueSum::zero()), |acc, action| { | __________^ 639 | | acc? + action.value_sum() 640 | | }) | |__________^ 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] | +++
explicit call to `.into_iter()` in function argument accepting `IntoIterator`: src/lib.rs#L244
warning: explicit call to `.into_iter()` in function argument accepting `IntoIterator` --> src/note_encryption.rs:244:18 | 244 | .zip(ephemeral_keys.into_iter()) | ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider removing the `.into_iter()`: `ephemeral_keys` | note: this parameter accepts any `IntoIterator`, so you don't need to call `.into_iter()` --> /rustc/2059132d3c715514da0e4f9cd395a708e666e306/library/core/src/iter/traits/iterator.rs:603:12 = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_conversion
non-canonical implementation of `partial_cmp` on an `Ord` type: src/lib.rs#L102
warning: non-canonical implementation of `partial_cmp` on an `Ord` type --> src/primitives/redpallas.rs:102:1 | 102 | / impl<T: SigType> PartialOrd for VerificationKey<T> { 103 | | fn partial_cmp(&self, other: &Self) -> Option<Ordering> { | | _____________________________________________________________- 104 | || <[u8; 32]>::from(self).partial_cmp(&<[u8; 32]>::from(other)) 105 | || } | ||_____- help: change this to: `{ Some(self.cmp(other)) }` 106 | | } | |__^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#non_canonical_partial_ord_impl = note: `#[warn(clippy::non_canonical_partial_ord_impl)]` on by default
the borrowed expression implements the required traits: src/lib.rs#L121
warning: the borrowed expression implements the required traits --> src/tree.rs:121:65 | 121 | .map(|_| MerkleHashOrchard(pallas::Base::random(&mut rng))), | ^^^^^^^^ help: change this to: `rng` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args = note: `#[warn(clippy::needless_borrows_for_generic_args)]` on by default