fix clippy errors

gfx-rs · Oct 5, 2023 · e7c969b · e7c969b
1 parent e139963
commit e7c969b
Showing 1 changed file with 18 additions and 18 deletions.
diff --git a/src/proc/constant_evaluator.rs b/src/proc/constant_evaluator.rs
@@ -998,15 +998,15 @@ impl<'a> ConstantEvaluator<'a> {
 
                     _ => match (left_value, right_value) {
                         (Literal::I32(a), Literal::I32(b)) => Literal::I32(match op {
-                            BinaryOperator::Add => a
-                                .checked_add(b)
-                                .ok_or(ConstantEvaluatorError::Overflow("addition".into()))?,
-                            BinaryOperator::Subtract => a
-                                .checked_sub(b)
-                                .ok_or(ConstantEvaluatorError::Overflow("subtraction".into()))?,
-                            BinaryOperator::Multiply => a
-                                .checked_mul(b)
-                                .ok_or(ConstantEvaluatorError::Overflow("multiplication".into()))?,
+                            BinaryOperator::Add => a.checked_add(b).ok_or_else(|| {
+                                ConstantEvaluatorError::Overflow("addition".into())
+                            })?,
+                            BinaryOperator::Subtract => a.checked_sub(b).ok_or_else(|| {
+                                ConstantEvaluatorError::Overflow("subtraction".into())
+                            })?,
+                            BinaryOperator::Multiply => a.checked_mul(b).ok_or_else(|| {
+                                ConstantEvaluatorError::Overflow("multiplication".into())
+                            })?,
                             BinaryOperator::Divide => a.checked_div(b).ok_or_else(|| {
                                 if b == 0 {
                                     ConstantEvaluatorError::DivisionByZero
@@ -1036,15 +1036,15 @@ impl<'a> ConstantEvaluator<'a> {
                             _ => return Err(ConstantEvaluatorError::InvalidBinaryOpArgs),
                         }),
                         (Literal::U32(a), Literal::U32(b)) => Literal::U32(match op {
-                            BinaryOperator::Add => a
-                                .checked_add(b)
-                                .ok_or(ConstantEvaluatorError::Overflow("addition".into()))?,
-                            BinaryOperator::Subtract => a
-                                .checked_sub(b)
-                                .ok_or(ConstantEvaluatorError::Overflow("subtraction".into()))?,
-                            BinaryOperator::Multiply => a
-                                .checked_mul(b)
-                                .ok_or(ConstantEvaluatorError::Overflow("multiplication".into()))?,
+                            BinaryOperator::Add => a.checked_add(b).ok_or_else(|| {
+                                ConstantEvaluatorError::Overflow("addition".into())
+                            })?,
+                            BinaryOperator::Subtract => a.checked_sub(b).ok_or_else(|| {
+                                ConstantEvaluatorError::Overflow("subtraction".into())
+                            })?,
+                            BinaryOperator::Multiply => a.checked_mul(b).ok_or_else(|| {
+                                ConstantEvaluatorError::Overflow("multiplication".into())
+                            })?,
                             BinaryOperator::Divide => a
                                 .checked_div(b)
                                 .ok_or(ConstantEvaluatorError::DivisionByZero)?,