`assertions_on_result_states` fix suggestion when `assert!` not in a statement

[rust.git] / tests / ui / map_clone.stderr

diff --git a/tests/ui/map_clone.stderr b/tests/ui/map_clone.stderr
index afad65b007101b06ccd045b746455873e9a30102..d84a5bf8d4de6f3bc3543b4861543e19a92264f9 100644 (file)
--- a/tests/ui/map_clone.stderr
+++ b/tests/ui/map_clone.stderr
@@ -1,102 +1,40 @@
-error: you seem to be using .map() to clone the contents of an iterator, consider using `.cloned()`
-  --> $DIR/map_clone.rs:12:5
+error: you are using an explicit closure for copying elements
+  --> $DIR/map_clone.rs:11:22
    |
-12 |     x.iter().map(|y| y.clone());
-   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^
+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: try
-           x.iter().cloned()
 
-error: you seem to be using .map() to clone the contents of an iterator, consider using `.cloned()`
-  --> $DIR/map_clone.rs:14:5
+error: you are using an explicit closure for cloning elements
+  --> $DIR/map_clone.rs:12:26
    |
-14 |     x.iter().map(|&y| y);
-   |     ^^^^^^^^^^^^^^^^^^^^
-   |
-   = help: try
-           x.iter().cloned()
-
-error: you seem to be using .map() to clone the contents of an iterator, consider using `.cloned()`
-  --> $DIR/map_clone.rs:16:5
-   |
-16 |     x.iter().map(|y| *y);
-   |     ^^^^^^^^^^^^^^^^^^^^
-   |
-   = help: try
-           x.iter().cloned()
-
-error: you seem to be using .map() to clone the contents of an iterator, consider using `.cloned()`
-  --> $DIR/map_clone.rs:18:5
-   |
-18 |     x.iter().map(|y| { y.clone() });
-   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-   |
-   = help: try
-           x.iter().cloned()
-
-error: you seem to be using .map() to clone the contents of an iterator, consider using `.cloned()`
-  --> $DIR/map_clone.rs:20:5
-   |
-20 |     x.iter().map(|&y| { y });
-   |     ^^^^^^^^^^^^^^^^^^^^^^^^
-   |
-   = help: try
-           x.iter().cloned()
+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 seem to be using .map() to clone the contents of an iterator, consider using `.cloned()`
-  --> $DIR/map_clone.rs:22:5
+error: you are using an explicit closure for copying elements
+  --> $DIR/map_clone.rs:13:23
    |
-22 |     x.iter().map(|y| { *y });
-   |     ^^^^^^^^^^^^^^^^^^^^^^^^
-   |
-   = help: try
-           x.iter().cloned()
-
-error: you seem to be using .map() to clone the contents of an iterator, consider using `.cloned()`
-  --> $DIR/map_clone.rs:24:5
-   |
-24 |     x.iter().map(Clone::clone);
-   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^
-   |
-   = help: try
-           x.iter().cloned()
-
-error: you seem to be using .map() to clone the contents of an Option, consider using `.cloned()`
-  --> $DIR/map_clone.rs:30:5
-   |
-30 |     x.as_ref().map(|y| y.clone());
-   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-   |
-   = help: try
-           x.as_ref().cloned()
+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 seem to be using .map() to clone the contents of an Option, consider using `.cloned()`
-  --> $DIR/map_clone.rs:32:5
-   |
-32 |     x.as_ref().map(|&y| y);
-   |     ^^^^^^^^^^^^^^^^^^^^^^
+error: you are using an explicit closure for copying elements
+  --> $DIR/map_clone.rs:15:26
    |
-   = help: try
-           x.as_ref().cloned()
+LL |     let _: Option<u64> = Some(&16).map(|b| *b);
+   |                          ^^^^^^^^^^^^^^^^^^^^^ help: consider calling the dedicated `copied` method: `Some(&16).copied()`
 
-error: you seem to be using .map() to clone the contents of an Option, consider using `.cloned()`
-  --> $DIR/map_clone.rs:34:5
+error: you are using an explicit closure for copying elements
+  --> $DIR/map_clone.rs:16:25
    |
-34 |     x.as_ref().map(|y| *y);
-   |     ^^^^^^^^^^^^^^^^^^^^^^
-   |
-   = help: try
-           x.as_ref().cloned()
+LL |     let _: Option<u8> = Some(&1).map(|x| x.clone());
+   |                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider calling the dedicated `copied` method: `Some(&1).copied()`
 
-error: you seem to be using .map() to clone the contents of an Option, consider using `.cloned()`
-  --> $DIR/map_clone.rs:90:35
-   |
-90 |     let _: Option<UnusualDeref> = x.as_ref().map(|y| *y);
-   |                                   ^^^^^^^^^^^^^^^^^^^^^^
+error: you are needlessly cloning iterator elements
+  --> $DIR/map_clone.rs:27:29
    |
-   = help: try
-           x.as_ref().cloned()
+LL |     let _ = std::env::args().map(|v| v.clone());
+   |                             ^^^^^^^^^^^^^^^^^^^ help: remove the `map` call
 
-error: aborting due to 11 previous errors
+error: aborting due to 6 previous errors