diff options
author | bors[bot] <26634292+bors[bot]@users.noreply.github.com> | 2020-07-09 16:34:43 +0100 |
---|---|---|
committer | GitHub <[email protected]> | 2020-07-09 16:34:43 +0100 |
commit | 89c7c559953f103f224caacad6c9497d36660c76 (patch) | |
tree | fff7cbf2b489053af801e075c51288d7d7d6afca /crates/ra_ide/src/diagnostics.rs | |
parent | f1d084fbd9504497aed24b907b03247bfcd31675 (diff) | |
parent | f4a9d9a00f0462bce92ddbac24cb91825c8ab192 (diff) |
Merge #5270
5270: Add argument count mismatch diagnostic r=matklad a=jonas-schievink
Closes https://github.com/rust-analyzer/rust-analyzer/issues/4025.
This currently has one false positive on this line, where `max` is resolved to `Iterator::max` instead of `Ord::max`:
https://github.com/rust-analyzer/rust-analyzer/blob/8aa10c00a4c5b957d459fac5a103cd9688e8dcdd/crates/expect/src/lib.rs#L263
(I have no idea why it thinks that `usize` is an `Iterator`)
TODO:
* [x] Tests
* [x] Improve diagnostic text for method calls
Co-authored-by: Jonas Schievink <[email protected]>
Diffstat (limited to 'crates/ra_ide/src/diagnostics.rs')
-rw-r--r-- | crates/ra_ide/src/diagnostics.rs | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/crates/ra_ide/src/diagnostics.rs b/crates/ra_ide/src/diagnostics.rs index 00f6bb186..e69e9b4ec 100644 --- a/crates/ra_ide/src/diagnostics.rs +++ b/crates/ra_ide/src/diagnostics.rs | |||
@@ -99,14 +99,6 @@ pub(crate) fn diagnostics(db: &RootDatabase, file_id: FileId) -> Vec<Diagnostic> | |||
99 | fix, | 99 | fix, |
100 | }) | 100 | }) |
101 | }) | 101 | }) |
102 | .on::<hir::diagnostics::MissingMatchArms, _>(|d| { | ||
103 | res.borrow_mut().push(Diagnostic { | ||
104 | range: sema.diagnostics_range(d).range, | ||
105 | message: d.message(), | ||
106 | severity: Severity::Error, | ||
107 | fix: None, | ||
108 | }) | ||
109 | }) | ||
110 | .on::<hir::diagnostics::MissingOkInTailExpr, _>(|d| { | 102 | .on::<hir::diagnostics::MissingOkInTailExpr, _>(|d| { |
111 | let node = d.ast(db); | 103 | let node = d.ast(db); |
112 | let replacement = format!("Ok({})", node.syntax()); | 104 | let replacement = format!("Ok({})", node.syntax()); |