2019-09-30 08:58:53 +00:00
|
|
|
//! FIXME: write short doc here
|
|
|
|
|
2019-11-12 15:46:57 +00:00
|
|
|
use std::sync::Arc;
|
2019-01-05 15:32:07 +00:00
|
|
|
|
2020-04-07 15:09:02 +00:00
|
|
|
use hir_def::{path::path, resolver::HasResolver, AdtId, FunctionId};
|
|
|
|
use hir_expand::diagnostics::DiagnosticSink;
|
2020-04-07 15:58:05 +00:00
|
|
|
use ra_syntax::{ast, AstPtr};
|
2019-11-15 11:53:09 +00:00
|
|
|
use rustc_hash::FxHashSet;
|
2019-01-05 15:32:07 +00:00
|
|
|
|
2019-11-15 11:53:09 +00:00
|
|
|
use crate::{
|
|
|
|
db::HirDatabase,
|
2020-04-09 03:23:51 +00:00
|
|
|
diagnostics::{MissingFields, MissingMatchArms, MissingOkInTailExpr, MissingPatFields},
|
2020-02-19 16:53:32 +00:00
|
|
|
utils::variant_data,
|
2019-11-27 14:46:02 +00:00
|
|
|
ApplicationTy, InferenceResult, Ty, TypeCtor,
|
2020-03-24 11:40:58 +00:00
|
|
|
_match::{is_useful, MatchCheckCtx, Matrix, PatStack, Usefulness},
|
2019-11-15 11:53:09 +00:00
|
|
|
};
|
2019-01-05 15:32:07 +00:00
|
|
|
|
2019-11-12 15:46:57 +00:00
|
|
|
pub use hir_def::{
|
2019-11-14 08:56:13 +00:00
|
|
|
body::{
|
|
|
|
scope::{ExprScopes, ScopeEntry, ScopeId},
|
|
|
|
Body, BodySourceMap, ExprPtr, ExprSource, PatPtr, PatSource,
|
|
|
|
},
|
2019-11-12 15:46:57 +00:00
|
|
|
expr::{
|
|
|
|
ArithOp, Array, BinaryOp, BindingAnnotation, CmpOp, Expr, ExprId, Literal, LogicOp,
|
|
|
|
MatchArm, Ordering, Pat, PatId, RecordFieldPat, RecordLitField, Statement, UnaryOp,
|
|
|
|
},
|
2020-04-25 12:23:34 +00:00
|
|
|
LocalFieldId, VariantId,
|
2019-11-12 12:09:25 +00:00
|
|
|
};
|
2019-09-03 05:56:36 +00:00
|
|
|
|
2019-11-27 14:46:02 +00:00
|
|
|
pub struct ExprValidator<'a, 'b: 'a> {
|
|
|
|
func: FunctionId,
|
2019-11-15 11:53:09 +00:00
|
|
|
infer: Arc<InferenceResult>,
|
|
|
|
sink: &'a mut DiagnosticSink<'b>,
|
|
|
|
}
|
|
|
|
|
|
|
|
impl<'a, 'b> ExprValidator<'a, 'b> {
|
2019-11-27 14:46:02 +00:00
|
|
|
pub fn new(
|
|
|
|
func: FunctionId,
|
2019-11-15 11:53:09 +00:00
|
|
|
infer: Arc<InferenceResult>,
|
|
|
|
sink: &'a mut DiagnosticSink<'b>,
|
|
|
|
) -> ExprValidator<'a, 'b> {
|
|
|
|
ExprValidator { func, infer, sink }
|
|
|
|
}
|
|
|
|
|
2020-03-13 15:05:46 +00:00
|
|
|
pub fn validate_body(&mut self, db: &dyn HirDatabase) {
|
2019-11-27 14:46:02 +00:00
|
|
|
let body = db.body(self.func.into());
|
2019-11-15 11:53:09 +00:00
|
|
|
|
2020-04-07 15:09:02 +00:00
|
|
|
for (id, expr) in body.exprs.iter() {
|
|
|
|
if let Some((variant_def, missed_fields, true)) =
|
|
|
|
record_literal_missing_fields(db, &self.infer, id, expr)
|
|
|
|
{
|
2020-04-09 03:23:51 +00:00
|
|
|
self.create_record_literal_missing_fields_diagnostic(
|
|
|
|
id,
|
|
|
|
db,
|
|
|
|
variant_def,
|
|
|
|
missed_fields,
|
|
|
|
);
|
2020-04-07 15:09:02 +00:00
|
|
|
}
|
|
|
|
if let Expr::Match { expr, arms } = expr {
|
2020-03-24 11:40:58 +00:00
|
|
|
self.validate_match(id, *expr, arms, db, self.infer.clone());
|
2019-11-15 11:53:09 +00:00
|
|
|
}
|
|
|
|
}
|
2020-04-09 03:23:51 +00:00
|
|
|
for (id, pat) in body.pats.iter() {
|
|
|
|
if let Some((variant_def, missed_fields, true)) =
|
|
|
|
record_pattern_missing_fields(db, &self.infer, id, pat)
|
|
|
|
{
|
|
|
|
self.create_record_pattern_missing_fields_diagnostic(
|
|
|
|
id,
|
|
|
|
db,
|
|
|
|
variant_def,
|
|
|
|
missed_fields,
|
|
|
|
);
|
|
|
|
}
|
|
|
|
}
|
2019-11-24 15:48:29 +00:00
|
|
|
let body_expr = &body[body.body_expr];
|
2020-03-24 11:40:58 +00:00
|
|
|
if let Expr::Block { tail: Some(t), .. } = body_expr {
|
2019-11-24 15:48:29 +00:00
|
|
|
self.validate_results_in_tail_expr(body.body_expr, *t, db);
|
2019-11-15 11:53:09 +00:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2020-04-09 03:23:51 +00:00
|
|
|
fn create_record_literal_missing_fields_diagnostic(
|
|
|
|
&mut self,
|
|
|
|
id: ExprId,
|
|
|
|
db: &dyn HirDatabase,
|
|
|
|
variant_def: VariantId,
|
2020-04-25 12:23:34 +00:00
|
|
|
missed_fields: Vec<LocalFieldId>,
|
2020-04-09 03:23:51 +00:00
|
|
|
) {
|
|
|
|
// XXX: only look at source_map if we do have missing fields
|
|
|
|
let (_, source_map) = db.body_with_source_map(self.func.into());
|
|
|
|
|
|
|
|
if let Ok(source_ptr) = source_map.expr_syntax(id) {
|
2020-04-11 17:25:33 +00:00
|
|
|
let root = source_ptr.file_syntax(db.upcast());
|
|
|
|
if let ast::Expr::RecordLit(record_lit) = &source_ptr.value.to_node(&root) {
|
|
|
|
if let Some(field_list) = record_lit.record_field_list() {
|
|
|
|
let variant_data = variant_data(db.upcast(), variant_def);
|
|
|
|
let missed_fields = missed_fields
|
|
|
|
.into_iter()
|
|
|
|
.map(|idx| variant_data.fields()[idx].name.clone())
|
|
|
|
.collect();
|
|
|
|
self.sink.push(MissingFields {
|
|
|
|
file: source_ptr.file_id,
|
|
|
|
field_list: AstPtr::new(&field_list),
|
|
|
|
missed_fields,
|
|
|
|
})
|
2020-04-09 03:23:51 +00:00
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
fn create_record_pattern_missing_fields_diagnostic(
|
|
|
|
&mut self,
|
|
|
|
id: PatId,
|
|
|
|
db: &dyn HirDatabase,
|
|
|
|
variant_def: VariantId,
|
2020-04-25 12:23:34 +00:00
|
|
|
missed_fields: Vec<LocalFieldId>,
|
2020-04-09 03:23:51 +00:00
|
|
|
) {
|
|
|
|
// XXX: only look at source_map if we do have missing fields
|
|
|
|
let (_, source_map) = db.body_with_source_map(self.func.into());
|
|
|
|
|
|
|
|
if let Ok(source_ptr) = source_map.pat_syntax(id) {
|
2020-04-10 22:27:00 +00:00
|
|
|
if let Some(expr) = source_ptr.value.as_ref().left() {
|
2020-04-09 03:23:51 +00:00
|
|
|
let root = source_ptr.file_syntax(db.upcast());
|
|
|
|
if let ast::Pat::RecordPat(record_pat) = expr.to_node(&root) {
|
|
|
|
if let Some(field_list) = record_pat.record_field_pat_list() {
|
|
|
|
let variant_data = variant_data(db.upcast(), variant_def);
|
|
|
|
let missed_fields = missed_fields
|
|
|
|
.into_iter()
|
|
|
|
.map(|idx| variant_data.fields()[idx].name.clone())
|
|
|
|
.collect();
|
|
|
|
self.sink.push(MissingPatFields {
|
|
|
|
file: source_ptr.file_id,
|
|
|
|
field_list: AstPtr::new(&field_list),
|
|
|
|
missed_fields,
|
|
|
|
})
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2020-03-24 11:40:58 +00:00
|
|
|
fn validate_match(
|
|
|
|
&mut self,
|
|
|
|
id: ExprId,
|
2020-04-05 17:16:34 +00:00
|
|
|
match_expr: ExprId,
|
2020-03-24 11:40:58 +00:00
|
|
|
arms: &[MatchArm],
|
|
|
|
db: &dyn HirDatabase,
|
|
|
|
infer: Arc<InferenceResult>,
|
|
|
|
) {
|
|
|
|
let (body, source_map): (Arc<Body>, Arc<BodySourceMap>) =
|
|
|
|
db.body_with_source_map(self.func.into());
|
|
|
|
|
2020-04-05 17:16:34 +00:00
|
|
|
let match_expr_ty = match infer.type_of_expr.get(match_expr) {
|
|
|
|
Some(ty) => ty,
|
|
|
|
// If we can't resolve the type of the match expression
|
|
|
|
// we cannot perform exhaustiveness checks.
|
|
|
|
None => return,
|
|
|
|
};
|
2020-03-24 11:40:58 +00:00
|
|
|
|
2020-04-14 23:06:57 +00:00
|
|
|
let cx = MatchCheckCtx { match_expr, body, infer: infer.clone(), db };
|
2020-03-24 11:40:58 +00:00
|
|
|
let pats = arms.iter().map(|arm| arm.pat);
|
|
|
|
|
|
|
|
let mut seen = Matrix::empty();
|
|
|
|
for pat in pats {
|
2020-04-05 17:16:34 +00:00
|
|
|
if let Some(pat_ty) = infer.type_of_pat.get(pat) {
|
|
|
|
// We only include patterns whose type matches the type
|
|
|
|
// of the match expression. If we had a InvalidMatchArmPattern
|
|
|
|
// diagnostic or similar we could raise that in an else
|
|
|
|
// block here.
|
2020-04-05 19:42:24 +00:00
|
|
|
//
|
|
|
|
// When comparing the types, we also have to consider that rustc
|
|
|
|
// will automatically de-reference the match expression type if
|
|
|
|
// necessary.
|
2020-04-07 12:17:59 +00:00
|
|
|
//
|
|
|
|
// FIXME we should use the type checker for this.
|
2020-04-05 19:42:24 +00:00
|
|
|
if pat_ty == match_expr_ty
|
|
|
|
|| match_expr_ty
|
|
|
|
.as_reference()
|
|
|
|
.map(|(match_expr_ty, _)| match_expr_ty == pat_ty)
|
|
|
|
.unwrap_or(false)
|
|
|
|
{
|
2020-04-05 17:16:34 +00:00
|
|
|
// If we had a NotUsefulMatchArm diagnostic, we could
|
|
|
|
// check the usefulness of each pattern as we added it
|
|
|
|
// to the matrix here.
|
|
|
|
let v = PatStack::from_pattern(pat);
|
|
|
|
seen.push(&cx, v);
|
2020-04-11 03:14:53 +00:00
|
|
|
continue;
|
2020-04-05 17:16:34 +00:00
|
|
|
}
|
|
|
|
}
|
2020-04-11 03:14:53 +00:00
|
|
|
|
|
|
|
// If we can't resolve the type of a pattern, or the pattern type doesn't
|
|
|
|
// fit the match expression, we skip this diagnostic. Skipping the entire
|
|
|
|
// diagnostic rather than just not including this match arm is preferred
|
|
|
|
// to avoid the chance of false positives.
|
|
|
|
return;
|
2020-03-24 11:40:58 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
match is_useful(&cx, &seen, &PatStack::from_wild()) {
|
2020-04-05 01:02:27 +00:00
|
|
|
Ok(Usefulness::Useful) => (),
|
2020-03-24 11:40:58 +00:00
|
|
|
// if a wildcard pattern is not useful, then all patterns are covered
|
2020-04-05 01:02:27 +00:00
|
|
|
Ok(Usefulness::NotUseful) => return,
|
|
|
|
// this path is for unimplemented checks, so we err on the side of not
|
|
|
|
// reporting any errors
|
|
|
|
_ => return,
|
2020-03-24 11:40:58 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
if let Ok(source_ptr) = source_map.expr_syntax(id) {
|
2020-04-11 17:25:33 +00:00
|
|
|
let root = source_ptr.file_syntax(db.upcast());
|
|
|
|
if let ast::Expr::MatchExpr(match_expr) = &source_ptr.value.to_node(&root) {
|
|
|
|
if let (Some(match_expr), Some(arms)) =
|
|
|
|
(match_expr.expr(), match_expr.match_arm_list())
|
|
|
|
{
|
|
|
|
self.sink.push(MissingMatchArms {
|
|
|
|
file: source_ptr.file_id,
|
|
|
|
match_expr: AstPtr::new(&match_expr),
|
|
|
|
arms: AstPtr::new(&arms),
|
|
|
|
})
|
2020-03-24 11:40:58 +00:00
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2020-03-13 15:05:46 +00:00
|
|
|
fn validate_results_in_tail_expr(&mut self, body_id: ExprId, id: ExprId, db: &dyn HirDatabase) {
|
2019-11-15 11:53:09 +00:00
|
|
|
// the mismatch will be on the whole block currently
|
|
|
|
let mismatch = match self.infer.type_mismatch_for_expr(body_id) {
|
|
|
|
Some(m) => m,
|
|
|
|
None => return,
|
|
|
|
};
|
|
|
|
|
2019-12-13 21:32:44 +00:00
|
|
|
let std_result_path = path![std::result::Result];
|
2019-11-15 11:53:09 +00:00
|
|
|
|
2020-03-13 15:05:46 +00:00
|
|
|
let resolver = self.func.resolver(db.upcast());
|
|
|
|
let std_result_enum = match resolver.resolve_known_enum(db.upcast(), &std_result_path) {
|
2019-11-15 11:53:09 +00:00
|
|
|
Some(it) => it,
|
|
|
|
_ => return,
|
|
|
|
};
|
|
|
|
|
2019-11-26 11:29:12 +00:00
|
|
|
let std_result_ctor = TypeCtor::Adt(AdtId::EnumId(std_result_enum));
|
2019-11-15 11:53:09 +00:00
|
|
|
let params = match &mismatch.expected {
|
|
|
|
Ty::Apply(ApplicationTy { ctor, parameters }) if ctor == &std_result_ctor => parameters,
|
|
|
|
_ => return,
|
|
|
|
};
|
|
|
|
|
2020-02-18 13:32:19 +00:00
|
|
|
if params.len() == 2 && params[0] == mismatch.actual {
|
2019-11-27 14:46:02 +00:00
|
|
|
let (_, source_map) = db.body_with_source_map(self.func.into());
|
2019-11-15 11:53:09 +00:00
|
|
|
|
2020-03-06 13:44:44 +00:00
|
|
|
if let Ok(source_ptr) = source_map.expr_syntax(id) {
|
2020-04-11 17:25:33 +00:00
|
|
|
self.sink
|
|
|
|
.push(MissingOkInTailExpr { file: source_ptr.file_id, expr: source_ptr.value });
|
2019-11-15 11:53:09 +00:00
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
2020-04-07 15:09:02 +00:00
|
|
|
|
|
|
|
pub fn record_literal_missing_fields(
|
|
|
|
db: &dyn HirDatabase,
|
|
|
|
infer: &InferenceResult,
|
|
|
|
id: ExprId,
|
|
|
|
expr: &Expr,
|
2020-04-25 12:23:34 +00:00
|
|
|
) -> Option<(VariantId, Vec<LocalFieldId>, /*exhaustive*/ bool)> {
|
2020-04-07 15:09:02 +00:00
|
|
|
let (fields, exhausitve) = match expr {
|
|
|
|
Expr::RecordLit { path: _, fields, spread } => (fields, spread.is_none()),
|
|
|
|
_ => return None,
|
|
|
|
};
|
|
|
|
|
|
|
|
let variant_def = infer.variant_resolution_for_expr(id)?;
|
|
|
|
if let VariantId::UnionId(_) = variant_def {
|
|
|
|
return None;
|
|
|
|
}
|
|
|
|
|
|
|
|
let variant_data = variant_data(db.upcast(), variant_def);
|
|
|
|
|
|
|
|
let specified_fields: FxHashSet<_> = fields.iter().map(|f| &f.name).collect();
|
2020-04-25 12:23:34 +00:00
|
|
|
let missed_fields: Vec<LocalFieldId> = variant_data
|
2020-04-07 15:09:02 +00:00
|
|
|
.fields()
|
|
|
|
.iter()
|
|
|
|
.filter_map(|(f, d)| if specified_fields.contains(&d.name) { None } else { Some(f) })
|
|
|
|
.collect();
|
|
|
|
if missed_fields.is_empty() {
|
|
|
|
return None;
|
|
|
|
}
|
|
|
|
Some((variant_def, missed_fields, exhausitve))
|
|
|
|
}
|
|
|
|
|
|
|
|
pub fn record_pattern_missing_fields(
|
|
|
|
db: &dyn HirDatabase,
|
|
|
|
infer: &InferenceResult,
|
|
|
|
id: PatId,
|
|
|
|
pat: &Pat,
|
2020-04-25 12:23:34 +00:00
|
|
|
) -> Option<(VariantId, Vec<LocalFieldId>, /*exhaustive*/ bool)> {
|
2020-04-09 03:23:51 +00:00
|
|
|
let (fields, exhaustive) = match pat {
|
|
|
|
Pat::Record { path: _, args, ellipsis } => (args, !ellipsis),
|
2020-04-07 15:09:02 +00:00
|
|
|
_ => return None,
|
|
|
|
};
|
|
|
|
|
|
|
|
let variant_def = infer.variant_resolution_for_pat(id)?;
|
|
|
|
if let VariantId::UnionId(_) = variant_def {
|
|
|
|
return None;
|
|
|
|
}
|
|
|
|
|
|
|
|
let variant_data = variant_data(db.upcast(), variant_def);
|
|
|
|
|
|
|
|
let specified_fields: FxHashSet<_> = fields.iter().map(|f| &f.name).collect();
|
2020-04-25 12:23:34 +00:00
|
|
|
let missed_fields: Vec<LocalFieldId> = variant_data
|
2020-04-07 15:09:02 +00:00
|
|
|
.fields()
|
|
|
|
.iter()
|
|
|
|
.filter_map(|(f, d)| if specified_fields.contains(&d.name) { None } else { Some(f) })
|
|
|
|
.collect();
|
|
|
|
if missed_fields.is_empty() {
|
|
|
|
return None;
|
|
|
|
}
|
2020-04-09 03:23:51 +00:00
|
|
|
Some((variant_def, missed_fields, exhaustive))
|
2020-04-07 15:09:02 +00:00
|
|
|
}
|