Skip to content

Rename Builder::add_recipient to add_output. #240

Rename Builder::add_recipient to add_output.

Rename Builder::add_recipient to add_output. #240

Triggered via push December 12, 2023 18:25
Status Success
Total duration 41s
Artifacts

lints-beta.yml

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

Annotations

16 warnings
/home/runner/work/orchard/orchard/src/lib.rs#L6
unused import: `H`
/home/runner/work/orchard/orchard/src/lib.rs#L421
usage of `Iterator::fold` on a type that implements `Try`
/home/runner/work/orchard/orchard/src/lib.rs#L459
explicit call to `.into_iter()` in function argument accepting `IntoIterator`
/home/runner/work/orchard/orchard/src/lib.rs#L246
explicit call to `.into_iter()` in function argument accepting `IntoIterator`
/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
8 warnings emitted
unused import: `H`: src/lib.rs#L6
warning: unused import: `H` --> src/constants.rs:6:91 | 6 | pub use fixed_bases::{NullifierK, OrchardFixedBases, OrchardFixedBasesFull, ValueCommitV, H}; | ^ | = note: `#[warn(unused_imports)]` on by default
usage of `Iterator::fold` on a type that implements `Try`: src/lib.rs#L421
warning: usage of `Iterator::fold` on a type that implements `Try` --> src/builder.rs:421:14 | 421 | .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#L459
warning: explicit call to `.into_iter()` in function argument accepting `IntoIterator` --> src/builder.rs:459:22 | 459 | .zip(self.outputs.into_iter()) | ^^^^^^^^^^^^^^^^^^^^^^^^ help: consider removing the `.into_iter()`: `self.outputs` | note: this parameter accepts any `IntoIterator`, so you don't need to call `.into_iter()` --> /rustc/1a06ac5b5d7c9331e8de1aa1fd7e9d3533034b44/library/core/src/iter/traits/iterator.rs:643: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#L471
warning: usage of `Iterator::fold` on a type that implements `Try` --> src/builder.rs:471:14 | 471 | .fold(Some(ValueSum::zero()), |acc, action| { | ______________^ 472 | | acc? + action.value_sum() 473 | | }) | |______________^ 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
explicit call to `.into_iter()` in function argument accepting `IntoIterator`: src/lib.rs#L246
warning: explicit call to `.into_iter()` in function argument accepting `IntoIterator` --> src/note_encryption.rs:246:18 | 246 | .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/1a06ac5b5d7c9331e8de1aa1fd7e9d3533034b44/library/core/src/iter/traits/iterator.rs:643: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
usage of `Iterator::fold` on a type that implements `Try`: src/lib.rs#L195
warning: usage of `Iterator::fold` on a type that implements `Try` --> src/value.rs:195:14 | 195 | iter.fold(Ok(ValueSum(0)), |acc, v| (acc? + *v).ok_or(OverflowError)) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: use `try_fold` instead: `try_fold(ValueSum(0), |acc, v| ...)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_try_fold
usage of `Iterator::fold` on a type that implements `Try`: src/lib.rs#L201
warning: usage of `Iterator::fold` on a type that implements `Try` --> src/value.rs:201:14 | 201 | iter.fold(Ok(ValueSum(0)), |acc, v| (acc? + v).ok_or(OverflowError)) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: use `try_fold` instead: `try_fold(ValueSum(0), |acc, v| ...)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_try_fold
8 warnings emitted
warning: 8 warnings emitted