From 2b5970f993a091ad51624cdbfefee7dd97f65682 Mon Sep 17 00:00:00 2001 From: Michael Watzko Date: Thu, 19 Aug 2021 11:28:33 +0200 Subject: [PATCH] Simplify saturating_div --- library/core/src/num/int_macros.rs | 11 +++-------- 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/library/core/src/num/int_macros.rs b/library/core/src/num/int_macros.rs index 7c36f4cdd203..2c866812937e 100644 --- a/library/core/src/num/int_macros.rs +++ b/library/core/src/num/int_macros.rs @@ -946,14 +946,9 @@ macro_rules! int_impl { without modifying the original"] #[inline] pub const fn saturating_div(self, rhs: Self) -> Self { - let (result, overflowed) = self.overflowing_div(rhs); - - if !overflowed { - result - } else if (self < 0) == (rhs < 0) { - Self::MAX - } else { - Self::MIN + match self.overflowing_div(rhs) { + (result, false) => result, + (_result, true) => Self::MAX, // MIN / -1 is the only possible saturating overflow } }