Use match arm instead of if else

PyO3 · Jul 15, 2024 · ec20d12 · ec20d12
1 parent 72f8d2e
commit ec20d12
Showing 1 changed file with 30 additions and 40 deletions.
diff --git a/src/types/float.rs b/src/types/float.rs
@@ -145,109 +145,99 @@ macro_rules! impl_partial_eq_for_float {
         impl PartialEq<$float_type> for Bound<'_, PyFloat> {
             #[inline]
             fn eq(&self, other: &$float_type) -> bool {
-                if let Ok(val) = self.extract::<$float_type>() {
-                    val == *other
-                } else {
-                    false
+                match self.extract::<$float_type>() {
+                    Ok(val) => val == *other,
+                    Err(_) => false,
                 }
             }
         }
 
         impl PartialEq<$float_type> for &Bound<'_, PyFloat> {
             #[inline]
             fn eq(&self, other: &$float_type) -> bool {
-                if let Ok(val) = self.extract::<$float_type>() {
-                    val == *other
-                } else {
-                    false
+                match self.extract::<$float_type>() {
+                    Ok(val) => val == *other,
+                    Err(_) => false,
                 }
             }
         }
 
         impl PartialEq<&$float_type> for Bound<'_, PyFloat> {
             #[inline]
             fn eq(&self, other: &&$float_type) -> bool {
-                if let Ok(val) = self.extract::<$float_type>() {
-                    val == **other
-                } else {
-                    false
+                match self.extract::<$float_type>() {
+                    Ok(val) => val == **other,
+                    Err(_) => false,
                 }
             }
         }
 
         impl PartialEq<Bound<'_, PyFloat>> for $float_type {
             #[inline]
             fn eq(&self, other: &Bound<'_, PyFloat>) -> bool {
-                if let Ok(val) = other.extract::<$float_type>() {
-                    val == *self
-                } else {
-                    false
+                match other.extract::<$float_type>() {
+                    Ok(val) => val == *self,
+                    Err(_) => false,
                 }
             }
         }
 
         impl PartialEq<&'_ Bound<'_, PyFloat>> for $float_type {
             #[inline]
             fn eq(&self, other: &&'_ Bound<'_, PyFloat>) -> bool {
-                if let Ok(val) = other.extract::<$float_type>() {
-                    val == *self
-                } else {
-                    false
+                match other.extract::<$float_type>() {
+                    Ok(val) => val == *self,
+                    Err(_) => false,
                 }
             }
         }
 
         impl PartialEq<Bound<'_, PyFloat>> for &'_ $float_type {
             #[inline]
             fn eq(&self, other: &Bound<'_, PyFloat>) -> bool {
-                if let Ok(val) = other.extract::<$float_type>() {
-                    val == **self
-                } else {
-                    false
+                match other.extract::<$float_type>() {
+                    Ok(val) => val == **self,
+                    Err(_) => false,
                 }
             }
         }
 
         impl PartialEq<$float_type> for Borrowed<'_, '_, PyFloat> {
             #[inline]
             fn eq(&self, other: &$float_type) -> bool {
-                if let Ok(val) = self.extract::<$float_type>() {
-                    val == *other
-                } else {
-                    false
+                match self.extract::<$float_type>() {
+                    Ok(val) => val == *other,
+                    Err(_) => false,
                 }
             }
         }
 
         impl PartialEq<&$float_type> for Borrowed<'_, '_, PyFloat> {
             #[inline]
             fn eq(&self, other: &&$float_type) -> bool {
-                if let Ok(val) = self.extract::<$float_type>() {
-                    val == **other
-                } else {
-                    false
+                match self.extract::<$float_type>() {
+                    Ok(val) => val == **other,
+                    Err(_) => false,
                 }
             }
         }
 
         impl PartialEq<Borrowed<'_, '_, PyFloat>> for $float_type {
             #[inline]
             fn eq(&self, other: &Borrowed<'_, '_, PyFloat>) -> bool {
-                if let Ok(val) = other.extract::<$float_type>() {
-                    val == *self
-                } else {
-                    false
+                match other.extract::<$float_type>() {
+                    Ok(val) => val == *self,
+                    Err(_) => false,
                 }
             }
         }
 
         impl PartialEq<Borrowed<'_, '_, PyFloat>> for &$float_type {
             #[inline]
             fn eq(&self, other: &Borrowed<'_, '_, PyFloat>) -> bool {
-                if let Ok(val) = other.extract::<$float_type>() {
-                    val == **self
-                } else {
-                    false
+                match other.extract::<$float_type>() {
+                    Ok(val) => val == **self,
+                    Err(_) => false,
                 }
             }
         }