diff options
author | bors[bot] <26634292+bors[bot]@users.noreply.github.com> | 2020-04-10 14:52:25 +0100 |
---|---|---|
committer | GitHub <[email protected]> | 2020-04-10 14:52:25 +0100 |
commit | 38e0d0f3348900bb4078deb09d48c5b6483d43d6 (patch) | |
tree | 6076dca34f94c198541850c8a3ce39c2e5173c89 /crates/ra_hir | |
parent | 0a891d19ae6740c13ddbf8dfaf8b4703d5ca821b (diff) | |
parent | e63315b8f189396cf556f21d4ca27ae4281d17d7 (diff) |
Merge #3905
3905: add ellipsis field to hir pat record r=matklad a=JoshMcguigan
This PR corrects a `fixme`, adding an `ellipsis` field to the hir `Pat::Record` type. It will also be unlock some useful follow on work for #3894.
Additionally it adds a diagnostic for missing fields in record patterns.
~~Marking as a draft because I don't have any tests, and a small amount of manual testing on my branch from #3894 suggests it might *not* be working. Any thoughts on how I can best test this, or else pointers on where I might be going wrong?~~
Co-authored-by: Josh Mcguigan <[email protected]>
Diffstat (limited to 'crates/ra_hir')
-rw-r--r-- | crates/ra_hir/src/source_analyzer.rs | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/crates/ra_hir/src/source_analyzer.rs b/crates/ra_hir/src/source_analyzer.rs index 45631f8fd..226fb4534 100644 --- a/crates/ra_hir/src/source_analyzer.rs +++ b/crates/ra_hir/src/source_analyzer.rs | |||
@@ -255,7 +255,7 @@ impl SourceAnalyzer { | |||
255 | _ => return None, | 255 | _ => return None, |
256 | }; | 256 | }; |
257 | 257 | ||
258 | let (variant, missing_fields) = | 258 | let (variant, missing_fields, _exhaustive) = |
259 | record_pattern_missing_fields(db, infer, pat_id, &body[pat_id])?; | 259 | record_pattern_missing_fields(db, infer, pat_id, &body[pat_id])?; |
260 | let res = self.missing_fields(db, krate, substs, variant, missing_fields); | 260 | let res = self.missing_fields(db, krate, substs, variant, missing_fields); |
261 | Some(res) | 261 | Some(res) |