---

yaml
---
r: 97103
b: refs/heads/dist-snap
c: d9c87c7
h: refs/heads/master
i:
  97101: b5b2580
  97099: 751646d
  97095: 8d91bd9
  97087: 4879f2a
v: v3

Author: pcwalton

[refs]

@@ -6,7 +6,7 @@ refs/heads/try: c274a6888410ce3e357e014568b43310ed787d36
 refs/tags/release-0.1: 1f5c5126e96c79d22cb7862f75304136e204f105
 refs/heads/ndm: f3868061cd7988080c30d6d5bf352a5a5fe2460b
 refs/heads/try2: 147ecfdd8221e4a4d4e090486829a06da1e0ca3c
-refs/heads/dist-snap: f7393d86580b6b417063b48c9ad4c03829d03713
+refs/heads/dist-snap: d9c87c7a2374dbdc5d5662af8548e95bbb616584
 refs/tags/release-0.2: c870d2dffb391e14efb05aa27898f1f6333a9596
 refs/tags/release-0.3: b5f0d0f648d9a6153664837026ba1be43d3e2503
 refs/heads/try3: 9387340aab40a73e8424c48fd42f0c521a4875c0

branches/dist-snap/src/librustc/middle/borrowck/mod.rs

@@ -475,7 +475,8 @@ impl BorrowckCtxt {
     }
 
     pub fn is_move(&self, id: ast::NodeId) -> bool {
-        self.moves_map.contains(&id)
+        let moves_map = self.moves_map.borrow();
+        moves_map.get().contains(&id)
     }
 
     pub fn cat_expr(&self, expr: @ast::Expr) -> mc::cmt {

branches/dist-snap/src/librustc/middle/check_match.rs

@@ -889,7 +889,8 @@ fn check_legality_of_move_bindings(cx: &MatchCheckCtxt,
                     by_ref_span = Some(span);
                 }
                 BindByValue(_) => {
-                    if cx.moves_map.contains(&id) {
+                    let moves_map = cx.moves_map.borrow();
+                    if moves_map.get().contains(&id) {
                         any_by_move = true;
                     }
                 }
@@ -926,7 +927,8 @@ fn check_legality_of_move_bindings(cx: &MatchCheckCtxt,
             if pat_is_binding(def_map, p) {
                 match p.node {
                     PatIdent(_, _, sub) => {
-                        if cx.moves_map.contains(&p.id) {
+                        let moves_map = cx.moves_map.borrow();
+                        if moves_map.get().contains(&p.id) {
                             check_move(p, sub);
                         }
                     }

branches/dist-snap/src/librustc/middle/moves.rs

@@ -161,7 +161,7 @@ pub struct CaptureVar {
 
 pub type CaptureMap = @RefCell<HashMap<NodeId, @[CaptureVar]>>;
 
-pub type MovesMap = @mut HashSet<NodeId>;
+pub type MovesMap = @RefCell<HashSet<NodeId>>;
 
 /**
  * Set of variable node-ids that are moved.
@@ -215,7 +215,7 @@ pub fn compute_moves(tcx: ty::ctxt,
         tcx: tcx,
         method_map: method_map,
         move_maps: MoveMaps {
-            moves_map: @mut HashSet::new(),
+            moves_map: @RefCell::new(HashSet::new()),
             capture_map: @RefCell::new(HashMap::new()),
             moved_variables_set: @mut HashSet::new()
         }
@@ -283,7 +283,10 @@ impl VisitContext {
 
         let expr_ty = ty::expr_ty_adjusted(self.tcx, expr);
         if ty::type_moves_by_default(self.tcx, expr_ty) {
-            self.move_maps.moves_map.insert(expr.id);
+            {
+                let mut moves_map = self.move_maps.moves_map.borrow_mut();
+                moves_map.get().insert(expr.id);
+            }
             self.use_expr(expr, Move);
         } else {
             self.use_expr(expr, Read);
@@ -388,7 +391,12 @@ impl VisitContext {
                 // closures should be noncopyable, they shouldn't move by default;
                 // calling a closure should only consume it if it's once.
                 if mode == Move {
-                    self.move_maps.moves_map.insert(callee.id);
+                    {
+                        let mut moves_map = self.move_maps
+                                                .moves_map
+                                                .borrow_mut();
+                        moves_map.get().insert(callee.id);
+                    }
                 }
                 self.use_expr(callee, mode);
                 self.use_fn_args(callee.id, *args);
@@ -643,7 +651,10 @@ impl VisitContext {
                    id, bm, binding_moves);
 
             if binding_moves {
-                self.move_maps.moves_map.insert(id);
+                {
+                    let mut moves_map = self.move_maps.moves_map.borrow_mut();
+                    moves_map.get().insert(id);
+                }
             }
         })
     }