diff options
| author | bors <bors@rust-lang.org> | 2023-07-03 14:18:32 +0000 |
|---|---|---|
| committer | bors <bors@rust-lang.org> | 2023-07-03 14:18:32 +0000 |
| commit | 3f17c5c3884c34c481aab426ee7feb370557a589 (patch) | |
| tree | f57342c06d0e3957c52a47ebd7692d5977652139 /compiler/rustc_codegen_llvm/src/errors.rs | |
| parent | a9590617637cc53d759b4738ea1151d7b2737171 (diff) | |
| parent | d80581c7d2db05a899e3363c8fc937012311de69 (diff) | |
| download | rust-3f17c5c3884c34c481aab426ee7feb370557a589.tar.gz rust-3f17c5c3884c34c481aab426ee7feb370557a589.zip | |
Auto merge of #10924 - est31:manual_let_else_question_mark, r=Centri3,flip1995,Manishearth
Don't lint manual_let_else in cases where ? would work
Don't lint `manual_let_else` where the question mark operator `?` would be sufficient, that is, mostly in cases like:
```Rust
let v = if let Some(v) = ex { v } else { return None };
```
Also, this PR emits the `question_mark` lint for `let...else` patterns that could be written with `?` (also, only `return None` like cases).
```
changelog: [`manual_let_else`]: don't lint in cases where question_mark already lints
changelog: [`question_mark`]: lint for `let Some(...) = ex else { return None };`
```
Fixes #8755
Diffstat (limited to 'compiler/rustc_codegen_llvm/src/errors.rs')
0 files changed, 0 insertions, 0 deletions
