PR Followups

Author: andrewpollack

tests/ui/while_let_some_result.fixed

@@ -31,6 +31,10 @@ fn base_2(x: i32) {
     }
 }
 
+fn base_3(test_func: Box<Result<i32, &str>>) {
+    while let Some(b) = test_func.ok() {}
+}
+
 fn main() {
     let _ = base_1(1);
     let _ = base_2(2);

tests/ui/while_let_some_result.rs

@@ -31,6 +31,10 @@ fn base_2(x: i32) {
     }
 }
 
+fn base_3(test_func: Box<Result<i32, &str>>) {
+    while let Some(b) = test_func.ok() {}
+}
+
 fn main() {
     let _ = base_1(1);
     let _ = base_2(2);

tests/ui/while_let_some_result.stderr

@@ -10,5 +10,13 @@ help: consider matching on `Ok(a)` and removing the call to `ok` instead
 LL |     while let Ok(a) = wat.next() {
    |     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 
-error: aborting due to previous error
+error: local variable doesn't need to be boxed here
+  --> $DIR/while_let_some_result.rs:34:11
+   |
+LL | fn base_3(test_func: Box<Result<i32, &str>>) {
+   |           ^^^^^^^^^
+   |
+   = note: `-D clippy::boxed-local` implied by `-D warnings`
+
+error: aborting due to 2 previous errors