---

yaml
---
r: 147612
b: refs/heads/try2
c: f73dee1
h: refs/heads/master
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: c5f07cfc1adfb23523c986b1c7e1b099ebbd8605
+refs/heads/try2: f73dee17fc787cbcf022c74ec17d00329ddfe223
 refs/heads/dist-snap: ba4081a5a8573875fed17545846f6f6902c8ba8d
 refs/tags/release-0.2: c870d2dffb391e14efb05aa27898f1f6333a9596
 refs/tags/release-0.3: b5f0d0f648d9a6153664837026ba1be43d3e2503

branches/try2/src/librustc/driver/driver.rs

@@ -272,8 +272,7 @@ pub fn phase_3_run_analysis_passes(sess: Session,
                                           method_map, ty_cx));
 
     let maps = (external_exports, last_private_map);
-    let exported_items =
-        time(time_passes, "privacy checking", maps, |(a, b)|
+    let exported_items = time(time_passes, "privacy checking", maps, |(a, b)|
              middle::privacy::check_crate(ty_cx, &method_map, &exp_map2,
                                           a, b, crate));
 

branches/try2/src/librustc/metadata/encoder.rs

@@ -556,7 +556,8 @@ fn encode_reexports(ecx: &EncodeContext,
                     id: NodeId,
                     path: &[ast_map::path_elt]) {
     debug!("(encoding info for module) encoding reexports for {}", id);
-    match ecx.reexports2.find(&id) {
+    let reexports2 = ecx.reexports2.borrow();
+    match reexports2.get().find(&id) {
         Some(ref exports) => {
             debug!("(encoding info for module) found reexports for {}", id);
             for exp in exports.iter() {

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

@@ -293,8 +293,9 @@ impl<'a> Visitor<()> for EmbargoVisitor<'a> {
         // This code is here instead of in visit_item so that the
         // crate module gets processed as well.
         if self.prev_exported {
-            assert!(self.exp_map2.contains_key(&id), "wut {:?}", id);
-            for export in self.exp_map2.get(&id).iter() {
+            let exp_map2 = self.exp_map2.borrow();
+            assert!(exp_map2.get().contains_key(&id), "wut {:?}", id);
+            for export in exp_map2.get().get(&id).iter() {
                 if is_local(export.def_id) && export.reexport {
                     self.reexports.insert(export.def_id.node);
                 }

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

@@ -29,7 +29,7 @@ use syntax::opt_vec::OptVec;
 use syntax::visit;
 use syntax::visit::Visitor;
 
-use std::cell::Cell;
+use std::cell::{Cell, RefCell};
 use std::uint;
 use std::hashmap::{HashMap, HashSet};
 use std::util;
@@ -50,7 +50,7 @@ pub type TraitMap = HashMap<NodeId,@mut ~[DefId]>;
 
 // This is the replacement export map. It maps a module to all of the exports
 // within.
-pub type ExportMap2 = @mut HashMap<NodeId, ~[Export2]>;
+pub type ExportMap2 = @RefCell<HashMap<NodeId, ~[Export2]>>;
 
 pub struct Export2 {
     name: @str,        // The name of the target.
@@ -808,7 +808,7 @@ fn Resolver(session: Session,
         namespaces: ~[ TypeNS, ValueNS ],
 
         def_map: @mut HashMap::new(),
-        export_map2: @mut HashMap::new(),
+        export_map2: @RefCell::new(HashMap::new()),
         trait_map: HashMap::new(),
         used_imports: HashSet::new(),
         external_exports: HashSet::new(),
@@ -3249,7 +3249,8 @@ impl Resolver {
         self.add_exports_for_module(&mut exports2, module_);
         match module_.def_id.get() {
             Some(def_id) => {
-                self.export_map2.insert(def_id.node, exports2);
+                let mut export_map2 = self.export_map2.borrow_mut();
+                export_map2.get().insert(def_id.node, exports2);
                 debug!("(computing exports) writing exports for {} (some)",
                        def_id.node);
             }