about summary refs log tree commit diff
diff options
context:
space:
mode:
authorAda Alakbarova <ada.alakbarova@proton.me>2025-08-22 00:00:23 +0200
committerAda Alakbarova <ada.alakbarova@proton.me>2025-08-22 15:03:16 +0200
commited69835374e72a2c08f8bad0959b212d35cc267c (patch)
tree356fad5c1c3bcf580e31e98a73652a5bba99119a
parentcb3e5af74658fefa55b513a55d2789c84ef4bfd5 (diff)
downloadrust-ed69835374e72a2c08f8bad0959b212d35cc267c.tar.gz
rust-ed69835374e72a2c08f8bad0959b212d35cc267c.zip
basically `clippy::let_if_seq`
-rw-r--r--clippy_lints/src/matches/try_err.rs20
1 files changed, 5 insertions, 15 deletions
diff --git a/clippy_lints/src/matches/try_err.rs b/clippy_lints/src/matches/try_err.rs
index d595f21aa62..af90cb5e673 100644
--- a/clippy_lints/src/matches/try_err.rs
+++ b/clippy_lints/src/matches/try_err.rs
@@ -28,25 +28,15 @@ pub(super) fn check<'tcx>(cx: &LateContext<'tcx>, expr: &'tcx Expr<'_>, scrutine
         && is_res_lang_ctor(cx, path_res(cx, err_fun), ResultErr)
         && let Some(return_ty) = find_return_type(cx, &expr.kind)
     {
-        let prefix;
-        let suffix;
-        let err_ty;
-
-        if let Some(ty) = result_error_type(cx, return_ty) {
-            prefix = "Err(";
-            suffix = ")";
-            err_ty = ty;
+        let (prefix, suffix, err_ty) = if let Some(ty) = result_error_type(cx, return_ty) {
+            ("Err(", ")", ty)
         } else if let Some(ty) = poll_result_error_type(cx, return_ty) {
-            prefix = "Poll::Ready(Err(";
-            suffix = "))";
-            err_ty = ty;
+            ("Poll::Ready(Err(", "))", ty)
         } else if let Some(ty) = poll_option_result_error_type(cx, return_ty) {
-            prefix = "Poll::Ready(Some(Err(";
-            suffix = ")))";
-            err_ty = ty;
+            ("Poll::Ready(Some(Err(", ")))", ty)
         } else {
             return;
-        }
+        };
 
         span_lint_and_then(
             cx,