From 80e84e00010ee8a1c924619e6834d4a51eaf0f34 Mon Sep 17 00:00:00 2001 From: Jakub Wieczorek Date: Fri, 30 May 2014 17:38:15 +0200 Subject: [PATCH] Use RHS's struct def ID for error messages in pattern matching Fixes #14541 --- src/librustc/middle/typeck/check/_match.rs | 10 ++++------ src/test/compile-fail/issue-14541.rs | 20 ++++++++++++++++++++ 2 files changed, 24 insertions(+), 6 deletions(-) create mode 100644 src/test/compile-fail/issue-14541.rs diff --git a/src/librustc/middle/typeck/check/_match.rs b/src/librustc/middle/typeck/check/_match.rs index 072b4ea7fe1..36606ae816e 100644 --- a/src/librustc/middle/typeck/check/_match.rs +++ b/src/librustc/middle/typeck/check/_match.rs @@ -298,9 +298,8 @@ pub fn check_pat_variant(pcx: &pat_ctxt, pat: &ast::Pat, path: &ast::Path, /// `etc` is true if the pattern said '...' and false otherwise. pub fn check_struct_pat_fields(pcx: &pat_ctxt, span: Span, - path: &ast::Path, fields: &[ast::FieldPat], - class_fields: Vec , + class_fields: Vec, class_id: ast::DefId, substitutions: &ty::substs, etc: bool) { @@ -333,13 +332,12 @@ pub fn check_struct_pat_fields(pcx: &pat_ctxt, found_fields.insert(index); } None => { - let name = pprust::path_to_str(path); // Check the pattern anyway, so that attempts to look // up its type won't fail check_pat(pcx, field.pat, ty::mk_err()); tcx.sess.span_err(span, format!("struct `{}` does not have a field named `{}`", - name, + ty::item_path_str(tcx, class_id), token::get_ident(field.ident)).as_slice()); } } @@ -390,7 +388,7 @@ pub fn check_struct_pat(pcx: &pat_ctxt, pat_id: ast::NodeId, span: Span, } } - check_struct_pat_fields(pcx, span, path, fields, class_fields, struct_id, + check_struct_pat_fields(pcx, span, fields, class_fields, struct_id, substitutions, etc); } @@ -413,7 +411,7 @@ pub fn check_struct_like_enum_variant_pat(pcx: &pat_ctxt, // Get the struct fields from this struct-like enum variant. let class_fields = ty::lookup_struct_fields(tcx, variant_id); - check_struct_pat_fields(pcx, span, path, fields, class_fields, + check_struct_pat_fields(pcx, span, fields, class_fields, variant_id, substitutions, etc); } Some(&ast::DefStruct(..)) | Some(&ast::DefVariant(..)) => { diff --git a/src/test/compile-fail/issue-14541.rs b/src/test/compile-fail/issue-14541.rs new file mode 100644 index 00000000000..2dcfeac513c --- /dev/null +++ b/src/test/compile-fail/issue-14541.rs @@ -0,0 +1,20 @@ +// Copyright 2014 The Rust Project Developers. See the COPYRIGHT +// file at the top-level directory of this distribution and at +// http://rust-lang.org/COPYRIGHT. +// +// Licensed under the Apache License, Version 2.0 or the MIT license +// , at your +// option. This file may not be copied, modified, or distributed +// except according to those terms. + +struct vec2 { y: f32 } +struct vec3 { y: f32, z: f32 } + +fn make(v: vec2) { + let vec3 { y: _, z: _ } = v; + //~^ ERROR mismatched types: expected `vec2` but found `vec3` + //~^^ ERROR struct `vec2` does not have a field named `z` +} + +fn main() { } \ No newline at end of file