Use wrapping shift for unsigned types

kellerkindt · kellerkindt · commit acf0a0c3940e · 2021-08-28T13:28:35.000+02:00
diff --git a/library/core/src/num/saturating.rs b/library/core/src/num/saturating.rs
@@ -141,7 +141,7 @@ macro_rules! sh_impl_unsigned {
 
             #[inline]
             fn shl(self, other: $f) -> Saturating<$t> {
-                Saturating(self.0.shl((other & self::shift_max::$t as $f) as u32))
+                Saturating(self.0.wrapping_shl(other as u32))
             }
         }
         forward_ref_binop! { impl Shl, shl for Saturating<$t>, $f,
@@ -162,7 +162,7 @@ macro_rules! sh_impl_unsigned {
 
             #[inline]
             fn shr(self, other: $f) -> Saturating<$t> {
-                Saturating(self.0.shr((other & self::shift_max::$t as $f) as u32))
+                Saturating(self.0.wrapping_shr(other as u32))
             }
         }
         forward_ref_binop! { impl Shr, shr for Saturating<$t>, $f,

Original file line number	Diff line number	Diff line change
`@@ -141,7 +141,7 @@ macro_rules! sh_impl_unsigned {`
`141`	`141`
`142`	`142`	`#[inline]`
`143`	`143`	`fn shl(self, other: $f) -> Saturating<$t> {`
`144`		`- Saturating(self.0.shl((other & self::shift_max::$t as $f) as u32))`
	`144`	`+ Saturating(self.0.wrapping_shl(other as u32))`
`145`	`145`	`}`
`146`	`146`	`}`
`147`	`147`	`forward_ref_binop! { impl Shl, shl for Saturating<$t>, $f,`
`@@ -162,7 +162,7 @@ macro_rules! sh_impl_unsigned {`
`162`	`162`
`163`	`163`	`#[inline]`
`164`	`164`	`fn shr(self, other: $f) -> Saturating<$t> {`
`165`		`- Saturating(self.0.shr((other & self::shift_max::$t as $f) as u32))`
	`165`	`+ Saturating(self.0.wrapping_shr(other as u32))`
`166`	`166`	`}`
`167`	`167`	`}`
`168`	`168`	`forward_ref_binop! { impl Shr, shr for Saturating<$t>, $f,`