error: `if _ { .. } else { .. }` is an expression --> $DIR/let_if_seq.rs:66:5 | LL | / let mut foo = 0; LL | | LL | | LL | | if f() { LL | | foo = 42; LL | | } | |_____^ help: it is more idiomatic to write: `let foo = if f() { 42 } else { 0 };` | = note: you might not need `mut` at all = note: `-D clippy::useless-let-if-seq` implied by `-D warnings` = help: to override `-D warnings` add `#[allow(clippy::useless_let_if_seq)]` error: `if _ { .. } else { .. }` is an expression --> $DIR/let_if_seq.rs:73:5 | LL | / let mut bar = 0; LL | | LL | | LL | | if f() { ... | LL | | f(); LL | | } | |_____^ help: it is more idiomatic to write: `let 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:83:5 | LL | / let quz; LL | | 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:113:5 | LL | / let mut baz = 0; LL | | LL | | LL | | if f() { LL | | baz = 42; LL | | } | |_____^ help: it is more idiomatic to write: `let baz = if f() { 42 } else { 0 };` | = note: you might not need `mut` at all error: aborting due to 4 previous errors