diff options
author | Andreas Grois <andi@grois.info> | 2022-10-26 21:39:52 +0200 |
---|---|---|
committer | Andreas Grois <andi@grois.info> | 2022-10-26 21:39:52 +0200 |
commit | 09d9c9e7d8d30c7f718837a44cabc90142f85b3e (patch) | |
tree | 1b0e0482c4748cba1ae888b41897ede0a4ea4af3 | |
parent | 839e1096dde8e1e934a82d1cf0c4d3a43dc69f38 (diff) |
Minor, remove a pointless type conversion.
Seems not to have any performance impact, but still, cleaner this way.
-rw-r--r-- | src/passwordmaker/base_conversion/iterative_conversion_impl/mod.rs | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/passwordmaker/base_conversion/iterative_conversion_impl/mod.rs b/src/passwordmaker/base_conversion/iterative_conversion_impl/mod.rs index d3e0045..cee5418 100644 --- a/src/passwordmaker/base_conversion/iterative_conversion_impl/mod.rs +++ b/src/passwordmaker/base_conversion/iterative_conversion_impl/mod.rs @@ -371,7 +371,7 @@ impl<const N : usize> ArbitraryBytes<N>{ let m_extra_digits_dividend = m_plus_n_digits_dividend - n_digits_divisor; //step D1: Normalize. This brings the maximum error for each digit down to no more than 2. - let normalize_shift = divisor.get_digit_from_right(n_digits_divisor - 1).leading_zeros() as usize; + let normalize_shift = divisor.get_digit_from_right(n_digits_divisor - 1).leading_zeros(); //again, missing const generics ruin all the fun. let mut dividend = self.shift_left(normalize_shift); let divisor = divisor.shift_left(normalize_shift); @@ -434,7 +434,7 @@ impl<const N : usize> ArbitraryBytes<N>{ self.0[N-i-1] = val; } - fn shift_left<const M : usize>(&self, s : usize) -> <Self as PadWithAZero>::Output + fn shift_left<const M : usize>(&self, s : u32) -> <Self as PadWithAZero>::Output where Self : PadWithAZero<Output = ArbitraryBytes<M>> { debug_assert!(s < 32); @@ -445,7 +445,7 @@ impl<const N : usize> ArbitraryBytes<N>{ res } - fn shift_right(mut self, s : usize) -> Self { + fn shift_right(mut self, s : u32) -> Self { debug_assert!(s < 32); if s != 0 { let _ = self.0.iter_mut().fold(0u32, |carry, val| { |