---

yaml
---
r: 147703
b: refs/heads/try2
c: eaf6949
h: refs/heads/master
i:
  147701: 4d13796
  147699: 311282f
  147695: 0c79be4
v: v3

Author: pcwalton

[refs]

@@ -5,7 +5,7 @@ refs/heads/snap-stage3: 78a7676898d9f80ab540c6df5d4c9ce35bb50463
 refs/heads/try: 519addf6277dbafccbb4159db4b710c37eaa2ec5
 refs/tags/release-0.1: 1f5c5126e96c79d22cb7862f75304136e204f105
 refs/heads/ndm: f3868061cd7988080c30d6d5bf352a5a5fe2460b
-refs/heads/try2: 298d0b870ff9201ee05c5517ade88da84a61cdf1
+refs/heads/try2: eaf69494a5a126189b052102e1b36d1d8bced953
 refs/heads/dist-snap: ba4081a5a8573875fed17545846f6f6902c8ba8d
 refs/tags/release-0.2: c870d2dffb391e14efb05aa27898f1f6333a9596
 refs/tags/release-0.3: b5f0d0f648d9a6153664837026ba1be43d3e2503

branches/try2/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
     }
 }