mirror of
https://github.com/rust-lang/rust-clippy
synced 2024-12-23 03:23:33 +00:00
38d4ac7cea
Discussion previously happened in https://github.com/rust-lang/rust/pull/43498
28 lines
1.1 KiB
Text
28 lines
1.1 KiB
Text
error: avoid using `collect()` when not needed
|
|
--> $DIR/needless_collect.rs:7:28
|
|
|
|
|
LL | let len = sample.iter().collect::<Vec<_>>().len();
|
|
| ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace with: `.count()`
|
|
|
|
|
= note: `-D clippy::needless-collect` implied by `-D warnings`
|
|
|
|
error: avoid using `collect()` when not needed
|
|
--> $DIR/needless_collect.rs:8:21
|
|
|
|
|
LL | if sample.iter().collect::<Vec<_>>().is_empty() {
|
|
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace with: `.next().is_none()`
|
|
|
|
error: avoid using `collect()` when not needed
|
|
--> $DIR/needless_collect.rs:11:27
|
|
|
|
|
LL | sample.iter().cloned().collect::<Vec<_>>().contains(&1);
|
|
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace with: `.any(|&x| x == 1)`
|
|
|
|
error: avoid using `collect()` when not needed
|
|
--> $DIR/needless_collect.rs:12:34
|
|
|
|
|
LL | sample.iter().map(|x| (x, x)).collect::<HashMap<_, _>>().len();
|
|
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace with: `.count()`
|
|
|
|
error: aborting due to 4 previous errors
|
|
|