error: `if _ { .. } else { .. }` is an expression
- --> $DIR/let_if_seq.rs:57:5
- |
-57 | let mut foo = 0;
- | _____^ starting here...
-58 | |
-59 | |
-60 | |
-61 | | if f() {
-62 | | foo = 42;
-63 | | }
- | |_____^ ...ending here
+ --> $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;
- | _____^ starting here...
-66 | |
-67 | |
-68 | |
-... |
-74 | | f();
-75 | | }
- | |_____^ ...ending here
+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;
- | _____^ starting here...
-78 | |
-79 | |
-80 | |
-... |
-85 | | quz = 0;
-86 | | }
- | |_____^ ...ending here
+ --> $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;
- | _____^ starting here...
-112 | |
-113 | |
-114 | |
-115 | | if f() {
-116 | | baz = 42;
-117 | | }
- | |_____^ ...ending here
- |
-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