mirror of
https://github.com/rust-lang/rust-clippy
synced 2024-11-30 16:39:26 +00:00
95 lines
4 KiB
Text
95 lines
4 KiB
Text
error: you are using an explicit closure for copying elements
|
|
--> tests/ui/map_clone.rs:14:22
|
|
|
|
|
LL | let _: Vec<i8> = vec![5_i8; 6].iter().map(|x| *x).collect();
|
|
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider calling the dedicated `copied` method: `vec![5_i8; 6].iter().copied()`
|
|
|
|
|
= note: `-D clippy::map-clone` implied by `-D warnings`
|
|
= help: to override `-D warnings` add `#[allow(clippy::map_clone)]`
|
|
|
|
error: you are using an explicit closure for cloning elements
|
|
--> tests/ui/map_clone.rs:15:26
|
|
|
|
|
LL | let _: Vec<String> = vec![String::new()].iter().map(|x| x.clone()).collect();
|
|
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider calling the dedicated `cloned` method: `vec![String::new()].iter().cloned()`
|
|
|
|
error: you are using an explicit closure for copying elements
|
|
--> tests/ui/map_clone.rs:16:23
|
|
|
|
|
LL | let _: Vec<u32> = vec![42, 43].iter().map(|&x| x).collect();
|
|
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider calling the dedicated `copied` method: `vec![42, 43].iter().copied()`
|
|
|
|
error: you are using an explicit closure for copying elements
|
|
--> tests/ui/map_clone.rs:18:26
|
|
|
|
|
LL | let _: Option<u64> = Some(&16).map(|b| *b);
|
|
| ^^^^^^^^^^^^^^^^^^^^^ help: consider calling the dedicated `copied` method: `Some(&16).copied()`
|
|
|
|
error: you are using an explicit closure for copying elements
|
|
--> tests/ui/map_clone.rs:19:25
|
|
|
|
|
LL | let _: Option<u8> = Some(&1).map(|x| x.clone());
|
|
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider calling the dedicated `copied` method: `Some(&1).copied()`
|
|
|
|
error: you are needlessly cloning iterator elements
|
|
--> tests/ui/map_clone.rs:30:29
|
|
|
|
|
LL | let _ = std::env::args().map(|v| v.clone());
|
|
| ^^^^^^^^^^^^^^^^^^^ help: remove the `map` call
|
|
|
|
error: you are explicitly cloning with `.map()`
|
|
--> tests/ui/map_clone.rs:69:13
|
|
|
|
|
LL | let y = x.map(|x| String::clone(x));
|
|
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider calling the dedicated `cloned` method: `x.cloned()`
|
|
|
|
error: you are explicitly cloning with `.map()`
|
|
--> tests/ui/map_clone.rs:71:13
|
|
|
|
|
LL | let y = x.map(Clone::clone);
|
|
| ^^^^^^^^^^^^^^^^^^^ help: consider calling the dedicated `cloned` method: `x.cloned()`
|
|
|
|
error: you are explicitly cloning with `.map()`
|
|
--> tests/ui/map_clone.rs:74:13
|
|
|
|
|
LL | let y = x.map(String::clone);
|
|
| ^^^^^^^^^^^^^^^^^^^^ help: consider calling the dedicated `cloned` method: `x.cloned()`
|
|
|
|
error: you are explicitly cloning with `.map()`
|
|
--> tests/ui/map_clone.rs:80:13
|
|
|
|
|
LL | let y = x.map(|x| u32::clone(x));
|
|
| ^^^^^^^^^^^^^^^^^^^^^^^^ help: consider calling the dedicated `copied` method: `x.copied()`
|
|
|
|
error: you are explicitly cloning with `.map()`
|
|
--> tests/ui/map_clone.rs:83:13
|
|
|
|
|
LL | let y = x.map(|x| Clone::clone(x));
|
|
| ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider calling the dedicated `copied` method: `x.copied()`
|
|
|
|
error: you are explicitly cloning with `.map()`
|
|
--> tests/ui/map_clone.rs:95:13
|
|
|
|
|
LL | let y = x.map(|x| String::clone(x));
|
|
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider calling the dedicated `cloned` method: `x.cloned()`
|
|
|
|
error: you are explicitly cloning with `.map()`
|
|
--> tests/ui/map_clone.rs:98:13
|
|
|
|
|
LL | let y = x.map(|x| Clone::clone(x));
|
|
| ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider calling the dedicated `cloned` method: `x.cloned()`
|
|
|
|
error: you are explicitly cloning with `.map()`
|
|
--> tests/ui/map_clone.rs:104:13
|
|
|
|
|
LL | let y = x.map(|x| u32::clone(x));
|
|
| ^^^^^^^^^^^^^^^^^^^^^^^^ help: consider calling the dedicated `copied` method: `x.copied()`
|
|
|
|
error: you are explicitly cloning with `.map()`
|
|
--> tests/ui/map_clone.rs:107:13
|
|
|
|
|
LL | let y = x.map(|x| Clone::clone(x));
|
|
| ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider calling the dedicated `copied` method: `x.copied()`
|
|
|
|
error: aborting due to 15 previous errors
|
|
|