zecfoundation
/
zcashd
mirror of https://github.com/ZcashFoundation/zcashd.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

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 <daira@jacaranda.org>
This commit is contained in:
Daira Hopwood 2022-07-25 14:13:55 +01:00
parent a227652d7d
commit fe9dc18003
1 changed files with 2 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/rust/src/sapling.rs
View File

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