get rid of require_semi function

Author: J-ZhengLi

clippy_lints/src/question_mark.rs

@@ -122,11 +122,12 @@ fn check_if_let_some_or_err_and_early_return(cx: &LateContext<'_>, expr: &Expr<'
             let mut applicability = Applicability::MachineApplicable;
             let receiver_str = snippet_with_applicability(cx, let_expr.span, "..", &mut applicability);
             let by_ref = matches!(annot, BindingAnnotation::Ref | BindingAnnotation::RefMut);
+            let requires_semi = matches!(get_parent_node(expr), Some(Node::Stmt(_)));
             let replacement = format!(
                 "{}{}?{}",
                 receiver_str,
                 if by_ref { ".as_ref()" } else { "" },
-                if requires_semi(if_then, if_else) { ";" } else { "" }
+                if requires_semi { ";" } else { "" }
             );
             offer_suggestion(cx, expr, replacement, applicability);
         }
@@ -202,18 +203,6 @@ fn expr_return_none_or_err(
     }
 }
 
-/// Check whether the fixed code needs semicolon after `?`
-///
-/// It will need a semicolon if all block(s) has one.
-fn requires_semi(if_then: &Expr<'_>, if_else: Option<&Expr<'_>>) -> bool {
-    let if_then_kind = &peel_blocks_with_stmt(if_then).kind;
-    let if_then_is_ret_stmt = matches!(if_then_kind, ExprKind::Ret(..));
-    if if_else.is_none() || matches!(peel_blocks_with_stmt(if_else.unwrap()).kind, ExprKind::Ret(_)) {
-        return if_then_is_ret_stmt;
-    }
-    false
-}
-
 fn offer_suggestion(cx: &LateContext<'_>, expr: &Expr<'_>, suggestion: String, applicability: Applicability) {
     if !suggestion.is_empty() {
         span_lint_and_sugg(