Fix an invalid suggestion in needless_collect test

[giraffate]

A test, https://github.com/rust-lang/rust-clippy/blob/master/tests/ui/needless_collect_indirect.rs#L11-L12, suggests following codes, but the suggested codes don't work. An example is here: https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=6947d9f2806a83f41cc5eb8e39b09d0b.

error: avoid using `collect()` when not needed
  --> $DIR/needless_collect_indirect.rs:11:5
   |
LL | /     let indirect_contains = sample.iter().collect::<VecDeque<_>>();
LL | |     indirect_contains.contains(&&5);
   | |____^
   |
help: Check if the original Iterator contains an element instead of collecting then checking
   |
LL |     
LL |     sample.iter().any(|x| x == &&5);

changelog: none

[rust-highfive]

r? @phansch

(rust_highfive has picked a reviewer for you, use r? to override)

[flip1995]

Does this fix also work for non-Copy types?

[giraffate]

@flip1995 Thanks for your review. Do you mean this test case (String has no Copy trait. 2f5d418 ) ?

[flip1995]

Yes, perfect. Waiting on rustup

[flip1995]

@bors r+

[bors]

📌 Commit 2f5d418 has been approved by flip1995

[bors]

⌛ Testing commit 2f5d418 with merge 83bb5ec...

[bors]

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: flip1995
Pushing 83bb5ec to master...