about summary refs log tree commit diff
diff options
context:
space:
mode:
authorlengyijun <sjtu5140809011@gmail.com>2023-08-05 12:39:31 +0800
committerlengyijun <sjtu5140809011@gmail.com>2023-08-05 12:41:20 +0800
commite5b0483c852ed801b18fd1a43afa471f13937a47 (patch)
treef3ec84308ae2aaf2a23d94b20e0e5bc2b7719026
parent5818225a89bfb8645cbf9c10c0d8a10c5b063d71 (diff)
downloadrust-e5b0483c852ed801b18fd1a43afa471f13937a47.tar.gz
rust-e5b0483c852ed801b18fd1a43afa471f13937a47.zip
Small code style adjustments
-rw-r--r--clippy_lints/src/methods/mod.rs18
1 files changed, 6 insertions, 12 deletions
diff --git a/clippy_lints/src/methods/mod.rs b/clippy_lints/src/methods/mod.rs
index 17483656cec..42756b27d01 100644
--- a/clippy_lints/src/methods/mod.rs
+++ b/clippy_lints/src/methods/mod.rs
@@ -4020,10 +4020,8 @@ impl Methods {
                     }
                 },
                 ("last", []) => {
-                    if let Some((name2, recv2, args2, _span2, _)) = method_call(recv) {
-                        if let ("cloned", []) = (name2, args2) {
-                            iter_overeager_cloned::check(cx, expr, recv, recv2, false, false);
-                        }
+                    if let Some(("cloned", recv2, [], _span2, _)) = method_call(recv) {
+                        iter_overeager_cloned::check(cx, expr, recv, recv2, false, false);
                     }
                 },
                 ("lock", []) => {
@@ -4127,10 +4125,8 @@ impl Methods {
                 ("skip", [arg]) => {
                     iter_skip_zero::check(cx, expr, arg);
 
-                    if let Some((name2, recv2, args2, _span2, _)) = method_call(recv) {
-                        if let ("cloned", []) = (name2, args2) {
-                            iter_overeager_cloned::check(cx, expr, recv, recv2, false, false);
-                        }
+                    if let Some(("cloned", recv2, [], _span2, _)) = method_call(recv) {
+                        iter_overeager_cloned::check(cx, expr, recv, recv2, false, false);
                     }
                 }
                 ("sort", []) => {
@@ -4155,10 +4151,8 @@ impl Methods {
                 },
                 ("step_by", [arg]) => iterator_step_by_zero::check(cx, expr, arg),
                 ("take", [_arg]) => {
-                    if let Some((name2, recv2, args2, _span2, _)) = method_call(recv) {
-                        if let ("cloned", []) = (name2, args2) {
-                            iter_overeager_cloned::check(cx, expr, recv, recv2, false, false);
-                        }
+                    if let Some(("cloned", recv2, [], _span2, _)) = method_call(recv) {
+                        iter_overeager_cloned::check(cx, expr, recv, recv2, false, false);
                     }
                 },
                 ("take", []) => needless_option_take::check(cx, expr, recv),