fix: lint errors

biomejs · Apr 21, 2024 · b733e8d · b733e8d
1 parent 02473e3
commit b733e8d
Showing 1 changed file with 7 additions and 9 deletions.
diff --git a/crates/biome_js_analyze/src/lint/nursery/no_array_constructor.rs b/crates/biome_js_analyze/src/lint/nursery/no_array_constructor.rs
@@ -50,16 +50,14 @@ impl Rule for NoArrayConstructor {
             AnyJsExpression::JsCallExpression(call_expression) => {
                 let callee = call_expression.callee().ok()?;
                 let arguments = call_expression.arguments().ok()?;
-                return validate(callee, arguments);
+                validate(&callee, &arguments)
             }
             AnyJsExpression::JsNewExpression(new_expression) => {
                 let callee = new_expression.callee().ok()?;
                 let arguments = new_expression.arguments()?;
-                return validate(callee, arguments);
-            }
-            _ => {
-                return None;
+                validate(&callee, &arguments)
             }
+            _ => None,
         }
     }
 
@@ -80,19 +78,19 @@ impl Rule for NoArrayConstructor {
     }
 }
 
-fn validate(callee: AnyJsExpression, arguments: JsCallArguments) -> Option<()> {
+fn validate(callee: &AnyJsExpression, arguments: &JsCallArguments) -> Option<()> {
     let len = arguments.args().into_iter().count();
     if callee.text() == "Array" {
         if len == 1
             && !matches!(
-                arguments.args().into_iter().nth(0)?.ok()?,
+                arguments.args().into_iter().next()?.ok()?,
                 AnyJsCallArgument::JsSpread(_)
             )
         {
             return None;
         }
-        return Some(());
+        Some(())
     } else {
-        return None;
+        None
     }
 }