Skip to content

Commit 106fdd7

Browse files
committed
Some additional changes to ignore clippy warnings
1 parent d46dddc commit 106fdd7

File tree

3 files changed

+11
-8
lines changed

3 files changed

+11
-8
lines changed

src/ciphers/sha3.rs

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -271,10 +271,11 @@ fn h2b(h: &[u8], n: usize) -> Vec<bool> {
271271
}
272272

273273
fn b2h(s: &[bool]) -> Vec<u8> {
274-
let m = if !(s.len().is_multiple_of(U8BITS)) {
275-
(s.len() / 8) + 1
276-
} else {
274+
// changed this function a bit to stop the clippy warnings
275+
let m = if s.len().is_multiple_of(U8BITS) {
277276
s.len() / 8
277+
} else {
278+
(s.len() / 8) + 1
278279
};
279280
let mut bytes = vec![0u8; m];
280281

src/math/prime_check.rs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,8 @@ pub fn prime_check(num: usize) -> bool {
88

99
let stop: usize = (num as f64).sqrt() as usize + 1;
1010
for i in (3..stop).step_by(2) {
11-
if num % i == 0 {
11+
// changed % operation to is_multiple_of()
12+
if num.is_multiple_of(i) {
1213
return false;
1314
}
1415
}

src/math/prime_factors.rs

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6,14 +6,15 @@ pub fn prime_factors(n: u64) -> Vec<u64> {
66
let mut factors = Vec::new();
77
while i * i <= n {
88
// changed % operation to is_multiple_of()
9-
if !(n.is_multiple_of(i)) {
9+
// changed function a bit to remove clippy warnings
10+
if n.is_multiple_of(i) {
11+
n /= i;
12+
factors.push(i);
13+
} else {
1014
if i != 2 {
1115
i += 1;
1216
}
1317
i += 1;
14-
} else {
15-
n /= i;
16-
factors.push(i);
1718
}
1819
}
1920
if n > 1 {

0 commit comments

Comments
 (0)