change based on review

Author: DevAccentor

tests/ui/for_loops_over_fallibles.rs

@@ -21,12 +21,12 @@ fn for_loops_over_fallibles() {
     }
 
     // check over a `Result`
-    for x in result.into_iter() {
+    for x in result.iter_mut() {
         println!("{}", x);
     }
 
     // check over a `Result`
-    for x in result.iter_mut() {
+    for x in result.into_iter() {
         println!("{}", x);
     }
 

tests/ui/for_loops_over_fallibles.stderr

@@ -26,18 +26,18 @@ LL |     for x in result {
 error: for loop over `result`, which is a `Result`. This is more readably written as an `if let` statement
   --> $DIR/for_loops_over_fallibles.rs:24:14
    |
-LL |     for x in result.into_iter() {
+LL |     for x in result.iter_mut() {
    |              ^^^^^^
    |
-   = help: consider replacing `for x in result.into_iter()` with `if let Ok(x) = result`
+   = help: consider replacing `for x in result.iter_mut()` with `if let Ok(x) = result`
 
 error: for loop over `result`, which is a `Result`. This is more readably written as an `if let` statement
   --> $DIR/for_loops_over_fallibles.rs:29:14
    |
-LL |     for x in result.iter_mut() {
+LL |     for x in result.into_iter() {
    |              ^^^^^^
    |
-   = help: consider replacing `for x in result.iter_mut()` with `if let Ok(x) = result`
+   = help: consider replacing `for x in result.into_iter()` with `if let Ok(x) = result`
 
 error: for loop over `option.ok_or("x not found")`, which is a `Result`. This is more readably written as an `if let` statement
   --> $DIR/for_loops_over_fallibles.rs:33:14