---

yaml
---
r: 147659
b: refs/heads/try2
c: d5e3272
h: refs/heads/master
i:
  147657: 2addd92
  147655: 23c2a4c
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: 2551344928a7ad30b2ced2b31bf6cdc2eb98554e
+refs/heads/try2: d5e32729c9346c47f9128c5fe4b6b0aacfe1d4f4
 refs/heads/dist-snap: ba4081a5a8573875fed17545846f6f6902c8ba8d
 refs/tags/release-0.2: c870d2dffb391e14efb05aa27898f1f6333a9596
 refs/tags/release-0.3: b5f0d0f648d9a6153664837026ba1be43d3e2503

branches/try2/src/librustc/middle/region.rs

@@ -25,6 +25,7 @@ use driver::session::Session;
 use middle::ty::{FreeRegion};
 use middle::ty;
 
+use std::cell::RefCell;
 use std::hashmap::{HashMap, HashSet};
 use syntax::codemap::Span;
 use syntax::{ast, visit};
@@ -50,7 +51,7 @@ The region maps encode information about region relationships.
     necessarily how I think things ought to work
 */
 pub struct RegionMaps {
-    priv scope_map: HashMap<ast::NodeId, ast::NodeId>,
+    priv scope_map: RefCell<HashMap<ast::NodeId, ast::NodeId>>,
     priv free_region_map: HashMap<FreeRegion, ~[FreeRegion]>,
     priv cleanup_scopes: HashSet<ast::NodeId>
 }
@@ -93,7 +94,8 @@ impl RegionMaps {
         debug!("record_parent(sub={:?}, sup={:?})", sub, sup);
         assert!(sub != sup);
 
-        self.scope_map.insert(sub, sup);
+        let mut scope_map = self.scope_map.borrow_mut();
+        scope_map.get().insert(sub, sup);
     }
 
     pub fn record_cleanup_scope(&mut self, scope_id: ast::NodeId) {
@@ -108,13 +110,15 @@ impl RegionMaps {
     pub fn opt_encl_scope(&self, id: ast::NodeId) -> Option<ast::NodeId> {
         //! Returns the narrowest scope that encloses `id`, if any.
 
-        self.scope_map.find(&id).map(|x| *x)
+        let scope_map = self.scope_map.borrow();
+        scope_map.get().find(&id).map(|x| *x)
     }
 
     pub fn encl_scope(&self, id: ast::NodeId) -> ast::NodeId {
         //! Returns the narrowest scope that encloses `id`, if any.
 
-        match self.scope_map.find(&id) {
+        let scope_map = self.scope_map.borrow();
+        match scope_map.get().find(&id) {
             Some(&r) => r,
             None => { fail!("No enclosing scope for id {:?}", id); }
         }
@@ -157,7 +161,8 @@ impl RegionMaps {
 
         let mut s = subscope;
         while superscope != s {
-            match self.scope_map.find(&s) {
+            let scope_map = self.scope_map.borrow();
+            match scope_map.get().find(&s) {
                 None => {
                     debug!("is_subscope_of({:?}, {:?}, s={:?})=false",
                            subscope, superscope, s);
@@ -298,7 +303,8 @@ impl RegionMaps {
             let mut result = ~[scope];
             let mut scope = scope;
             loop {
-                match this.scope_map.find(&scope) {
+                let scope_map = this.scope_map.borrow();
+                match scope_map.get().find(&scope) {
                     None => return result,
                     Some(&superscope) => {
                         result.push(superscope);
@@ -497,7 +503,7 @@ pub fn resolve_crate(sess: Session,
                      crate: &ast::Crate) -> @mut RegionMaps
 {
     let region_maps = @mut RegionMaps {
-        scope_map: HashMap::new(),
+        scope_map: RefCell::new(HashMap::new()),
         free_region_map: HashMap::new(),
         cleanup_scopes: HashSet::new(),
     };