]> git.lizzy.rs Git - rust.git/blob - clippy_lints/src/utils/hir_utils.rs
Merge pull request #2021 from marcusklaas/needless-loop-2
[rust.git] / clippy_lints / src / utils / hir_utils.rs
1 use consts::constant;
2 use rustc::lint::*;
3 use rustc::hir::*;
4 use std::hash::{Hash, Hasher};
5 use std::collections::hash_map::DefaultHasher;
6 use syntax::ast::Name;
7 use syntax::ptr::P;
8 use utils::differing_macro_contexts;
9
10 /// Type used to check whether two ast are the same. This is different from the
11 /// operator
12 /// `==` on ast types as this operator would compare true equality with ID and
13 /// span.
14 ///
15 /// Note that some expressions kinds are not considered but could be added.
16 pub struct SpanlessEq<'a, 'tcx: 'a> {
17     /// Context used to evaluate constant expressions.
18     cx: &'a LateContext<'a, 'tcx>,
19     /// If is true, never consider as equal expressions containing function
20     /// calls.
21     ignore_fn: bool,
22 }
23
24 impl<'a, 'tcx: 'a> SpanlessEq<'a, 'tcx> {
25     pub fn new(cx: &'a LateContext<'a, 'tcx>) -> Self {
26         Self {
27             cx: cx,
28             ignore_fn: false,
29         }
30     }
31
32     pub fn ignore_fn(self) -> Self {
33         Self {
34             cx: self.cx,
35             ignore_fn: true,
36         }
37     }
38
39     /// Check whether two statements are the same.
40     pub fn eq_stmt(&self, left: &Stmt, right: &Stmt) -> bool {
41         match (&left.node, &right.node) {
42             (&StmtDecl(ref l, _), &StmtDecl(ref r, _)) => {
43                 if let (&DeclLocal(ref l), &DeclLocal(ref r)) = (&l.node, &r.node) {
44                     both(&l.ty, &r.ty, |l, r| self.eq_ty(l, r)) && both(&l.init, &r.init, |l, r| self.eq_expr(l, r))
45                 } else {
46                     false
47                 }
48             },
49             (&StmtExpr(ref l, _), &StmtExpr(ref r, _)) | (&StmtSemi(ref l, _), &StmtSemi(ref r, _)) => {
50                 self.eq_expr(l, r)
51             },
52             _ => false,
53         }
54     }
55
56     /// Check whether two blocks are the same.
57     pub fn eq_block(&self, left: &Block, right: &Block) -> bool {
58         over(&left.stmts, &right.stmts, |l, r| self.eq_stmt(l, r)) &&
59             both(&left.expr, &right.expr, |l, r| self.eq_expr(l, r))
60     }
61
62     pub fn eq_expr(&self, left: &Expr, right: &Expr) -> bool {
63         if self.ignore_fn && differing_macro_contexts(left.span, right.span) {
64             return false;
65         }
66
67         if let (Some(l), Some(r)) = (constant(self.cx, left), constant(self.cx, right)) {
68             if l == r {
69                 return true;
70             }
71         }
72
73         match (&left.node, &right.node) {
74             (&ExprAddrOf(l_mut, ref le), &ExprAddrOf(r_mut, ref re)) => l_mut == r_mut && self.eq_expr(le, re),
75             (&ExprAgain(li), &ExprAgain(ri)) => {
76                 both(&li.ident, &ri.ident, |l, r| l.node.name.as_str() == r.node.name.as_str())
77             },
78             (&ExprAssign(ref ll, ref lr), &ExprAssign(ref rl, ref rr)) => self.eq_expr(ll, rl) && self.eq_expr(lr, rr),
79             (&ExprAssignOp(ref lo, ref ll, ref lr), &ExprAssignOp(ref ro, ref rl, ref rr)) => {
80                 lo.node == ro.node && self.eq_expr(ll, rl) && self.eq_expr(lr, rr)
81             },
82             (&ExprBlock(ref l), &ExprBlock(ref r)) => self.eq_block(l, r),
83             (&ExprBinary(l_op, ref ll, ref lr), &ExprBinary(r_op, ref rl, ref rr)) => {
84                 l_op.node == r_op.node && self.eq_expr(ll, rl) && self.eq_expr(lr, rr) ||
85                     swap_binop(l_op.node, ll, lr).map_or(false, |(l_op, ll, lr)| {
86                         l_op == r_op.node && self.eq_expr(ll, rl) && self.eq_expr(lr, rr)
87                     })
88             },
89             (&ExprBreak(li, ref le), &ExprBreak(ri, ref re)) => {
90                 both(&li.ident, &ri.ident, |l, r| l.node.name.as_str() == r.node.name.as_str()) &&
91                     both(le, re, |l, r| self.eq_expr(l, r))
92             },
93             (&ExprBox(ref l), &ExprBox(ref r)) => self.eq_expr(l, r),
94             (&ExprCall(ref l_fun, ref l_args), &ExprCall(ref r_fun, ref r_args)) => {
95                 !self.ignore_fn && self.eq_expr(l_fun, r_fun) && self.eq_exprs(l_args, r_args)
96             },
97             (&ExprCast(ref lx, ref lt), &ExprCast(ref rx, ref rt)) |
98             (&ExprType(ref lx, ref lt), &ExprType(ref rx, ref rt)) => self.eq_expr(lx, rx) && self.eq_ty(lt, rt),
99             (&ExprField(ref l_f_exp, ref l_f_ident), &ExprField(ref r_f_exp, ref r_f_ident)) => {
100                 l_f_ident.node == r_f_ident.node && self.eq_expr(l_f_exp, r_f_exp)
101             },
102             (&ExprIndex(ref la, ref li), &ExprIndex(ref ra, ref ri)) => self.eq_expr(la, ra) && self.eq_expr(li, ri),
103             (&ExprIf(ref lc, ref lt, ref le), &ExprIf(ref rc, ref rt, ref re)) => {
104                 self.eq_expr(lc, rc) && self.eq_expr(&**lt, &**rt) && both(le, re, |l, r| self.eq_expr(l, r))
105             },
106             (&ExprLit(ref l), &ExprLit(ref r)) => l.node == r.node,
107             (&ExprLoop(ref lb, ref ll, ref lls), &ExprLoop(ref rb, ref rl, ref rls)) => {
108                 lls == rls && self.eq_block(lb, rb) && both(ll, rl, |l, r| l.node.as_str() == r.node.as_str())
109             },
110             (&ExprMatch(ref le, ref la, ref ls), &ExprMatch(ref re, ref ra, ref rs)) => {
111                 ls == rs && self.eq_expr(le, re) && over(la, ra, |l, r| {
112                     self.eq_expr(&l.body, &r.body) && both(&l.guard, &r.guard, |l, r| self.eq_expr(l, r)) &&
113                         over(&l.pats, &r.pats, |l, r| self.eq_pat(l, r))
114                 })
115             },
116             (&ExprMethodCall(ref l_path, _, ref l_args), &ExprMethodCall(ref r_path, _, ref r_args)) => {
117                 !self.ignore_fn && l_path == r_path && self.eq_exprs(l_args, r_args)
118             },
119             (&ExprRepeat(ref le, ll_id), &ExprRepeat(ref re, rl_id)) => {
120                 self.eq_expr(le, re) &&
121                     self.eq_expr(&self.cx.tcx.hir.body(ll_id).value, &self.cx.tcx.hir.body(rl_id).value)
122             },
123             (&ExprRet(ref l), &ExprRet(ref r)) => both(l, r, |l, r| self.eq_expr(l, r)),
124             (&ExprPath(ref l), &ExprPath(ref r)) => self.eq_qpath(l, r),
125             (&ExprStruct(ref l_path, ref lf, ref lo), &ExprStruct(ref r_path, ref rf, ref ro)) => {
126                 self.eq_qpath(l_path, r_path) && both(lo, ro, |l, r| self.eq_expr(l, r)) &&
127                     over(lf, rf, |l, r| self.eq_field(l, r))
128             },
129             (&ExprTup(ref l_tup), &ExprTup(ref r_tup)) => self.eq_exprs(l_tup, r_tup),
130             (&ExprTupField(ref le, li), &ExprTupField(ref re, ri)) => li.node == ri.node && self.eq_expr(le, re),
131             (&ExprUnary(l_op, ref le), &ExprUnary(r_op, ref re)) => l_op == r_op && self.eq_expr(le, re),
132             (&ExprArray(ref l), &ExprArray(ref r)) => self.eq_exprs(l, r),
133             (&ExprWhile(ref lc, ref lb, ref ll), &ExprWhile(ref rc, ref rb, ref rl)) => {
134                 self.eq_expr(lc, rc) && self.eq_block(lb, rb) && both(ll, rl, |l, r| l.node.as_str() == r.node.as_str())
135             },
136             _ => false,
137         }
138     }
139
140     fn eq_exprs(&self, left: &P<[Expr]>, right: &P<[Expr]>) -> bool {
141         over(left, right, |l, r| self.eq_expr(l, r))
142     }
143
144     fn eq_field(&self, left: &Field, right: &Field) -> bool {
145         left.name.node == right.name.node && self.eq_expr(&left.expr, &right.expr)
146     }
147
148     fn eq_lifetime(&self, left: &Lifetime, right: &Lifetime) -> bool {
149         left.name == right.name
150     }
151
152     /// Check whether two patterns are the same.
153     pub fn eq_pat(&self, left: &Pat, right: &Pat) -> bool {
154         match (&left.node, &right.node) {
155             (&PatKind::Box(ref l), &PatKind::Box(ref r)) => self.eq_pat(l, r),
156             (&PatKind::TupleStruct(ref lp, ref la, ls), &PatKind::TupleStruct(ref rp, ref ra, rs)) => {
157                 self.eq_qpath(lp, rp) && over(la, ra, |l, r| self.eq_pat(l, r)) && ls == rs
158             },
159             (&PatKind::Binding(ref lb, _, ref li, ref lp), &PatKind::Binding(ref rb, _, ref ri, ref rp)) => {
160                 lb == rb && li.node.as_str() == ri.node.as_str() && both(lp, rp, |l, r| self.eq_pat(l, r))
161             },
162             (&PatKind::Path(ref l), &PatKind::Path(ref r)) => self.eq_qpath(l, r),
163             (&PatKind::Lit(ref l), &PatKind::Lit(ref r)) => self.eq_expr(l, r),
164             (&PatKind::Tuple(ref l, ls), &PatKind::Tuple(ref r, rs)) => {
165                 ls == rs && over(l, r, |l, r| self.eq_pat(l, r))
166             },
167             (&PatKind::Range(ref ls, ref le, ref li), &PatKind::Range(ref rs, ref re, ref ri)) => {
168                 self.eq_expr(ls, rs) && self.eq_expr(le, re) && (*li == *ri)
169             },
170             (&PatKind::Ref(ref le, ref lm), &PatKind::Ref(ref re, ref rm)) => lm == rm && self.eq_pat(le, re),
171             (&PatKind::Slice(ref ls, ref li, ref le), &PatKind::Slice(ref rs, ref ri, ref re)) => {
172                 over(ls, rs, |l, r| self.eq_pat(l, r)) && over(le, re, |l, r| self.eq_pat(l, r)) &&
173                     both(li, ri, |l, r| self.eq_pat(l, r))
174             },
175             (&PatKind::Wild, &PatKind::Wild) => true,
176             _ => false,
177         }
178     }
179
180     fn eq_qpath(&self, left: &QPath, right: &QPath) -> bool {
181         match (left, right) {
182             (&QPath::Resolved(ref lty, ref lpath), &QPath::Resolved(ref rty, ref rpath)) => {
183                 both(lty, rty, |l, r| self.eq_ty(l, r)) && self.eq_path(lpath, rpath)
184             },
185             (&QPath::TypeRelative(ref lty, ref lseg), &QPath::TypeRelative(ref rty, ref rseg)) => {
186                 self.eq_ty(lty, rty) && self.eq_path_segment(lseg, rseg)
187             },
188             _ => false,
189         }
190     }
191
192     fn eq_path(&self, left: &Path, right: &Path) -> bool {
193         left.is_global() == right.is_global() &&
194             over(&left.segments, &right.segments, |l, r| self.eq_path_segment(l, r))
195     }
196
197     fn eq_path_parameters(&self, left: &PathParameters, right: &PathParameters) -> bool {
198         if !(left.parenthesized || right.parenthesized) {
199             over(&left.lifetimes, &right.lifetimes, |l, r| self.eq_lifetime(l, r)) &&
200                 over(&left.types, &right.types, |l, r| self.eq_ty(l, r)) &&
201                 over(&left.bindings, &right.bindings, |l, r| self.eq_type_binding(l, r))
202         } else if left.parenthesized && right.parenthesized {
203             over(left.inputs(), right.inputs(), |l, r| self.eq_ty(l, r)) &&
204                 both(
205                     &Some(&left.bindings[0].ty),
206                     &Some(&right.bindings[0].ty),
207                     |l, r| self.eq_ty(l, r),
208                 )
209         } else {
210             false
211         }
212     }
213
214     fn eq_path_segment(&self, left: &PathSegment, right: &PathSegment) -> bool {
215         // The == of idents doesn't work with different contexts,
216         // we have to be explicit about hygiene
217         left.name.as_str() == right.name.as_str() && self.eq_path_parameters(&left.parameters, &right.parameters)
218     }
219
220     fn eq_ty(&self, left: &Ty, right: &Ty) -> bool {
221         match (&left.node, &right.node) {
222             (&TySlice(ref l_vec), &TySlice(ref r_vec)) => self.eq_ty(l_vec, r_vec),
223             (&TyArray(ref lt, ll_id), &TyArray(ref rt, rl_id)) => {
224                 self.eq_ty(lt, rt) &&
225                     self.eq_expr(&self.cx.tcx.hir.body(ll_id).value, &self.cx.tcx.hir.body(rl_id).value)
226             },
227             (&TyPtr(ref l_mut), &TyPtr(ref r_mut)) => l_mut.mutbl == r_mut.mutbl && self.eq_ty(&*l_mut.ty, &*r_mut.ty),
228             (&TyRptr(_, ref l_rmut), &TyRptr(_, ref r_rmut)) => {
229                 l_rmut.mutbl == r_rmut.mutbl && self.eq_ty(&*l_rmut.ty, &*r_rmut.ty)
230             },
231             (&TyPath(ref l), &TyPath(ref r)) => self.eq_qpath(l, r),
232             (&TyTup(ref l), &TyTup(ref r)) => over(l, r, |l, r| self.eq_ty(l, r)),
233             (&TyInfer, &TyInfer) => true,
234             _ => false,
235         }
236     }
237
238     fn eq_type_binding(&self, left: &TypeBinding, right: &TypeBinding) -> bool {
239         left.name == right.name && self.eq_ty(&left.ty, &right.ty)
240     }
241 }
242
243 fn swap_binop<'a>(binop: BinOp_, lhs: &'a Expr, rhs: &'a Expr) -> Option<(BinOp_, &'a Expr, &'a Expr)> {
244     match binop {
245         BiAdd | BiMul | BiBitXor | BiBitAnd | BiEq | BiNe | BiBitOr => Some((binop, rhs, lhs)),
246         BiLt => Some((BiGt, rhs, lhs)),
247         BiLe => Some((BiGe, rhs, lhs)),
248         BiGe => Some((BiLe, rhs, lhs)),
249         BiGt => Some((BiLt, rhs, lhs)),
250         BiShl | BiShr | BiRem | BiSub | BiDiv | BiAnd | BiOr => None,
251     }
252 }
253
254 /// Check if the two `Option`s are both `None` or some equal values as per
255 /// `eq_fn`.
256 fn both<X, F>(l: &Option<X>, r: &Option<X>, mut eq_fn: F) -> bool
257 where
258     F: FnMut(&X, &X) -> bool,
259 {
260     l.as_ref()
261         .map_or_else(|| r.is_none(), |x| r.as_ref().map_or(false, |y| eq_fn(x, y)))
262 }
263
264 /// Check if two slices are equal as per `eq_fn`.
265 fn over<X, F>(left: &[X], right: &[X], mut eq_fn: F) -> bool
266 where
267     F: FnMut(&X, &X) -> bool,
268 {
269     left.len() == right.len() && left.iter().zip(right).all(|(x, y)| eq_fn(x, y))
270 }
271
272
273 /// Type used to hash an ast element. This is different from the `Hash` trait
274 /// on ast types as this
275 /// trait would consider IDs and spans.
276 ///
277 /// All expressions kind are hashed, but some might have a weaker hash.
278 pub struct SpanlessHash<'a, 'tcx: 'a> {
279     /// Context used to evaluate constant expressions.
280     cx: &'a LateContext<'a, 'tcx>,
281     s: DefaultHasher,
282 }
283
284 impl<'a, 'tcx: 'a> SpanlessHash<'a, 'tcx> {
285     pub fn new(cx: &'a LateContext<'a, 'tcx>) -> Self {
286         Self {
287             cx: cx,
288             s: DefaultHasher::new(),
289         }
290     }
291
292     pub fn finish(&self) -> u64 {
293         self.s.finish()
294     }
295
296     pub fn hash_block(&mut self, b: &Block) {
297         for s in &b.stmts {
298             self.hash_stmt(s);
299         }
300
301         if let Some(ref e) = b.expr {
302             self.hash_expr(e);
303         }
304
305         b.rules.hash(&mut self.s);
306     }
307
308     pub fn hash_expr(&mut self, e: &Expr) {
309         if let Some(e) = constant(self.cx, e) {
310             return e.hash(&mut self.s);
311         }
312
313         match e.node {
314             ExprAddrOf(m, ref e) => {
315                 let c: fn(_, _) -> _ = ExprAddrOf;
316                 c.hash(&mut self.s);
317                 m.hash(&mut self.s);
318                 self.hash_expr(e);
319             },
320             ExprAgain(i) => {
321                 let c: fn(_) -> _ = ExprAgain;
322                 c.hash(&mut self.s);
323                 if let Some(i) = i.ident {
324                     self.hash_name(&i.node.name);
325                 }
326             },
327             ExprYield(ref e) => {
328                 let c: fn(_) -> _ = ExprYield;
329                 c.hash(&mut self.s);
330                 self.hash_expr(e);
331             },
332             ExprAssign(ref l, ref r) => {
333                 let c: fn(_, _) -> _ = ExprAssign;
334                 c.hash(&mut self.s);
335                 self.hash_expr(l);
336                 self.hash_expr(r);
337             },
338             ExprAssignOp(ref o, ref l, ref r) => {
339                 let c: fn(_, _, _) -> _ = ExprAssignOp;
340                 c.hash(&mut self.s);
341                 o.hash(&mut self.s);
342                 self.hash_expr(l);
343                 self.hash_expr(r);
344             },
345             ExprBlock(ref b) => {
346                 let c: fn(_) -> _ = ExprBlock;
347                 c.hash(&mut self.s);
348                 self.hash_block(b);
349             },
350             ExprBinary(op, ref l, ref r) => {
351                 let c: fn(_, _, _) -> _ = ExprBinary;
352                 c.hash(&mut self.s);
353                 op.node.hash(&mut self.s);
354                 self.hash_expr(l);
355                 self.hash_expr(r);
356             },
357             ExprBreak(i, ref j) => {
358                 let c: fn(_, _) -> _ = ExprBreak;
359                 c.hash(&mut self.s);
360                 if let Some(i) = i.ident {
361                     self.hash_name(&i.node.name);
362                 }
363                 if let Some(ref j) = *j {
364                     self.hash_expr(&*j);
365                 }
366             },
367             ExprBox(ref e) => {
368                 let c: fn(_) -> _ = ExprBox;
369                 c.hash(&mut self.s);
370                 self.hash_expr(e);
371             },
372             ExprCall(ref fun, ref args) => {
373                 let c: fn(_, _) -> _ = ExprCall;
374                 c.hash(&mut self.s);
375                 self.hash_expr(fun);
376                 self.hash_exprs(args);
377             },
378             ExprCast(ref e, ref _ty) => {
379                 let c: fn(_, _) -> _ = ExprCast;
380                 c.hash(&mut self.s);
381                 self.hash_expr(e);
382                 // TODO: _ty
383             },
384             ExprClosure(cap, _, eid, _, _) => {
385                 let c: fn(_, _, _, _, _) -> _ = ExprClosure;
386                 c.hash(&mut self.s);
387                 cap.hash(&mut self.s);
388                 self.hash_expr(&self.cx.tcx.hir.body(eid).value);
389             },
390             ExprField(ref e, ref f) => {
391                 let c: fn(_, _) -> _ = ExprField;
392                 c.hash(&mut self.s);
393                 self.hash_expr(e);
394                 self.hash_name(&f.node);
395             },
396             ExprIndex(ref a, ref i) => {
397                 let c: fn(_, _) -> _ = ExprIndex;
398                 c.hash(&mut self.s);
399                 self.hash_expr(a);
400                 self.hash_expr(i);
401             },
402             ExprInlineAsm(..) => {
403                 let c: fn(_, _, _) -> _ = ExprInlineAsm;
404                 c.hash(&mut self.s);
405             },
406             ExprIf(ref cond, ref t, ref e) => {
407                 let c: fn(_, _, _) -> _ = ExprIf;
408                 c.hash(&mut self.s);
409                 self.hash_expr(cond);
410                 self.hash_expr(&**t);
411                 if let Some(ref e) = *e {
412                     self.hash_expr(e);
413                 }
414             },
415             ExprLit(ref l) => {
416                 let c: fn(_) -> _ = ExprLit;
417                 c.hash(&mut self.s);
418                 l.hash(&mut self.s);
419             },
420             ExprLoop(ref b, ref i, _) => {
421                 let c: fn(_, _, _) -> _ = ExprLoop;
422                 c.hash(&mut self.s);
423                 self.hash_block(b);
424                 if let Some(i) = *i {
425                     self.hash_name(&i.node);
426                 }
427             },
428             ExprMatch(ref e, ref arms, ref s) => {
429                 let c: fn(_, _, _) -> _ = ExprMatch;
430                 c.hash(&mut self.s);
431                 self.hash_expr(e);
432
433                 for arm in arms {
434                     // TODO: arm.pat?
435                     if let Some(ref e) = arm.guard {
436                         self.hash_expr(e);
437                     }
438                     self.hash_expr(&arm.body);
439                 }
440
441                 s.hash(&mut self.s);
442             },
443             ExprMethodCall(ref path, ref _tys, ref args) => {
444                 let c: fn(_, _, _) -> _ = ExprMethodCall;
445                 c.hash(&mut self.s);
446                 self.hash_name(&path.name);
447                 self.hash_exprs(args);
448             },
449             ExprRepeat(ref e, l_id) => {
450                 let c: fn(_, _) -> _ = ExprRepeat;
451                 c.hash(&mut self.s);
452                 self.hash_expr(e);
453                 self.hash_expr(&self.cx.tcx.hir.body(l_id).value);
454             },
455             ExprRet(ref e) => {
456                 let c: fn(_) -> _ = ExprRet;
457                 c.hash(&mut self.s);
458                 if let Some(ref e) = *e {
459                     self.hash_expr(e);
460                 }
461             },
462             ExprPath(ref qpath) => {
463                 let c: fn(_) -> _ = ExprPath;
464                 c.hash(&mut self.s);
465                 self.hash_qpath(qpath);
466             },
467             ExprStruct(ref path, ref fields, ref expr) => {
468                 let c: fn(_, _, _) -> _ = ExprStruct;
469                 c.hash(&mut self.s);
470
471                 self.hash_qpath(path);
472
473                 for f in fields {
474                     self.hash_name(&f.name.node);
475                     self.hash_expr(&f.expr);
476                 }
477
478                 if let Some(ref e) = *expr {
479                     self.hash_expr(e);
480                 }
481             },
482             ExprTup(ref tup) => {
483                 let c: fn(_) -> _ = ExprTup;
484                 c.hash(&mut self.s);
485                 self.hash_exprs(tup);
486             },
487             ExprTupField(ref le, li) => {
488                 let c: fn(_, _) -> _ = ExprTupField;
489                 c.hash(&mut self.s);
490
491                 self.hash_expr(le);
492                 li.node.hash(&mut self.s);
493             },
494             ExprType(ref e, ref _ty) => {
495                 let c: fn(_, _) -> _ = ExprType;
496                 c.hash(&mut self.s);
497                 self.hash_expr(e);
498                 // TODO: _ty
499             },
500             ExprUnary(lop, ref le) => {
501                 let c: fn(_, _) -> _ = ExprUnary;
502                 c.hash(&mut self.s);
503
504                 lop.hash(&mut self.s);
505                 self.hash_expr(le);
506             },
507             ExprArray(ref v) => {
508                 let c: fn(_) -> _ = ExprArray;
509                 c.hash(&mut self.s);
510
511                 self.hash_exprs(v);
512             },
513             ExprWhile(ref cond, ref b, l) => {
514                 let c: fn(_, _, _) -> _ = ExprWhile;
515                 c.hash(&mut self.s);
516
517                 self.hash_expr(cond);
518                 self.hash_block(b);
519                 if let Some(l) = l {
520                     self.hash_name(&l.node);
521                 }
522             },
523         }
524     }
525
526     pub fn hash_exprs(&mut self, e: &P<[Expr]>) {
527         for e in e {
528             self.hash_expr(e);
529         }
530     }
531
532     pub fn hash_name(&mut self, n: &Name) {
533         n.as_str().hash(&mut self.s);
534     }
535
536     pub fn hash_qpath(&mut self, p: &QPath) {
537         match *p {
538             QPath::Resolved(_, ref path) => {
539                 self.hash_path(path);
540             },
541             QPath::TypeRelative(_, ref path) => {
542                 self.hash_name(&path.name);
543             },
544         }
545         // self.cx.tables.qpath_def(p, id).hash(&mut self.s);
546     }
547
548     pub fn hash_path(&mut self, p: &Path) {
549         p.is_global().hash(&mut self.s);
550         for p in &p.segments {
551             self.hash_name(&p.name);
552         }
553     }
554
555     pub fn hash_stmt(&mut self, b: &Stmt) {
556         match b.node {
557             StmtDecl(ref decl, _) => {
558                 let c: fn(_, _) -> _ = StmtDecl;
559                 c.hash(&mut self.s);
560
561                 if let DeclLocal(ref local) = decl.node {
562                     if let Some(ref init) = local.init {
563                         self.hash_expr(init);
564                     }
565                 }
566             },
567             StmtExpr(ref expr, _) => {
568                 let c: fn(_, _) -> _ = StmtExpr;
569                 c.hash(&mut self.s);
570                 self.hash_expr(expr);
571             },
572             StmtSemi(ref expr, _) => {
573                 let c: fn(_, _) -> _ = StmtSemi;
574                 c.hash(&mut self.s);
575                 self.hash_expr(expr);
576             },
577         }
578     }
579 }