clippy

Author: dylwil3

crates/ruff_linter/src/rules/flake8_return/rules/function.rs

@@ -462,7 +462,7 @@ fn add_return_none(checker: &Checker, stmt: &Stmt, range: TextRange) {
     }
 }
 
-fn has_implicit_return<'a>(checker: &Checker, stmt: &'a Stmt) -> bool {
+fn has_implicit_return(checker: &Checker, stmt: &Stmt) -> bool {
     match stmt {
         Stmt::If(ast::StmtIf {
             body,
@@ -486,21 +486,18 @@ fn has_implicit_return<'a>(checker: &Checker, stmt: &'a Stmt) -> bool {
             }
 
             // Check if we don't have an else clause
-            if matches!(
+            matches!(
                 elif_else_clauses.last(),
                 None | Some(ast::ElifElseClause { test: Some(_), .. })
-            ) {
-                return true;
-            }
-            false
+            )
         }
         Stmt::Assert(ast::StmtAssert { test, .. }) if is_const_false(test) => false,
         Stmt::While(ast::StmtWhile { test, .. }) if is_const_true(test) => false,
         Stmt::For(ast::StmtFor { orelse, .. }) | Stmt::While(ast::StmtWhile { orelse, .. }) => {
             if let Some(last_stmt) = orelse.last() {
                 has_implicit_return(checker, last_stmt)
             } else {
-                return true;
+                true
             }
         }
         Stmt::Match(ast::StmtMatch { cases, .. }) => cases.iter().any(|case| {
@@ -521,9 +518,7 @@ fn has_implicit_return<'a>(checker: &Checker, stmt: &'a Stmt) -> bool {
         {
             false
         }
-        _ => {
-            return true;
-        }
+        _ => true,
     }
 }