Skip to content
This repository was archived by the owner on Mar 11, 2025. It is now read-only.

Commit 814310a

Browse files
committed
cargo clippy
1 parent abfb964 commit 814310a

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

token/confidential-transfer/proof-generation/src/transfer_with_fee.rs

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -244,11 +244,15 @@ pub fn transfer_with_fee_split_proof_data(
244244
const DELTA_BIT_LENGTH: usize = 48;
245245
const MAX_FEE_BASIS_POINTS: u64 = 10_000;
246246

247-
let delta_fee_complement = MAX_FEE_BASIS_POINTS - delta_fee;
247+
let delta_fee_complement = MAX_FEE_BASIS_POINTS
248+
.checked_sub(delta_fee)
249+
.ok_or(TokenProofGenerationError::FeeCalculation)?;
248250

249251
let max_fee_basis_points_commitment =
250252
Pedersen::with(MAX_FEE_BASIS_POINTS, &PedersenOpening::default());
253+
#[allow(clippy::arithmetic_side_effects)]
251254
let claimed_complement_commitment = max_fee_basis_points_commitment - claimed_commitment;
255+
#[allow(clippy::arithmetic_side_effects)]
252256
let claimed_complement_opening = PedersenOpening::default() - &claimed_opening;
253257

254258
let range_proof_data = BatchedRangeProofU256Data::new(
@@ -320,6 +324,7 @@ fn calculate_fee(transfer_amount: u64, fee_rate_basis_points: u16) -> Option<(u6
320324
Some((fee as u64, delta_fee as u64))
321325
}
322326

327+
#[allow(clippy::arithmetic_side_effects)]
323328
fn compute_delta_commitment_and_opening(
324329
(combined_commitment, combined_opening): (&PedersenCommitment, &PedersenOpening),
325330
(combined_fee_commitment, combined_fee_opening): (&PedersenCommitment, &PedersenOpening),

0 commit comments

Comments
 (0)