chore(clippy): fix clippy warnings

dnsl48 · Feb 11, 2024 · bf533c5 · bf533c5
1 parent bdced7b
commit bf533c5
Show file tree

Hide file tree

Showing 2 changed files with 36 additions and 35 deletions.
diff --git a/src/decimal/mod.rs b/src/decimal/mod.rs
@@ -407,38 +407,36 @@ where
 
                             let mut precision: usize = 0;
                             loop {
-                                s_state = match division::divide_rem_resume(
+                                s_state = if let Ok(s) = division::divide_rem_resume(
                                     s_state.take().unwrap(),
                                     |s, d| {
                                         s_digit = d;
                                         Ok(Err(s))
                                     },
                                 ) {
-                                    Ok(s) => {
-                                        if s.remainder.is_zero() {
-                                            None
-                                        } else {
-                                            Some(s)
-                                        }
+                                    if s.remainder.is_zero() {
+                                        None
+                                    } else {
+                                        Some(s)
                                     }
-                                    Err(_) => return false,
+                                } else {
+                                    return false;
                                 };
 
-                                o_state = match division::divide_rem_resume(
+                                o_state = if let Ok(s) = division::divide_rem_resume(
                                     o_state.take().unwrap(),
                                     |s, d| {
                                         o_digit = d;
                                         Ok(Err(s))
                                     },
                                 ) {
-                                    Ok(s) => {
-                                        if s.remainder.is_zero() {
-                                            None
-                                        } else {
-                                            Some(s)
-                                        }
+                                    if s.remainder.is_zero() {
+                                        None
+                                    } else {
+                                        Some(s)
                                     }
-                                    Err(_) => return false,
+                                } else {
+                                    return false
                                 };
 
                                 if s_digit != o_digit {

diff --git a/src/fraction/display.rs b/src/fraction/display.rs
@@ -299,39 +299,42 @@ where
                 let mut length = 0;
 
                 if let Some(precision) = format.precision() {
-                    match division::divide_to_callback(
+                    let callback = |digit| {
+                        length += 1;
+                        division::write_digit(buffer, digit)
+                    };
+
+                    if division::divide_to_callback(
                         numer,
                         denom,
                         *precision,
                         format.alternate(),
-                        |digit| {
-                            length += 1;
-                            division::write_digit(buffer, digit)
-                        },
-                    ) {
-                        Ok(_) => Ok(()),
-                        Err(_) => Err(fmt::Error),
-                    }?;
+                        callback,
+                    ).is_err() {
+                        return Err(fmt::Error);
+                    }
                 } else {
-                    match division::divide_to_callback(numer, T::one(), 0, false, |digit| {
+                    let callback = |digit| {
                         length += 1;
                         division::write_digit(buffer, digit)
-                    }) {
-                        Ok(_) => Ok(()),
-                        Err(_) => Err(fmt::Error),
-                    }?;
+                    };
+
+                    if division::divide_to_callback(numer, T::one(), 0, false, callback).is_err() {
+                        return Err(fmt::Error);
+                    }
 
                     if !ratio.numer().is_zero() && !ratio.denom().is_one() {
                         length += 1;
                         buffer.write_char('/')?;
 
-                        match division::divide_to_callback(denom, T::one(), 0, false, |digit| {
+                        let callback = |digit| {
                             length += 1;
                             division::write_digit(buffer, digit)
-                        }) {
-                            Ok(_) => Ok(()),
-                            Err(_) => Err(fmt::Error),
-                        }?;
+                        };
+
+                        if division::divide_to_callback(denom, T::one(), 0, false, callback).is_err() {
+                            return Err(fmt::Error)
+                        };
                     }
                 }