`match` with default branch that would better be `let-else`.

[ltratt]

`match` with default branch that would better be `let-else`.

[y21]

This lint already exists: https://rust-lang.github.io/rust-clippy/master/index.html#/manual_let_else

warning: this could be rewritten as `let...else`
  --> src/main.rs:10:3
   |
10 | /   let z = match x {
11 | |     Y(z) => z,
12 | |     _ => todo!()
13 | |   };
   | |____^ help: consider writing: `let Y(z) = x else { todo!() };`
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_let_else

Since it's in the pedantic category, it's allowed by default and needs to be manually enabled.

[ltratt]

You're quite right! I've checked and this does fire on most (though not all, for reasons I haven't yet understood) instances of the idiom in our codebase. FWIW this lint doesn't feel pedantic to me, but maybe I'm biased.

[GuillaumeGomez]

Then please open another issue to change the category. I think it should be in the style passes. Anyway, better to have an issue with a title related to what you suggest so you can attract more commenters. ;)