]> git.lizzy.rs Git - rust.git/blob - src/librustc/middle/check_match.rs
Auto merge of #31144 - jseyfried:remove_import_ordering_restriction, r=nrc
[rust.git] / src / librustc / middle / check_match.rs
1 // Copyright 2012-2014 The Rust Project Developers. See the COPYRIGHT
2 // file at the top-level directory of this distribution and at
3 // http://rust-lang.org/COPYRIGHT.
4 //
5 // Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
6 // http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
7 // <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
8 // option. This file may not be copied, modified, or distributed
9 // except according to those terms.
10
11 pub use self::Constructor::*;
12 use self::Usefulness::*;
13 use self::WitnessPreference::*;
14
15 use dep_graph::DepNode;
16 use middle::const_eval::{compare_const_vals, ConstVal};
17 use middle::const_eval::{eval_const_expr, eval_const_expr_partial};
18 use middle::const_eval::{const_expr_to_pat, lookup_const_by_id};
19 use middle::const_eval::EvalHint::ExprTypeChecked;
20 use middle::def::*;
21 use middle::def_id::{DefId};
22 use middle::expr_use_visitor::{ConsumeMode, Delegate, ExprUseVisitor};
23 use middle::expr_use_visitor::{LoanCause, MutateMode};
24 use middle::expr_use_visitor as euv;
25 use middle::infer;
26 use middle::mem_categorization::{cmt};
27 use middle::pat_util::*;
28 use middle::ty::*;
29 use middle::ty;
30 use std::cmp::Ordering;
31 use std::fmt;
32 use std::iter::{FromIterator, IntoIterator, repeat};
33
34 use rustc_front::hir;
35 use rustc_front::hir::Pat;
36 use rustc_front::intravisit::{self, Visitor, FnKind};
37 use rustc_front::util as front_util;
38 use rustc_back::slice;
39
40 use syntax::ast::{self, DUMMY_NODE_ID, NodeId};
41 use syntax::ast_util;
42 use syntax::codemap::{Span, Spanned, DUMMY_SP};
43 use rustc_front::fold::{Folder, noop_fold_pat};
44 use rustc_front::print::pprust::pat_to_string;
45 use syntax::ptr::P;
46 use util::nodemap::FnvHashMap;
47
48 pub const DUMMY_WILD_PAT: &'static Pat = &Pat {
49     id: DUMMY_NODE_ID,
50     node: hir::PatWild,
51     span: DUMMY_SP
52 };
53
54 struct Matrix<'a>(Vec<Vec<&'a Pat>>);
55
56 /// Pretty-printer for matrices of patterns, example:
57 /// ++++++++++++++++++++++++++
58 /// + _     + []             +
59 /// ++++++++++++++++++++++++++
60 /// + true  + [First]        +
61 /// ++++++++++++++++++++++++++
62 /// + true  + [Second(true)] +
63 /// ++++++++++++++++++++++++++
64 /// + false + [_]            +
65 /// ++++++++++++++++++++++++++
66 /// + _     + [_, _, ..tail] +
67 /// ++++++++++++++++++++++++++
68 impl<'a> fmt::Debug for Matrix<'a> {
69     fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
70         try!(write!(f, "\n"));
71
72         let &Matrix(ref m) = self;
73         let pretty_printed_matrix: Vec<Vec<String>> = m.iter().map(|row| {
74             row.iter()
75                .map(|&pat| pat_to_string(&*pat))
76                .collect::<Vec<String>>()
77         }).collect();
78
79         let column_count = m.iter().map(|row| row.len()).max().unwrap_or(0);
80         assert!(m.iter().all(|row| row.len() == column_count));
81         let column_widths: Vec<usize> = (0..column_count).map(|col| {
82             pretty_printed_matrix.iter().map(|row| row[col].len()).max().unwrap_or(0)
83         }).collect();
84
85         let total_width = column_widths.iter().cloned().sum::<usize>() + column_count * 3 + 1;
86         let br = repeat('+').take(total_width).collect::<String>();
87         try!(write!(f, "{}\n", br));
88         for row in pretty_printed_matrix {
89             try!(write!(f, "+"));
90             for (column, pat_str) in row.into_iter().enumerate() {
91                 try!(write!(f, " "));
92                 try!(write!(f, "{:1$}", pat_str, column_widths[column]));
93                 try!(write!(f, " +"));
94             }
95             try!(write!(f, "\n"));
96             try!(write!(f, "{}\n", br));
97         }
98         Ok(())
99     }
100 }
101
102 impl<'a> FromIterator<Vec<&'a Pat>> for Matrix<'a> {
103     fn from_iter<T: IntoIterator<Item=Vec<&'a Pat>>>(iter: T) -> Matrix<'a> {
104         Matrix(iter.into_iter().collect())
105     }
106 }
107
108 //NOTE: appears to be the only place other then InferCtxt to contain a ParamEnv
109 pub struct MatchCheckCtxt<'a, 'tcx: 'a> {
110     pub tcx: &'a ty::ctxt<'tcx>,
111     pub param_env: ParameterEnvironment<'a, 'tcx>,
112 }
113
114 #[derive(Clone, PartialEq)]
115 pub enum Constructor {
116     /// The constructor of all patterns that don't vary by constructor,
117     /// e.g. struct patterns and fixed-length arrays.
118     Single,
119     /// Enum variants.
120     Variant(DefId),
121     /// Literal values.
122     ConstantValue(ConstVal),
123     /// Ranges of literal values (2..5).
124     ConstantRange(ConstVal, ConstVal),
125     /// Array patterns of length n.
126     Slice(usize),
127     /// Array patterns with a subslice.
128     SliceWithSubslice(usize, usize)
129 }
130
131 #[derive(Clone, PartialEq)]
132 enum Usefulness {
133     Useful,
134     UsefulWithWitness(Vec<P<Pat>>),
135     NotUseful
136 }
137
138 #[derive(Copy, Clone)]
139 enum WitnessPreference {
140     ConstructWitness,
141     LeaveOutWitness
142 }
143
144 impl<'a, 'tcx, 'v> Visitor<'v> for MatchCheckCtxt<'a, 'tcx> {
145     fn visit_expr(&mut self, ex: &hir::Expr) {
146         check_expr(self, ex);
147     }
148     fn visit_local(&mut self, l: &hir::Local) {
149         check_local(self, l);
150     }
151     fn visit_fn(&mut self, fk: FnKind<'v>, fd: &'v hir::FnDecl,
152                 b: &'v hir::Block, s: Span, n: NodeId) {
153         check_fn(self, fk, fd, b, s, n);
154     }
155 }
156
157 pub fn check_crate(tcx: &ty::ctxt) {
158     tcx.visit_all_items_in_krate(DepNode::MatchCheck, &mut MatchCheckCtxt {
159         tcx: tcx,
160         param_env: tcx.empty_parameter_environment(),
161     });
162     tcx.sess.abort_if_errors();
163 }
164
165 fn check_expr(cx: &mut MatchCheckCtxt, ex: &hir::Expr) {
166     intravisit::walk_expr(cx, ex);
167     match ex.node {
168         hir::ExprMatch(ref scrut, ref arms, source) => {
169             for arm in arms {
170                 // First, check legality of move bindings.
171                 check_legality_of_move_bindings(cx,
172                                                 arm.guard.is_some(),
173                                                 &arm.pats);
174
175                 // Second, if there is a guard on each arm, make sure it isn't
176                 // assigning or borrowing anything mutably.
177                 match arm.guard {
178                     Some(ref guard) => check_for_mutation_in_guard(cx, &**guard),
179                     None => {}
180                 }
181             }
182
183             let mut static_inliner = StaticInliner::new(cx.tcx, None);
184             let inlined_arms = arms.iter().map(|arm| {
185                 (arm.pats.iter().map(|pat| {
186                     static_inliner.fold_pat((*pat).clone())
187                 }).collect(), arm.guard.as_ref().map(|e| &**e))
188             }).collect::<Vec<(Vec<P<Pat>>, Option<&hir::Expr>)>>();
189
190             // Bail out early if inlining failed.
191             if static_inliner.failed {
192                 return;
193             }
194
195             for pat in inlined_arms
196                 .iter()
197                 .flat_map(|&(ref pats, _)| pats) {
198                 // Third, check legality of move bindings.
199                 check_legality_of_bindings_in_at_patterns(cx, &**pat);
200
201                 // Fourth, check if there are any references to NaN that we should warn about.
202                 check_for_static_nan(cx, &**pat);
203
204                 // Fifth, check if for any of the patterns that match an enumerated type
205                 // are bindings with the same name as one of the variants of said type.
206                 check_for_bindings_named_the_same_as_variants(cx, &**pat);
207             }
208
209             // Fourth, check for unreachable arms.
210             check_arms(cx, &inlined_arms[..], source);
211
212             // Finally, check if the whole match expression is exhaustive.
213             // Check for empty enum, because is_useful only works on inhabited types.
214             let pat_ty = cx.tcx.node_id_to_type(scrut.id);
215             if inlined_arms.is_empty() {
216                 if !pat_ty.is_empty(cx.tcx) {
217                     // We know the type is inhabited, so this must be wrong
218                     let mut err = struct_span_err!(cx.tcx.sess, ex.span, E0002,
219                                                    "non-exhaustive patterns: type {} is non-empty",
220                                                    pat_ty);
221                     span_help!(&mut err, ex.span,
222                         "Please ensure that all possible cases are being handled; \
223                          possibly adding wildcards or more match arms.");
224                     err.emit();
225                 }
226                 // If the type *is* empty, it's vacuously exhaustive
227                 return;
228             }
229
230             let matrix: Matrix = inlined_arms
231                 .iter()
232                 .filter(|&&(_, guard)| guard.is_none())
233                 .flat_map(|arm| &arm.0)
234                 .map(|pat| vec![&**pat])
235                 .collect();
236             check_exhaustive(cx, ex.span, &matrix, source);
237         },
238         _ => ()
239     }
240 }
241
242 fn check_for_bindings_named_the_same_as_variants(cx: &MatchCheckCtxt, pat: &Pat) {
243     front_util::walk_pat(pat, |p| {
244         match p.node {
245             hir::PatIdent(hir::BindByValue(hir::MutImmutable), ident, None) => {
246                 let pat_ty = cx.tcx.pat_ty(p);
247                 if let ty::TyEnum(edef, _) = pat_ty.sty {
248                     let def = cx.tcx.def_map.borrow().get(&p.id).map(|d| d.full_def());
249                     if let Some(Def::Local(..)) = def {
250                         if edef.variants.iter().any(|variant|
251                             variant.name == ident.node.unhygienic_name
252                                 && variant.kind() == VariantKind::Unit
253                         ) {
254                             let ty_path = cx.tcx.item_path_str(edef.did);
255                             let mut err = struct_span_warn!(cx.tcx.sess, p.span, E0170,
256                                 "pattern binding `{}` is named the same as one \
257                                  of the variants of the type `{}`",
258                                 ident.node, ty_path);
259                             fileline_help!(err, p.span,
260                                 "if you meant to match on a variant, \
261                                  consider making the path in the pattern qualified: `{}::{}`",
262                                 ty_path, ident.node);
263                             err.emit();
264                         }
265                     }
266                 }
267             }
268             _ => ()
269         }
270         true
271     });
272 }
273
274 // Check that we do not match against a static NaN (#6804)
275 fn check_for_static_nan(cx: &MatchCheckCtxt, pat: &Pat) {
276     front_util::walk_pat(pat, |p| {
277         if let hir::PatLit(ref expr) = p.node {
278             match eval_const_expr_partial(cx.tcx, &**expr, ExprTypeChecked, None) {
279                 Ok(ConstVal::Float(f)) if f.is_nan() => {
280                     span_warn!(cx.tcx.sess, p.span, E0003,
281                                "unmatchable NaN in pattern, \
282                                 use the is_nan method in a guard instead");
283                 }
284                 Ok(_) => {}
285
286                 Err(err) => {
287                     let mut diag = struct_span_err!(cx.tcx.sess, err.span, E0471,
288                                                     "constant evaluation error: {}",
289                                                     err.description());
290                     if !p.span.contains(err.span) {
291                         diag.span_note(p.span, "in pattern here");
292                     }
293                     diag.emit();
294                 }
295             }
296         }
297         true
298     });
299 }
300
301 // Check for unreachable patterns
302 fn check_arms(cx: &MatchCheckCtxt,
303               arms: &[(Vec<P<Pat>>, Option<&hir::Expr>)],
304               source: hir::MatchSource) {
305     let mut seen = Matrix(vec![]);
306     let mut printed_if_let_err = false;
307     for &(ref pats, guard) in arms {
308         for pat in pats {
309             let v = vec![&**pat];
310
311             match is_useful(cx, &seen, &v[..], LeaveOutWitness) {
312                 NotUseful => {
313                     match source {
314                         hir::MatchSource::IfLetDesugar { .. } => {
315                             if printed_if_let_err {
316                                 // we already printed an irrefutable if-let pattern error.
317                                 // We don't want two, that's just confusing.
318                             } else {
319                                 // find the first arm pattern so we can use its span
320                                 let &(ref first_arm_pats, _) = &arms[0];
321                                 let first_pat = &first_arm_pats[0];
322                                 let span = first_pat.span;
323                                 span_err!(cx.tcx.sess, span, E0162, "irrefutable if-let pattern");
324                                 printed_if_let_err = true;
325                             }
326                         },
327
328                         hir::MatchSource::WhileLetDesugar => {
329                             // find the first arm pattern so we can use its span
330                             let &(ref first_arm_pats, _) = &arms[0];
331                             let first_pat = &first_arm_pats[0];
332                             let span = first_pat.span;
333                             span_err!(cx.tcx.sess, span, E0165, "irrefutable while-let pattern");
334                         },
335
336                         hir::MatchSource::ForLoopDesugar => {
337                             // this is a bug, because on `match iter.next()` we cover
338                             // `Some(<head>)` and `None`. It's impossible to have an unreachable
339                             // pattern
340                             // (see libsyntax/ext/expand.rs for the full expansion of a for loop)
341                             cx.tcx.sess.span_bug(pat.span, "unreachable for-loop pattern")
342                         },
343
344                         hir::MatchSource::Normal => {
345                             span_err!(cx.tcx.sess, pat.span, E0001, "unreachable pattern")
346                         },
347                     }
348                 }
349                 Useful => (),
350                 UsefulWithWitness(_) => unreachable!()
351             }
352             if guard.is_none() {
353                 let Matrix(mut rows) = seen;
354                 rows.push(v);
355                 seen = Matrix(rows);
356             }
357         }
358     }
359 }
360
361 fn raw_pat<'a>(p: &'a Pat) -> &'a Pat {
362     match p.node {
363         hir::PatIdent(_, _, Some(ref s)) => raw_pat(&**s),
364         _ => p
365     }
366 }
367
368 fn check_exhaustive(cx: &MatchCheckCtxt, sp: Span, matrix: &Matrix, source: hir::MatchSource) {
369     match is_useful(cx, matrix, &[DUMMY_WILD_PAT], ConstructWitness) {
370         UsefulWithWitness(pats) => {
371             let witnesses = if pats.is_empty() {
372                 vec![DUMMY_WILD_PAT]
373             } else {
374                 pats.iter().map(|w| &**w ).collect()
375             };
376             match source {
377                 hir::MatchSource::ForLoopDesugar => {
378                     // `witnesses[0]` has the form `Some(<head>)`, peel off the `Some`
379                     let witness = match witnesses[0].node {
380                         hir::PatEnum(_, Some(ref pats)) => match &pats[..] {
381                             [ref pat] => &**pat,
382                             _ => unreachable!(),
383                         },
384                         _ => unreachable!(),
385                     };
386                     span_err!(cx.tcx.sess, sp, E0297,
387                         "refutable pattern in `for` loop binding: \
388                                 `{}` not covered",
389                                 pat_to_string(witness));
390                 },
391                 _ => {
392                     let pattern_strings: Vec<_> = witnesses.iter().map(|w| {
393                         pat_to_string(w)
394                     }).collect();
395                     const LIMIT: usize = 3;
396                     let joined_patterns = match pattern_strings.len() {
397                         0 => unreachable!(),
398                         1 => format!("`{}`", pattern_strings[0]),
399                         2...LIMIT => {
400                             let (tail, head) = pattern_strings.split_last().unwrap();
401                             format!("`{}`", head.join("`, `") + "` and `" + tail)
402                         },
403                         _ => {
404                             let (head, tail) = pattern_strings.split_at(LIMIT);
405                             format!("`{}` and {} more", head.join("`, `"), tail.len())
406                         }
407                     };
408                     span_err!(cx.tcx.sess, sp, E0004,
409                         "non-exhaustive patterns: {} not covered",
410                         joined_patterns
411                     );
412                 },
413             }
414         }
415         NotUseful => {
416             // This is good, wildcard pattern isn't reachable
417         },
418         _ => unreachable!()
419     }
420 }
421
422 fn const_val_to_expr(value: &ConstVal) -> P<hir::Expr> {
423     let node = match value {
424         &ConstVal::Bool(b) => ast::LitBool(b),
425         _ => unreachable!()
426     };
427     P(hir::Expr {
428         id: 0,
429         node: hir::ExprLit(P(Spanned { node: node, span: DUMMY_SP })),
430         span: DUMMY_SP,
431         attrs: None,
432     })
433 }
434
435 pub struct StaticInliner<'a, 'tcx: 'a> {
436     pub tcx: &'a ty::ctxt<'tcx>,
437     pub failed: bool,
438     pub renaming_map: Option<&'a mut FnvHashMap<(NodeId, Span), NodeId>>,
439 }
440
441 impl<'a, 'tcx> StaticInliner<'a, 'tcx> {
442     pub fn new<'b>(tcx: &'b ty::ctxt<'tcx>,
443                    renaming_map: Option<&'b mut FnvHashMap<(NodeId, Span), NodeId>>)
444                    -> StaticInliner<'b, 'tcx> {
445         StaticInliner {
446             tcx: tcx,
447             failed: false,
448             renaming_map: renaming_map
449         }
450     }
451 }
452
453 struct RenamingRecorder<'map> {
454     substituted_node_id: NodeId,
455     origin_span: Span,
456     renaming_map: &'map mut FnvHashMap<(NodeId, Span), NodeId>
457 }
458
459 impl<'map> ast_util::IdVisitingOperation for RenamingRecorder<'map> {
460     fn visit_id(&mut self, node_id: NodeId) {
461         let key = (node_id, self.origin_span);
462         self.renaming_map.insert(key, self.substituted_node_id);
463     }
464 }
465
466 impl<'a, 'tcx> Folder for StaticInliner<'a, 'tcx> {
467     fn fold_pat(&mut self, pat: P<Pat>) -> P<Pat> {
468         return match pat.node {
469             hir::PatIdent(..) | hir::PatEnum(..) | hir::PatQPath(..) => {
470                 let def = self.tcx.def_map.borrow().get(&pat.id).map(|d| d.full_def());
471                 match def {
472                     Some(Def::AssociatedConst(did)) |
473                     Some(Def::Const(did)) => match lookup_const_by_id(self.tcx, did,
474                                                                     Some(pat.id), None) {
475                         Some(const_expr) => {
476                             const_expr_to_pat(self.tcx, const_expr, pat.span).map(|new_pat| {
477
478                                 if let Some(ref mut renaming_map) = self.renaming_map {
479                                     // Record any renamings we do here
480                                     record_renamings(const_expr, &pat, renaming_map);
481                                 }
482
483                                 new_pat
484                             })
485                         }
486                         None => {
487                             self.failed = true;
488                             span_err!(self.tcx.sess, pat.span, E0158,
489                                 "statics cannot be referenced in patterns");
490                             pat
491                         }
492                     },
493                     _ => noop_fold_pat(pat, self)
494                 }
495             }
496             _ => noop_fold_pat(pat, self)
497         };
498
499         fn record_renamings(const_expr: &hir::Expr,
500                             substituted_pat: &hir::Pat,
501                             renaming_map: &mut FnvHashMap<(NodeId, Span), NodeId>) {
502             let mut renaming_recorder = RenamingRecorder {
503                 substituted_node_id: substituted_pat.id,
504                 origin_span: substituted_pat.span,
505                 renaming_map: renaming_map,
506             };
507
508             let mut id_visitor = front_util::IdVisitor::new(&mut renaming_recorder);
509
510             id_visitor.visit_expr(const_expr);
511         }
512     }
513 }
514
515 /// Constructs a partial witness for a pattern given a list of
516 /// patterns expanded by the specialization step.
517 ///
518 /// When a pattern P is discovered to be useful, this function is used bottom-up
519 /// to reconstruct a complete witness, e.g. a pattern P' that covers a subset
520 /// of values, V, where each value in that set is not covered by any previously
521 /// used patterns and is covered by the pattern P'. Examples:
522 ///
523 /// left_ty: tuple of 3 elements
524 /// pats: [10, 20, _]           => (10, 20, _)
525 ///
526 /// left_ty: struct X { a: (bool, &'static str), b: usize}
527 /// pats: [(false, "foo"), 42]  => X { a: (false, "foo"), b: 42 }
528 fn construct_witness<'a,'tcx>(cx: &MatchCheckCtxt<'a,'tcx>, ctor: &Constructor,
529                               pats: Vec<&Pat>, left_ty: Ty<'tcx>) -> P<Pat> {
530     let pats_len = pats.len();
531     let mut pats = pats.into_iter().map(|p| P((*p).clone()));
532     let pat = match left_ty.sty {
533         ty::TyTuple(_) => hir::PatTup(pats.collect()),
534
535         ty::TyEnum(adt, _) | ty::TyStruct(adt, _)  => {
536             let v = adt.variant_of_ctor(ctor);
537             if let VariantKind::Struct = v.kind() {
538                 let field_pats: hir::HirVec<_> = v.fields.iter()
539                     .zip(pats)
540                     .filter(|&(_, ref pat)| pat.node != hir::PatWild)
541                     .map(|(field, pat)| Spanned {
542                         span: DUMMY_SP,
543                         node: hir::FieldPat {
544                             name: field.name,
545                             pat: pat,
546                             is_shorthand: false,
547                         }
548                     }).collect();
549                 let has_more_fields = field_pats.len() < pats_len;
550                 hir::PatStruct(def_to_path(cx.tcx, v.did), field_pats, has_more_fields)
551             } else {
552                 hir::PatEnum(def_to_path(cx.tcx, v.did), Some(pats.collect()))
553             }
554         }
555
556         ty::TyRef(_, ty::TypeAndMut { ty, mutbl }) => {
557             match ty.sty {
558                ty::TyArray(_, n) => match ctor {
559                     &Single => {
560                         assert_eq!(pats_len, n);
561                         hir::PatVec(pats.collect(), None, hir::HirVec::new())
562                     },
563                     _ => unreachable!()
564                 },
565                 ty::TySlice(_) => match ctor {
566                     &Slice(n) => {
567                         assert_eq!(pats_len, n);
568                         hir::PatVec(pats.collect(), None, hir::HirVec::new())
569                     },
570                     _ => unreachable!()
571                 },
572                 ty::TyStr => hir::PatWild,
573
574                 _ => {
575                     assert_eq!(pats_len, 1);
576                     hir::PatRegion(pats.nth(0).unwrap(), mutbl)
577                 }
578             }
579         }
580
581         ty::TyArray(_, len) => {
582             assert_eq!(pats_len, len);
583             hir::PatVec(pats.collect(), None, hir::HirVec::new())
584         }
585
586         _ => {
587             match *ctor {
588                 ConstantValue(ref v) => hir::PatLit(const_val_to_expr(v)),
589                 _ => hir::PatWild,
590             }
591         }
592     };
593
594     P(hir::Pat {
595         id: 0,
596         node: pat,
597         span: DUMMY_SP
598     })
599 }
600
601 impl<'tcx, 'container> ty::AdtDefData<'tcx, 'container> {
602     fn variant_of_ctor(&self,
603                        ctor: &Constructor)
604                        -> &VariantDefData<'tcx, 'container> {
605         match ctor {
606             &Variant(vid) => self.variant_with_id(vid),
607             _ => self.struct_variant()
608         }
609     }
610 }
611
612 fn missing_constructors(cx: &MatchCheckCtxt, &Matrix(ref rows): &Matrix,
613                        left_ty: Ty, max_slice_length: usize) -> Vec<Constructor> {
614     let used_constructors: Vec<Constructor> = rows.iter()
615         .flat_map(|row| pat_constructors(cx, row[0], left_ty, max_slice_length))
616         .collect();
617     all_constructors(cx, left_ty, max_slice_length)
618         .into_iter()
619         .filter(|c| !used_constructors.contains(c))
620         .collect()
621 }
622
623 /// This determines the set of all possible constructors of a pattern matching
624 /// values of type `left_ty`. For vectors, this would normally be an infinite set
625 /// but is instead bounded by the maximum fixed length of slice patterns in
626 /// the column of patterns being analyzed.
627 fn all_constructors(_cx: &MatchCheckCtxt, left_ty: Ty,
628                     max_slice_length: usize) -> Vec<Constructor> {
629     match left_ty.sty {
630         ty::TyBool =>
631             [true, false].iter().map(|b| ConstantValue(ConstVal::Bool(*b))).collect(),
632
633         ty::TyRef(_, ty::TypeAndMut { ty, .. }) => match ty.sty {
634             ty::TySlice(_) =>
635                 (0..max_slice_length+1).map(|length| Slice(length)).collect(),
636             _ => vec![Single]
637         },
638
639         ty::TyEnum(def, _) => def.variants.iter().map(|v| Variant(v.did)).collect(),
640         _ => vec![Single]
641     }
642 }
643
644 // Algorithm from http://moscova.inria.fr/~maranget/papers/warn/index.html
645 //
646 // Whether a vector `v` of patterns is 'useful' in relation to a set of such
647 // vectors `m` is defined as there being a set of inputs that will match `v`
648 // but not any of the sets in `m`.
649 //
650 // This is used both for reachability checking (if a pattern isn't useful in
651 // relation to preceding patterns, it is not reachable) and exhaustiveness
652 // checking (if a wildcard pattern is useful in relation to a matrix, the
653 // matrix isn't exhaustive).
654
655 // Note: is_useful doesn't work on empty types, as the paper notes.
656 // So it assumes that v is non-empty.
657 fn is_useful(cx: &MatchCheckCtxt,
658              matrix: &Matrix,
659              v: &[&Pat],
660              witness: WitnessPreference)
661              -> Usefulness {
662     let &Matrix(ref rows) = matrix;
663     debug!("{:?}", matrix);
664     if rows.is_empty() {
665         return match witness {
666             ConstructWitness => UsefulWithWitness(vec!()),
667             LeaveOutWitness => Useful
668         };
669     }
670     if rows[0].is_empty() {
671         return NotUseful;
672     }
673     assert!(rows.iter().all(|r| r.len() == v.len()));
674     let real_pat = match rows.iter().find(|r| (*r)[0].id != DUMMY_NODE_ID) {
675         Some(r) => raw_pat(r[0]),
676         None if v.is_empty() => return NotUseful,
677         None => v[0]
678     };
679     let left_ty = if real_pat.id == DUMMY_NODE_ID {
680         cx.tcx.mk_nil()
681     } else {
682         let left_ty = cx.tcx.pat_ty(&*real_pat);
683
684         match real_pat.node {
685             hir::PatIdent(hir::BindByRef(..), _, _) => {
686                 left_ty.builtin_deref(false, NoPreference).unwrap().ty
687             }
688             _ => left_ty,
689         }
690     };
691
692     let max_slice_length = rows.iter().filter_map(|row| match row[0].node {
693         hir::PatVec(ref before, _, ref after) => Some(before.len() + after.len()),
694         _ => None
695     }).max().map_or(0, |v| v + 1);
696
697     let constructors = pat_constructors(cx, v[0], left_ty, max_slice_length);
698     if constructors.is_empty() {
699         let constructors = missing_constructors(cx, matrix, left_ty, max_slice_length);
700         if constructors.is_empty() {
701             all_constructors(cx, left_ty, max_slice_length).into_iter().map(|c| {
702                 match is_useful_specialized(cx, matrix, v, c.clone(), left_ty, witness) {
703                     UsefulWithWitness(pats) => UsefulWithWitness({
704                         let arity = constructor_arity(cx, &c, left_ty);
705                         let mut result = {
706                             let pat_slice = &pats[..];
707                             let subpats: Vec<_> = (0..arity).map(|i| {
708                                 pat_slice.get(i).map_or(DUMMY_WILD_PAT, |p| &**p)
709                             }).collect();
710                             vec![construct_witness(cx, &c, subpats, left_ty)]
711                         };
712                         result.extend(pats.into_iter().skip(arity));
713                         result
714                     }),
715                     result => result
716                 }
717             }).find(|result| result != &NotUseful).unwrap_or(NotUseful)
718         } else {
719             let matrix = rows.iter().filter_map(|r| {
720                 if pat_is_binding_or_wild(&cx.tcx.def_map.borrow(), raw_pat(r[0])) {
721                     Some(r[1..].to_vec())
722                 } else {
723                     None
724                 }
725             }).collect();
726             match is_useful(cx, &matrix, &v[1..], witness) {
727                 UsefulWithWitness(pats) => {
728                     let mut new_pats: Vec<_> = constructors.into_iter().map(|constructor| {
729                         let arity = constructor_arity(cx, &constructor, left_ty);
730                         let wild_pats = vec![DUMMY_WILD_PAT; arity];
731                         construct_witness(cx, &constructor, wild_pats, left_ty)
732                     }).collect();
733                     new_pats.extend(pats);
734                     UsefulWithWitness(new_pats)
735                 },
736                 result => result
737             }
738         }
739     } else {
740         constructors.into_iter().map(|c|
741             is_useful_specialized(cx, matrix, v, c.clone(), left_ty, witness)
742         ).find(|result| result != &NotUseful).unwrap_or(NotUseful)
743     }
744 }
745
746 fn is_useful_specialized(cx: &MatchCheckCtxt, &Matrix(ref m): &Matrix,
747                          v: &[&Pat], ctor: Constructor, lty: Ty,
748                          witness: WitnessPreference) -> Usefulness {
749     let arity = constructor_arity(cx, &ctor, lty);
750     let matrix = Matrix(m.iter().filter_map(|r| {
751         specialize(cx, &r[..], &ctor, 0, arity)
752     }).collect());
753     match specialize(cx, v, &ctor, 0, arity) {
754         Some(v) => is_useful(cx, &matrix, &v[..], witness),
755         None => NotUseful
756     }
757 }
758
759 /// Determines the constructors that the given pattern can be specialized to.
760 ///
761 /// In most cases, there's only one constructor that a specific pattern
762 /// represents, such as a specific enum variant or a specific literal value.
763 /// Slice patterns, however, can match slices of different lengths. For instance,
764 /// `[a, b, ..tail]` can match a slice of length 2, 3, 4 and so on.
765 ///
766 /// On the other hand, a wild pattern and an identifier pattern cannot be
767 /// specialized in any way.
768 fn pat_constructors(cx: &MatchCheckCtxt, p: &Pat,
769                     left_ty: Ty, max_slice_length: usize) -> Vec<Constructor> {
770     let pat = raw_pat(p);
771     match pat.node {
772         hir::PatIdent(..) =>
773             match cx.tcx.def_map.borrow().get(&pat.id).map(|d| d.full_def()) {
774                 Some(Def::Const(..)) | Some(Def::AssociatedConst(..)) =>
775                     cx.tcx.sess.span_bug(pat.span, "const pattern should've \
776                                                     been rewritten"),
777                 Some(Def::Struct(..)) => vec!(Single),
778                 Some(Def::Variant(_, id)) => vec!(Variant(id)),
779                 _ => vec!()
780             },
781         hir::PatEnum(..) =>
782             match cx.tcx.def_map.borrow().get(&pat.id).map(|d| d.full_def()) {
783                 Some(Def::Const(..)) | Some(Def::AssociatedConst(..)) =>
784                     cx.tcx.sess.span_bug(pat.span, "const pattern should've \
785                                                     been rewritten"),
786                 Some(Def::Variant(_, id)) => vec!(Variant(id)),
787                 _ => vec!(Single)
788             },
789         hir::PatQPath(..) =>
790             cx.tcx.sess.span_bug(pat.span, "const pattern should've \
791                                             been rewritten"),
792         hir::PatStruct(..) =>
793             match cx.tcx.def_map.borrow().get(&pat.id).map(|d| d.full_def()) {
794                 Some(Def::Const(..)) | Some(Def::AssociatedConst(..)) =>
795                     cx.tcx.sess.span_bug(pat.span, "const pattern should've \
796                                                     been rewritten"),
797                 Some(Def::Variant(_, id)) => vec!(Variant(id)),
798                 _ => vec!(Single)
799             },
800         hir::PatLit(ref expr) =>
801             vec!(ConstantValue(eval_const_expr(cx.tcx, &**expr))),
802         hir::PatRange(ref lo, ref hi) =>
803             vec!(ConstantRange(eval_const_expr(cx.tcx, &**lo), eval_const_expr(cx.tcx, &**hi))),
804         hir::PatVec(ref before, ref slice, ref after) =>
805             match left_ty.sty {
806                 ty::TyArray(_, _) => vec!(Single),
807                 _                      => if slice.is_some() {
808                     (before.len() + after.len()..max_slice_length+1)
809                         .map(|length| Slice(length))
810                         .collect()
811                 } else {
812                     vec!(Slice(before.len() + after.len()))
813                 }
814             },
815         hir::PatBox(_) | hir::PatTup(_) | hir::PatRegion(..) =>
816             vec!(Single),
817         hir::PatWild =>
818             vec!(),
819     }
820 }
821
822 /// This computes the arity of a constructor. The arity of a constructor
823 /// is how many subpattern patterns of that constructor should be expanded to.
824 ///
825 /// For instance, a tuple pattern (_, 42, Some([])) has the arity of 3.
826 /// A struct pattern's arity is the number of fields it contains, etc.
827 pub fn constructor_arity(_cx: &MatchCheckCtxt, ctor: &Constructor, ty: Ty) -> usize {
828     match ty.sty {
829         ty::TyTuple(ref fs) => fs.len(),
830         ty::TyBox(_) => 1,
831         ty::TyRef(_, ty::TypeAndMut { ty, .. }) => match ty.sty {
832             ty::TySlice(_) => match *ctor {
833                 Slice(length) => length,
834                 ConstantValue(_) => 0,
835                 _ => unreachable!()
836             },
837             ty::TyStr => 0,
838             _ => 1
839         },
840         ty::TyEnum(adt, _) | ty::TyStruct(adt, _) => {
841             adt.variant_of_ctor(ctor).fields.len()
842         }
843         ty::TyArray(_, n) => n,
844         _ => 0
845     }
846 }
847
848 fn range_covered_by_constructor(ctor: &Constructor,
849                                 from: &ConstVal, to: &ConstVal) -> Option<bool> {
850     let (c_from, c_to) = match *ctor {
851         ConstantValue(ref value)        => (value, value),
852         ConstantRange(ref from, ref to) => (from, to),
853         Single                          => return Some(true),
854         _                               => unreachable!()
855     };
856     let cmp_from = compare_const_vals(c_from, from);
857     let cmp_to = compare_const_vals(c_to, to);
858     match (cmp_from, cmp_to) {
859         (Some(cmp_from), Some(cmp_to)) => {
860             Some(cmp_from != Ordering::Less && cmp_to != Ordering::Greater)
861         }
862         _ => None
863     }
864 }
865
866 /// This is the main specialization step. It expands the first pattern in the given row
867 /// into `arity` patterns based on the constructor. For most patterns, the step is trivial,
868 /// for instance tuple patterns are flattened and box patterns expand into their inner pattern.
869 ///
870 /// OTOH, slice patterns with a subslice pattern (..tail) can be expanded into multiple
871 /// different patterns.
872 /// Structure patterns with a partial wild pattern (Foo { a: 42, .. }) have their missing
873 /// fields filled with wild patterns.
874 pub fn specialize<'a>(cx: &MatchCheckCtxt, r: &[&'a Pat],
875                       constructor: &Constructor, col: usize, arity: usize) -> Option<Vec<&'a Pat>> {
876     let &Pat {
877         id: pat_id, ref node, span: pat_span
878     } = raw_pat(r[col]);
879     let head: Option<Vec<&Pat>> = match *node {
880         hir::PatWild =>
881             Some(vec![DUMMY_WILD_PAT; arity]),
882
883         hir::PatIdent(_, _, _) => {
884             let opt_def = cx.tcx.def_map.borrow().get(&pat_id).map(|d| d.full_def());
885             match opt_def {
886                 Some(Def::Const(..)) | Some(Def::AssociatedConst(..)) =>
887                     cx.tcx.sess.span_bug(pat_span, "const pattern should've \
888                                                     been rewritten"),
889                 Some(Def::Variant(_, id)) => if *constructor == Variant(id) {
890                     Some(vec!())
891                 } else {
892                     None
893                 },
894                 _ => Some(vec![DUMMY_WILD_PAT; arity])
895             }
896         }
897
898         hir::PatEnum(_, ref args) => {
899             let def = cx.tcx.def_map.borrow().get(&pat_id).unwrap().full_def();
900             match def {
901                 Def::Const(..) | Def::AssociatedConst(..) =>
902                     cx.tcx.sess.span_bug(pat_span, "const pattern should've \
903                                                     been rewritten"),
904                 Def::Variant(_, id) if *constructor != Variant(id) => None,
905                 Def::Variant(..) | Def::Struct(..) => {
906                     Some(match args {
907                         &Some(ref args) => args.iter().map(|p| &**p).collect(),
908                         &None => vec![DUMMY_WILD_PAT; arity],
909                     })
910                 }
911                 _ => None
912             }
913         }
914
915         hir::PatQPath(_, _) => {
916             cx.tcx.sess.span_bug(pat_span, "const pattern should've \
917                                             been rewritten")
918         }
919
920         hir::PatStruct(_, ref pattern_fields, _) => {
921             let def = cx.tcx.def_map.borrow().get(&pat_id).unwrap().full_def();
922             let adt = cx.tcx.node_id_to_type(pat_id).ty_adt_def().unwrap();
923             let variant = adt.variant_of_ctor(constructor);
924             let def_variant = adt.variant_of_def(def);
925             if variant.did == def_variant.did {
926                 Some(variant.fields.iter().map(|sf| {
927                     match pattern_fields.iter().find(|f| f.node.name == sf.name) {
928                         Some(ref f) => &*f.node.pat,
929                         _ => DUMMY_WILD_PAT
930                     }
931                 }).collect())
932             } else {
933                 None
934             }
935         }
936
937         hir::PatTup(ref args) =>
938             Some(args.iter().map(|p| &**p).collect()),
939
940         hir::PatBox(ref inner) | hir::PatRegion(ref inner, _) =>
941             Some(vec![&**inner]),
942
943         hir::PatLit(ref expr) => {
944             let expr_value = eval_const_expr(cx.tcx, &**expr);
945             match range_covered_by_constructor(constructor, &expr_value, &expr_value) {
946                 Some(true) => Some(vec![]),
947                 Some(false) => None,
948                 None => {
949                     span_err!(cx.tcx.sess, pat_span, E0298, "mismatched types between arms");
950                     None
951                 }
952             }
953         }
954
955         hir::PatRange(ref from, ref to) => {
956             let from_value = eval_const_expr(cx.tcx, &**from);
957             let to_value = eval_const_expr(cx.tcx, &**to);
958             match range_covered_by_constructor(constructor, &from_value, &to_value) {
959                 Some(true) => Some(vec![]),
960                 Some(false) => None,
961                 None => {
962                     span_err!(cx.tcx.sess, pat_span, E0299, "mismatched types between arms");
963                     None
964                 }
965             }
966         }
967
968         hir::PatVec(ref before, ref slice, ref after) => {
969             match *constructor {
970                 // Fixed-length vectors.
971                 Single => {
972                     let mut pats: Vec<&Pat> = before.iter().map(|p| &**p).collect();
973                     pats.extend(repeat(DUMMY_WILD_PAT).take(arity - before.len() - after.len()));
974                     pats.extend(after.iter().map(|p| &**p));
975                     Some(pats)
976                 },
977                 Slice(length) if before.len() + after.len() <= length && slice.is_some() => {
978                     let mut pats: Vec<&Pat> = before.iter().map(|p| &**p).collect();
979                     pats.extend(repeat(DUMMY_WILD_PAT).take(arity - before.len() - after.len()));
980                     pats.extend(after.iter().map(|p| &**p));
981                     Some(pats)
982                 },
983                 Slice(length) if before.len() + after.len() == length => {
984                     let mut pats: Vec<&Pat> = before.iter().map(|p| &**p).collect();
985                     pats.extend(after.iter().map(|p| &**p));
986                     Some(pats)
987                 },
988                 SliceWithSubslice(prefix, suffix)
989                     if before.len() == prefix
990                         && after.len() == suffix
991                         && slice.is_some() => {
992                     let mut pats: Vec<&Pat> = before.iter().map(|p| &**p).collect();
993                     pats.extend(after.iter().map(|p| &**p));
994                     Some(pats)
995                 }
996                 _ => None
997             }
998         }
999     };
1000     head.map(|mut head| {
1001         head.extend_from_slice(&r[..col]);
1002         head.extend_from_slice(&r[col + 1..]);
1003         head
1004     })
1005 }
1006
1007 fn check_local(cx: &mut MatchCheckCtxt, loc: &hir::Local) {
1008     intravisit::walk_local(cx, loc);
1009
1010     let pat = StaticInliner::new(cx.tcx, None).fold_pat(loc.pat.clone());
1011     check_irrefutable(cx, &pat, false);
1012
1013     // Check legality of move bindings and `@` patterns.
1014     check_legality_of_move_bindings(cx, false, slice::ref_slice(&loc.pat));
1015     check_legality_of_bindings_in_at_patterns(cx, &*loc.pat);
1016 }
1017
1018 fn check_fn(cx: &mut MatchCheckCtxt,
1019             kind: FnKind,
1020             decl: &hir::FnDecl,
1021             body: &hir::Block,
1022             sp: Span,
1023             fn_id: NodeId) {
1024     match kind {
1025         FnKind::Closure => {}
1026         _ => cx.param_env = ParameterEnvironment::for_item(cx.tcx, fn_id),
1027     }
1028
1029     intravisit::walk_fn(cx, kind, decl, body, sp);
1030
1031     for input in &decl.inputs {
1032         check_irrefutable(cx, &input.pat, true);
1033         check_legality_of_move_bindings(cx, false, slice::ref_slice(&input.pat));
1034         check_legality_of_bindings_in_at_patterns(cx, &*input.pat);
1035     }
1036 }
1037
1038 fn check_irrefutable(cx: &MatchCheckCtxt, pat: &Pat, is_fn_arg: bool) {
1039     let origin = if is_fn_arg {
1040         "function argument"
1041     } else {
1042         "local binding"
1043     };
1044
1045     is_refutable(cx, pat, |uncovered_pat| {
1046         span_err!(cx.tcx.sess, pat.span, E0005,
1047             "refutable pattern in {}: `{}` not covered",
1048             origin,
1049             pat_to_string(uncovered_pat),
1050         );
1051     });
1052 }
1053
1054 fn is_refutable<A, F>(cx: &MatchCheckCtxt, pat: &Pat, refutable: F) -> Option<A> where
1055     F: FnOnce(&Pat) -> A,
1056 {
1057     let pats = Matrix(vec!(vec!(pat)));
1058     match is_useful(cx, &pats, &[DUMMY_WILD_PAT], ConstructWitness) {
1059         UsefulWithWitness(pats) => {
1060             assert_eq!(pats.len(), 1);
1061             Some(refutable(&*pats[0]))
1062         },
1063         NotUseful => None,
1064         Useful => unreachable!()
1065     }
1066 }
1067
1068 // Legality of move bindings checking
1069 fn check_legality_of_move_bindings(cx: &MatchCheckCtxt,
1070                                    has_guard: bool,
1071                                    pats: &[P<Pat>]) {
1072     let tcx = cx.tcx;
1073     let def_map = &tcx.def_map;
1074     let mut by_ref_span = None;
1075     for pat in pats {
1076         pat_bindings(def_map, &**pat, |bm, _, span, _path| {
1077             match bm {
1078                 hir::BindByRef(_) => {
1079                     by_ref_span = Some(span);
1080                 }
1081                 hir::BindByValue(_) => {
1082                 }
1083             }
1084         })
1085     }
1086
1087     let check_move = |p: &Pat, sub: Option<&Pat>| {
1088         // check legality of moving out of the enum
1089
1090         // x @ Foo(..) is legal, but x @ Foo(y) isn't.
1091         if sub.map_or(false, |p| pat_contains_bindings(&def_map.borrow(), &*p)) {
1092             span_err!(cx.tcx.sess, p.span, E0007, "cannot bind by-move with sub-bindings");
1093         } else if has_guard {
1094             span_err!(cx.tcx.sess, p.span, E0008, "cannot bind by-move into a pattern guard");
1095         } else if by_ref_span.is_some() {
1096             let mut err = struct_span_err!(cx.tcx.sess, p.span, E0009,
1097                                            "cannot bind by-move and by-ref in the same pattern");
1098             span_note!(&mut err, by_ref_span.unwrap(), "by-ref binding occurs here");
1099             err.emit();
1100         }
1101     };
1102
1103     for pat in pats {
1104         front_util::walk_pat(&**pat, |p| {
1105             if pat_is_binding(&def_map.borrow(), &*p) {
1106                 match p.node {
1107                     hir::PatIdent(hir::BindByValue(_), _, ref sub) => {
1108                         let pat_ty = tcx.node_id_to_type(p.id);
1109                         //FIXME: (@jroesch) this code should be floated up as well
1110                         let infcx = infer::new_infer_ctxt(cx.tcx,
1111                                                           &cx.tcx.tables,
1112                                                           Some(cx.param_env.clone()));
1113                         if infcx.type_moves_by_default(pat_ty, pat.span) {
1114                             check_move(p, sub.as_ref().map(|p| &**p));
1115                         }
1116                     }
1117                     hir::PatIdent(hir::BindByRef(_), _, _) => {
1118                     }
1119                     _ => {
1120                         cx.tcx.sess.span_bug(
1121                             p.span,
1122                             &format!("binding pattern {} is not an \
1123                                      identifier: {:?}",
1124                                     p.id,
1125                                     p.node));
1126                     }
1127                 }
1128             }
1129             true
1130         });
1131     }
1132 }
1133
1134 /// Ensures that a pattern guard doesn't borrow by mutable reference or
1135 /// assign.
1136 fn check_for_mutation_in_guard<'a, 'tcx>(cx: &'a MatchCheckCtxt<'a, 'tcx>,
1137                                          guard: &hir::Expr) {
1138     let mut checker = MutationChecker {
1139         cx: cx,
1140     };
1141
1142     let infcx = infer::new_infer_ctxt(cx.tcx,
1143                                       &cx.tcx.tables,
1144                                       Some(checker.cx.param_env.clone()));
1145
1146     let mut visitor = ExprUseVisitor::new(&mut checker, &infcx);
1147     visitor.walk_expr(guard);
1148 }
1149
1150 struct MutationChecker<'a, 'tcx: 'a> {
1151     cx: &'a MatchCheckCtxt<'a, 'tcx>,
1152 }
1153
1154 impl<'a, 'tcx> Delegate<'tcx> for MutationChecker<'a, 'tcx> {
1155     fn matched_pat(&mut self, _: &Pat, _: cmt, _: euv::MatchMode) {}
1156     fn consume(&mut self, _: NodeId, _: Span, _: cmt, _: ConsumeMode) {}
1157     fn consume_pat(&mut self, _: &Pat, _: cmt, _: ConsumeMode) {}
1158     fn borrow(&mut self,
1159               _: NodeId,
1160               span: Span,
1161               _: cmt,
1162               _: Region,
1163               kind: BorrowKind,
1164               _: LoanCause) {
1165         match kind {
1166             MutBorrow => {
1167                 span_err!(self.cx.tcx.sess, span, E0301,
1168                           "cannot mutably borrow in a pattern guard")
1169             }
1170             ImmBorrow | UniqueImmBorrow => {}
1171         }
1172     }
1173     fn decl_without_init(&mut self, _: NodeId, _: Span) {}
1174     fn mutate(&mut self, _: NodeId, span: Span, _: cmt, mode: MutateMode) {
1175         match mode {
1176             MutateMode::JustWrite | MutateMode::WriteAndRead => {
1177                 span_err!(self.cx.tcx.sess, span, E0302, "cannot assign in a pattern guard")
1178             }
1179             MutateMode::Init => {}
1180         }
1181     }
1182 }
1183
1184 /// Forbids bindings in `@` patterns. This is necessary for memory safety,
1185 /// because of the way rvalues are handled in the borrow check. (See issue
1186 /// #14587.)
1187 fn check_legality_of_bindings_in_at_patterns(cx: &MatchCheckCtxt, pat: &Pat) {
1188     AtBindingPatternVisitor { cx: cx, bindings_allowed: true }.visit_pat(pat);
1189 }
1190
1191 struct AtBindingPatternVisitor<'a, 'b:'a, 'tcx:'b> {
1192     cx: &'a MatchCheckCtxt<'b, 'tcx>,
1193     bindings_allowed: bool
1194 }
1195
1196 impl<'a, 'b, 'tcx, 'v> Visitor<'v> for AtBindingPatternVisitor<'a, 'b, 'tcx> {
1197     fn visit_pat(&mut self, pat: &Pat) {
1198         if !self.bindings_allowed && pat_is_binding(&self.cx.tcx.def_map.borrow(), pat) {
1199             span_err!(self.cx.tcx.sess, pat.span, E0303,
1200                                       "pattern bindings are not allowed \
1201                                        after an `@`");
1202         }
1203
1204         match pat.node {
1205             hir::PatIdent(_, _, Some(_)) => {
1206                 let bindings_were_allowed = self.bindings_allowed;
1207                 self.bindings_allowed = false;
1208                 intravisit::walk_pat(self, pat);
1209                 self.bindings_allowed = bindings_were_allowed;
1210             }
1211             _ => intravisit::walk_pat(self, pat),
1212         }
1213     }
1214 }