---

yaml
---
r: 81105
b: refs/heads/snap-stage3
c: 3748164
h: refs/heads/master
i:
  81103: d38f981
v: v3

Author: pnkfelix

[refs]

@@ -1,7 +1,7 @@
 ---
 refs/heads/master: 4c6bf4872012c010f84dc7fa2cdfe87522533f89
 refs/heads/snap-stage1: e33de59e47c5076a89eadeb38f4934f58a3618a6
-refs/heads/snap-stage3: 1ce1aae28273ded3be991d1f29339ce08a5fc220
+refs/heads/snap-stage3: 37481641b1101108eb394789dd2adf9e095e39d9
 refs/heads/try: 70152ff55722878cde684ee6462c14c65f2c4729
 refs/tags/release-0.1: 1f5c5126e96c79d22cb7862f75304136e204f105
 refs/heads/ndm: f3868061cd7988080c30d6d5bf352a5a5fe2460b

branches/snap-stage3/src/librustc/middle/check_const.rs

@@ -226,7 +226,9 @@ struct env {
     idstack: @mut ~[NodeId]
 }
 
-struct CheckItemRecursionVisitor;
+struct CheckItemRecursionVisitor {
+    env: env,
+}
 
 // Make sure a const item doesn't recursively refer to itself
 // FIXME: Should use the dependency graph when it's available (#1356)
@@ -242,34 +244,34 @@ pub fn check_item_recursion(sess: Session,
         idstack: @mut ~[]
     };
 
-    let mut visitor = CheckItemRecursionVisitor;
-    visitor.visit_item(it, env);
+    let mut visitor = CheckItemRecursionVisitor { env: env };
+    visitor.visit_item(it, ());
 }
 
-impl Visitor<env> for CheckItemRecursionVisitor {
-    fn visit_item(&mut self, it: @item, env: env) {
-        if env.idstack.iter().any(|x| x == &(it.id)) {
-            env.sess.span_fatal(env.root_it.span, "recursive constant");
+impl Visitor<()> for CheckItemRecursionVisitor {
+    fn visit_item(&mut self, it: @item, _: ()) {
+        if self.env.idstack.iter().any(|x| x == &(it.id)) {
+            self.env.sess.span_fatal(self.env.root_it.span, "recursive constant");
         }
-        env.idstack.push(it.id);
-        visit::walk_item(self, it, env);
-        env.idstack.pop();
+        self.env.idstack.push(it.id);
+        visit::walk_item(self, it, ());
+        self.env.idstack.pop();
     }
 
-    fn visit_expr(&mut self, e: @Expr, env: env) {
+    fn visit_expr(&mut self, e: @Expr, _: ()) {
         match e.node {
-            ExprPath(*) => match env.def_map.find(&e.id) {
+            ExprPath(*) => match self.env.def_map.find(&e.id) {
                 Some(&DefStatic(def_id, _)) if ast_util::is_local(def_id) =>
-                    match env.ast_map.get_copy(&def_id.node) {
+                    match self.env.ast_map.get_copy(&def_id.node) {
                         ast_map::node_item(it, _) => {
-                            self.visit_item(it, env);
+                            self.visit_item(it, ());
                         }
                         _ => fail!("const not bound to an item")
                     },
                 _ => ()
             },
             _ => ()
         }
-        visit::walk_expr(self, e, env);
+        visit::walk_expr(self, e, ());
     }
 }