milksad
/
rust-bitcoin-unsafe-fast
14
0
Fork 0
Code Issues Pull Requests Activity

kani: Don't bother checking signed to unsigned conversion 

Now that we use MAX_MONEY a signed amount always fits in an unsigned
amount.
This commit is contained in:
Tobin C. Harding 2024-12-18 09:36:54 +11:00
parent 50224eecc2
commit e378cdd8fa
No known key found for this signature in database
GPG Key ID: 40BF9E4C269D6607
1 changed files with 0 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
units/src/amount/verification.rs
View File

@ -42,15 +42,6 @@ fn u_amount_homomorphic() {
let mut amt = Amount::from_sat(max); let mut amt = Amount::from_sat(max);
amt -= Amount::from_sat(min); amt -= Amount::from_sat(min);
assert_eq!(amt, Amount::from_sat(max - min)); assert_eq!(amt, Amount::from_sat(max - min));
assert_eq!(
Amount::from_sat(n1).to_signed(),
if n1 <= i64::MAX as u64 {
Ok(SignedAmount::from_sat(n1.try_into().unwrap()))
} else {
Err(OutOfRangeError::too_big(true))
},
);
} }
#[kani::unwind(4)] #[kani::unwind(4)]
@ -81,13 +72,4 @@ fn s_amount_homomorphic() {
let mut amt = SignedAmount::from_sat(n1); let mut amt = SignedAmount::from_sat(n1);
amt -= SignedAmount::from_sat(n2); amt -= SignedAmount::from_sat(n2);
assert_eq!(amt, SignedAmount::from_sat(n1 - n2)); assert_eq!(amt, SignedAmount::from_sat(n1 - n2));
assert_eq!(
SignedAmount::from_sat(n1).to_unsigned(),
if n1 >= 0 {
Ok(Amount::from_sat(n1.try_into().unwrap()))
} else {
Err(OutOfRangeError { is_signed: false, is_greater_than_max: false })
},
);
} }