From fe9dc18003663d29b983a28a30b9ffe1732ac9c3 Mon Sep 17 00:00:00 2001 From: Daira Hopwood Date: Mon, 25 Jul 2022 14:13:55 +0100 Subject: [PATCH] Narrow the use of `#![allow(clippy::too_many_arguments)]` in src/rust/src/sapling.rs. This depends on the fix for https://github.com/dtolnay/cxx/issues/1061 in cxx 1.0.70. Signed-off-by: Daira Hopwood --- src/rust/src/sapling.rs | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/src/rust/src/sapling.rs b/src/rust/src/sapling.rs index 9c25628fc..0ba8cdf4e 100644 --- a/src/rust/src/sapling.rs +++ b/src/rust/src/sapling.rs @@ -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}; @@ -66,6 +61,7 @@ mod ffi { type Verifier; fn init_verifier() -> Box; + #[allow(clippy::too_many_arguments)] fn check_spend( self: &mut Verifier, cv: &[u8; 32], @@ -226,6 +222,7 @@ fn init_verifier() -> Box { } impl Verifier { + #[allow(clippy::too_many_arguments)] fn check_spend( &mut self, cv: &[u8; 32],