From da519c8587c46ec519fada1aee7cc32ed2c1cd6a Mon Sep 17 00:00:00 2001
From: Tim Chevalier <chevalier@alum.wellesley.edu>
Date: Thu, 5 Jan 2012 11:33:22 -0800
Subject: [PATCH] Eliminate a use of map::get in freevars

map::get is evil -- just say no!
---
 src/comp/middle/freevars.rs | 35 +++++++++++++++++++++--------------
 1 file changed, 21 insertions(+), 14 deletions(-)

diff --git a/src/comp/middle/freevars.rs b/src/comp/middle/freevars.rs
index 7b348a9c8a1..46ceb4257ff 100644
--- a/src/comp/middle/freevars.rs
+++ b/src/comp/middle/freevars.rs
@@ -1,6 +1,7 @@
 // A pass that annotates for each loops and functions with the free
 // variables that they contain.
 
+import syntax::print::pprust::path_to_str;
 import int;
 import std::map::*;
 import option::*;
@@ -47,21 +48,27 @@ fn collect_freevars(def_map: resolve::def_map, blk: ast::blk)
                 visit::visit_expr(expr, depth + 1, v);
               }
               ast::expr_path(path) {
-                let def = def_map.get(expr.id), i = 0;
-                while i < depth {
-                    alt copy def {
-                      ast::def_upvar(_, inner, _) { def = *inner; }
-                      _ { break; }
+                  let i = 0;
+                  alt def_map.find(expr.id) {
+                    none. { fail ("Not found: " + path_to_str(path)) }
+                    some(df) {
+                      let def = df;
+                      while i < depth {
+                        alt copy def {
+                          ast::def_upvar(_, inner, _) { def = *inner; }
+                          _ { break; }
+                        }
+                        i += 1;
+                      }
+                      if i == depth { // Made it to end of loop
+                        let dnum = ast_util::def_id_of_def(def).node;
+                        if !seen.contains_key(dnum) {
+                           *refs += [@{def:def, span:expr.span}];
+                           seen.insert(dnum, ());
+                        }
+                      }
                     }
-                    i += 1;
-                }
-                if i == depth { // Made it to end of loop
-                    let dnum = ast_util::def_id_of_def(def).node;
-                    if !seen.contains_key(dnum) {
-                        *refs += [@{def:def, span:expr.span}];
-                        seen.insert(dnum, ());
-                    }
-                }
+                  }
               }
               _ { visit::visit_expr(expr, depth, v); }
             }