match_bool: fix suggestion if guard is present

[samueltardieu]

Without this check, the lint would suggest that

    match test {
        true if option == 5 => 10,
        _ => 1,
    };

is replaced by if test { 10 } else { 1 }.

changelog: [match_bool]: omit suggestion when guards are present in match expression

[rustbot]

r? @llogiq

rustbot has assigned @llogiq.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

[llogiq]

Wouldn't it be better to && the guard onto the scrutinee? E.g. in your example, suggest if test && option == 5 { 10 } else { 1 }?

[samueltardieu]

Wouldn't it be better to && the guard onto the scrutinee? E.g. in your example, suggest if test && option == 5 { 10 } else { 1 }?

Indeed, done.

I also enabled rustfix on the ui test.

[llogiq]

Thanks!