A new lint for shared code in if blocks

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

diff --git a/tests/ui/let_if_seq.stderr b/tests/ui/let_if_seq.stderr
index 65139c9ba92de90c42b6a3e2c33fb68d6986a2df..9cf2e10a5ee56940e5a06d103a6bcd01323dbf05 100644 (file)
--- a/tests/ui/let_if_seq.stderr
+++ b/tests/ui/let_if_seq.stderr
@@ -1,70 +1,50 @@
 error: `if _ { .. } else { .. }` is an expression
-  --> $DIR/let_if_seq.rs:57:5
-   |
-57 | /     let mut foo = 0;
-58 | |
-59 | |
-60 | |
-61 | |     if f() {
-62 | |         foo = 42;
-63 | |     }
-   | |_____^
+  --> $DIR/let_if_seq.rs:65:5
    |
-note: lint level defined here
-  --> $DIR/let_if_seq.rs:5:9
+LL | /     let mut foo = 0;
+LL | |     if f() {
+LL | |         foo = 42;
+LL | |     }
+   | |_____^ help: it is more idiomatic to write: `let <mut> foo = if f() { 42 } else { 0 };`
    |
-5  | #![deny(useless_let_if_seq)]
-   |         ^^^^^^^^^^^^^^^^^^
-help: it is more idiomatic to write
-   |     let <mut> foo = if f() { 42 } else { 0 };
+   = note: `-D clippy::useless-let-if-seq` implied by `-D warnings`
    = note: you might not need `mut` at all
 
 error: `if _ { .. } else { .. }` is an expression
-  --> $DIR/let_if_seq.rs:65:5
+  --> $DIR/let_if_seq.rs:70:5
    |
-65 | /     let mut bar = 0;
-66 | |
-67 | |
-68 | |
-...  |
-74 | |         f();
-75 | |     }
-   | |_____^
+LL | /     let mut bar = 0;
+LL | |     if f() {
+LL | |         f();
+LL | |         bar = 42;
+LL | |     } else {
+LL | |         f();
+LL | |     }
+   | |_____^ help: it is more idiomatic to write: `let <mut> bar = if f() { ..; 42 } else { ..; 0 };`
    |
-help: it is more idiomatic to write
-   |     let <mut> bar = if f() { ..; 42 } else { ..; 0 };
    = note: you might not need `mut` at all
 
 error: `if _ { .. } else { .. }` is an expression
-  --> $DIR/let_if_seq.rs:77:5
-   |
-77 | /     let quz;
-78 | |
-79 | |
-80 | |
-...  |
-85 | |         quz = 0;
-86 | |     }
-   | |_____^
+  --> $DIR/let_if_seq.rs:78:5
    |
-help: it is more idiomatic to write
-   |     let quz = if f() { 42 } else { 0 };
+LL | /     let quz;
+LL | |     if f() {
+LL | |         quz = 42;
+LL | |     } else {
+LL | |         quz = 0;
+LL | |     }
+   | |_____^ help: it is more idiomatic to write: `let quz = if f() { 42 } else { 0 };`
 
 error: `if _ { .. } else { .. }` is an expression
-   --> $DIR/let_if_seq.rs:111:5
-    |
-111 | /     let mut baz = 0;
-112 | |
-113 | |
-114 | |
-115 | |     if f() {
-116 | |         baz = 42;
-117 | |     }
-    | |_____^
-    |
-help: it is more idiomatic to write
-    |     let <mut> baz = if f() { 42 } else { 0 };
-    = note: you might not need `mut` at all
+  --> $DIR/let_if_seq.rs:107:5
+   |
+LL | /     let mut baz = 0;
+LL | |     if f() {
+LL | |         baz = 42;
+LL | |     }
+   | |_____^ help: it is more idiomatic to write: `let <mut> baz = if f() { 42 } else { 0 };`
+   |
+   = note: you might not need `mut` at all
 
 error: aborting due to 4 previous errors