Skip to content

Commit

Permalink
Narrow the use of #![allow(clippy::too_many_arguments)] in src/rust…
Browse files Browse the repository at this point in the history
…/src/sapling.rs.

This depends on the fix for dtolnay/cxx#1061 in cxx 1.0.70.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>
  • Loading branch information
daira committed Jul 25, 2022
1 parent a227652 commit fe9dc18
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions src/rust/src/sapling.rs
Expand Up @@ -2,11 +2,6 @@
// Distributed under the MIT software license, see the accompanying
// file COPYING or https://www.opensource.org/licenses/mit-license.php .

// This is added because `check_spend` takes several arguments over FFI. This
// annotation gets removed by the cxx procedural macro so it needs to be enabled
// on the entire module.
#![allow(clippy::too_many_arguments)]

use std::convert::TryInto;

use bellman::groth16::{prepare_verifying_key, Proof};
Expand Down Expand Up @@ -66,6 +61,7 @@ mod ffi {
type Verifier;

fn init_verifier() -> Box<Verifier>;
#[allow(clippy::too_many_arguments)]
fn check_spend(
self: &mut Verifier,
cv: &[u8; 32],
Expand Down Expand Up @@ -226,6 +222,7 @@ fn init_verifier() -> Box<Verifier> {
}

impl Verifier {
#[allow(clippy::too_many_arguments)]
fn check_spend(
&mut self,
cv: &[u8; 32],
Expand Down

0 comments on commit fe9dc18

Please sign in to comment.