RGBCube/uutils-coreutils
1
Fork 0
mirror of https://github.com/RGBCube/uutils-coreutils synced 2025-07-29 12:07:46 +00:00
Code Issues Activity

factor::numeric::gcd: Refactor divisor test

Browse source 
Should be clearer; does not handle the `gcd(0, 0)` case, which is already
covered by the `zero` test.
This commit is contained in:
nicoo 2021-01-20 19:10:07 +01:00 • committed by Sylvestre Ledru
parent 42b048d316
commit ea20b22d12
1 changed files with 6 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

13
src/uu/factor/src/numeric/gcd.rs
View file

@ -76,14 +76,13 @@ mod tests {
gcd(0, a) == a
}
fn divisor(a: u64, b: u64) -> bool {
// Test that gcd(a, b) divides a and b
fn divisor(a: u64, b: u64) -> () {
// Test that gcd(a, b) divides a and b, unless a == b == 0
if a == 0 && b == 0 { return; }
let g = gcd(a, b);
if g != 0 {
a % g == 0 && b % g == 0
} else {
a == 0 && b == 0 // for g == 0
}
assert_eq!(a % g, 0);
assert_eq!(b % g, 0);
}
fn commutative(a: u64, b: u64) -> bool {