---

yaml
---
r: 152788
b: refs/heads/try2
c: 609552e
h: refs/heads/master
v: v3

Author: Keegan McAllister

[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: c1898b9acb324990367bb6672145c30b51e0399f
+refs/heads/try2: 609552e19516299b061687d52094d0b755b213d3
 refs/heads/dist-snap: ba4081a5a8573875fed17545846f6f6902c8ba8d
 refs/tags/release-0.2: c870d2dffb391e14efb05aa27898f1f6333a9596
 refs/tags/release-0.3: b5f0d0f648d9a6153664837026ba1be43d3e2503

branches/try2/src/librustc/lint/context.rs

@@ -56,7 +56,9 @@ pub struct LintStore {
     lints: Vec<(&'static Lint, bool)>,
 
     /// Trait objects for each lint pass.
-    passes: Vec<RefCell<LintPassObject>>,
+    /// This is only `None` while iterating over the objects. See the definition
+    /// of run_lints.
+    passes: Option<Vec<LintPassObject>>,
 
     /// Lints indexed by name.
     by_name: HashMap<&'static str, LintId>,
@@ -84,7 +86,7 @@ impl LintStore {
     pub fn new() -> LintStore {
         LintStore {
             lints: vec!(),
-            passes: vec!(),
+            passes: Some(vec!()),
             by_name: HashMap::new(),
             levels: HashMap::new(),
         }
@@ -117,7 +119,7 @@ impl LintStore {
                 self.levels.insert(id, (lint.default_level, Default));
             }
         }
-        self.passes.push(RefCell::new(pass));
+        self.passes.get_mut_ref().push(pass);
     }
 
     pub fn register_builtin(&mut self, sess: Option<&Session>) {
@@ -181,11 +183,15 @@ pub struct Context<'a> {
 }
 
 /// Convenience macro for calling a `LintPass` method on every pass in the context.
-macro_rules! run_lints ( ($cx:expr, $f:ident, $($args:expr),*) => (
-    for obj in $cx.lints.passes.iter() {
-        obj.borrow_mut().$f($cx, $($args),*);
-    }
-))
+macro_rules! run_lints ( ($cx:expr, $f:ident, $($args:expr),*) => ({
+    // Move the vector of passes out of `$cx` so that we can
+    // iterate over it mutably while passing `$cx` to the methods.
+    let mut passes = $cx.lints.passes.take_unwrap();
+    for obj in passes.mut_iter() {
+        obj.$f($cx, $($args),*);
+    }
+    $cx.lints.passes = Some(passes);
+}))
 
 /// Emit a lint as a warning or an error (or not at all)
 /// according to `level`.