---

yaml
---
r: 89087
b: refs/heads/snap-stage3
c: eaf6949
h: refs/heads/master
i:
  89085: 387b7fa
  89083: 381a585
  89079: 27beea3
  89071: 70a9786
  89055: 918a05e
  89023: 8e96055
  88959: fa97590
  88831: b60752b
  88575: fc71c77
  88063: c522f9e
v: v3

Author: pcwalton

[refs]

@@ -1,7 +1,7 @@
 ---
 refs/heads/master: deeca5d586bfaa4aa60246f671a8d611d38f6248
 refs/heads/snap-stage1: e33de59e47c5076a89eadeb38f4934f58a3618a6
-refs/heads/snap-stage3: 298d0b870ff9201ee05c5517ade88da84a61cdf1
+refs/heads/snap-stage3: eaf69494a5a126189b052102e1b36d1d8bced953
 refs/heads/try: b160761e35efcd1207112b3b782c06633cf441a8
 refs/tags/release-0.1: 1f5c5126e96c79d22cb7862f75304136e204f105
 refs/heads/ndm: f3868061cd7988080c30d6d5bf352a5a5fe2460b

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

@@ -594,7 +594,7 @@ pub struct Liveness {
     users: @mut ~[Users],
     // The list of node IDs for the nested loop scopes
     // we're in.
-    loop_scope: @mut ~[NodeId],
+    loop_scope: @RefCell<~[NodeId]>,
     // mappings from loop node ID to LiveNode
     // ("break" label should map to loop node ID,
     // it probably doesn't now)
@@ -612,7 +612,7 @@ fn Liveness(ir: @IrMaps, specials: Specials) -> Liveness {
         users: @mut vec::from_elem(ir.num_live_nodes.get() *
                                    ir.num_vars.get(),
                                    invalid_users()),
-        loop_scope: @mut ~[],
+        loop_scope: @RefCell::new(~[]),
         break_ln: @RefCell::new(HashMap::new()),
         cont_ln: @RefCell::new(HashMap::new()),
     }
@@ -762,7 +762,8 @@ impl Liveness {
             None => {
                 // Vanilla 'break' or 'loop', so use the enclosing
                 // loop scope
-                if self.loop_scope.len() == 0 {
+                let loop_scope = self.loop_scope.borrow();
+                if loop_scope.get().len() == 0 {
                     self.tcx.sess.span_bug(sp, "break outside loop");
                 } else {
                     // FIXME(#5275): this shouldn't have to be a method...
@@ -773,8 +774,8 @@ impl Liveness {
     }
 
     pub fn last_loop_scope(&self) -> NodeId {
-        let loop_scope = &mut *self.loop_scope;
-        *loop_scope.last()
+        let loop_scope = self.loop_scope.borrow();
+        *loop_scope.get().last()
     }
 
     pub fn ln_str(&self, ln: LiveNode) -> ~str {
@@ -1418,16 +1419,22 @@ impl Liveness {
                            cont_ln: LiveNode,
                            f: || -> R)
                            -> R {
-      debug!("with_loop_nodes: {} {}", loop_node_id, *break_ln);
-        self.loop_scope.push(loop_node_id);
+        debug!("with_loop_nodes: {} {}", loop_node_id, *break_ln);
+        {
+            let mut loop_scope = self.loop_scope.borrow_mut();
+            loop_scope.get().push(loop_node_id);
+        }
         {
             let mut this_break_ln = self.break_ln.borrow_mut();
             let mut this_cont_ln = self.cont_ln.borrow_mut();
             this_break_ln.get().insert(loop_node_id, break_ln);
             this_cont_ln.get().insert(loop_node_id, cont_ln);
         }
         let r = f();
-        self.loop_scope.pop();
+        {
+            let mut loop_scope = self.loop_scope.borrow_mut();
+            loop_scope.get().pop();
+        }
         r
     }
 }