---

yaml
---
r: 44285
b: refs/heads/snap-stage3
c: abc4ea2
h: refs/heads/master
i:
  44283: 2c25a8d
v: v3

Author: bstrie

[refs]

@@ -1,7 +1,7 @@
 ---
 refs/heads/master: 19dfec2aaf746535de1521f68421f9980dbf25de
 refs/heads/snap-stage1: e33de59e47c5076a89eadeb38f4934f58a3618a6
-refs/heads/snap-stage3: 6e5705a87743503232ec927efebd00861f1ce717
+refs/heads/snap-stage3: abc4ea2001c24f3a1e3d9edf3bebb1b2bb8629ec
 refs/heads/try: ef355f6332f83371e4acf04fc4eb940ab41d78d3
 refs/tags/release-0.1: 1f5c5126e96c79d22cb7862f75304136e204f105
 refs/heads/ndm: f3868061cd7988080c30d6d5bf352a5a5fe2460b

branches/snap-stage3/src/librustc/middle/astencode.rs

@@ -860,7 +860,7 @@ fn encode_side_tables_for_id(ecx: @e::EncodeContext,
             }
         }
     }
-    do option::iter(&tcx.node_types.find(&(id as uint))) |&ty| {
+    do option::iter(&(*tcx.node_types).find(id as uint)) |ty| {
         do ebml_w.tag(c::tag_table_node_type) {
             ebml_w.id(id);
             do ebml_w.tag(c::tag_table_val) {
@@ -1135,7 +1135,7 @@ fn decode_side_tables(xcx: @ExtendedDecodeContext,
                 let ty = val_dsr.read_ty(xcx);
                 debug!("inserting ty for node %?: %s",
                        id, ty_to_str(dcx.tcx, ty));
-                dcx.tcx.node_types.insert(id as uint, ty);
+                (*dcx.tcx.node_types).insert(id as uint, ty);
             } else if tag == (c::tag_table_node_type_subst as uint) {
                 let tys = val_dsr.read_tys(xcx);
                 dcx.tcx.node_type_substs.insert(id, tys);

branches/snap-stage3/src/librustc/middle/lint.rs

@@ -34,7 +34,8 @@ use core::uint;
 use core::vec;
 use std::oldmap::{Map, HashMap};
 use std::oldmap;
-use std::smallintmap::SmallIntMap;
+use std::oldsmallintmap::{Map, SmallIntMap};
+use std::oldsmallintmap;
 use syntax::ast_util::{path_to_ident};
 use syntax::attr;
 use syntax::codemap::span;
@@ -274,7 +275,7 @@ pub fn get_lint_dict() -> LintDict {
 }
 
 // This is a highly not-optimal set of data structure decisions.
-type LintModes = @mut SmallIntMap<level>;
+type LintModes = SmallIntMap<level>;
 type LintModeMap = HashMap<ast::node_id, LintModes>;
 
 // settings_map maps node ids of items with non-default lint settings
@@ -287,14 +288,14 @@ pub struct LintSettings {
 
 pub fn mk_lint_settings() -> LintSettings {
     LintSettings {
-        default_settings: @mut SmallIntMap::new(),
+        default_settings: oldsmallintmap::mk(),
         settings_map: HashMap()
     }
 }
 
 pub fn get_lint_level(modes: LintModes, lint: lint) -> level {
-    match modes.find(&(lint as uint)) {
-      Some(&c) => c,
+    match modes.find(lint as uint) {
+      Some(c) => c,
       None => allow
     }
 }
@@ -313,7 +314,8 @@ pub fn get_lint_settings_level(settings: LintSettings,
 // This is kind of unfortunate. It should be somewhere else, or we should use
 // a persistent data structure...
 fn clone_lint_modes(modes: LintModes) -> LintModes {
-    @mut (copy *modes)
+    oldsmallintmap::SmallIntMap_(@oldsmallintmap::SmallIntMap_
+    {v: copy modes.v})
 }
 
 struct Context {
@@ -330,7 +332,7 @@ impl Context {
 
     fn set_level(&self, lint: lint, level: level) {
         if level == allow {
-            self.curr.remove(&(lint as uint));
+            self.curr.remove(lint as uint);
         } else {
             self.curr.insert(lint as uint, level);
         }
@@ -438,7 +440,7 @@ fn build_settings_item(i: @ast::item, &&cx: Context, v: visit::vt<Context>) {
 pub fn build_settings_crate(sess: session::Session, crate: @ast::crate) {
     let cx = Context {
         dict: get_lint_dict(),
-        curr: @mut SmallIntMap::new(),
+        curr: oldsmallintmap::mk(),
         is_default: true,
         sess: sess
     };
@@ -456,7 +458,7 @@ pub fn build_settings_crate(sess: session::Session, crate: @ast::crate) {
 
     do cx.with_lint_attrs(/*bad*/copy crate.node.attrs) |cx| {
         // Copy out the default settings
-        for cx.curr.each |&(k, &v)| {
+        for cx.curr.each |k, v| {
             sess.lint_settings.default_settings.insert(k, v);
         }
 

branches/snap-stage3/src/librustc/middle/trans/base.rs

@@ -76,6 +76,7 @@ use core::option::{is_none, is_some};
 use core::option;
 use core::uint;
 use std::oldmap::HashMap;
+use std::oldsmallintmap;
 use std::{oldmap, time, list};
 use syntax::ast_map::{path, path_elt_to_str, path_mod, path_name};
 use syntax::ast_util::{def_id_of_def, local_def, path_to_ident};

branches/snap-stage3/src/librustc/middle/ty.rs

@@ -42,8 +42,7 @@ use core::uint;
 use core::vec;
 use core::hashmap::linear::LinearMap;
 use std::oldmap::HashMap;
-use std::oldmap;
-use std::smallintmap::SmallIntMap;
+use std::{oldmap, oldsmallintmap};
 use syntax::ast::*;
 use syntax::ast_util::{is_local, local_def};
 use syntax::ast_util;
@@ -768,7 +767,7 @@ type type_cache = HashMap<ast::def_id, ty_param_bounds_and_ty>;
 
 type constness_cache = HashMap<ast::def_id, const_eval::constness>;
 
-pub type node_type_table = @mut SmallIntMap<t>;
+pub type node_type_table = @oldsmallintmap::SmallIntMap<t>;
 
 fn mk_rcache() -> creader_cache {
     type val = {cnum: int, pos: uint, len: uint};
@@ -813,7 +812,7 @@ pub fn mk_ctxt(s: session::Session,
         def_map: dm,
         region_map: region_map,
         region_paramd_items: region_paramd_items,
-        node_types: @mut SmallIntMap::new(),
+        node_types: @oldsmallintmap::mk(),
         node_type_substs: oldmap::HashMap(),
         items: amap,
         intrinsic_defs: oldmap::HashMap(),
@@ -2788,8 +2787,8 @@ pub fn br_hashmap<V:Copy>() -> HashMap<bound_region, V> {
 
 pub fn node_id_to_type(cx: ctxt, id: ast::node_id) -> t {
     //io::println(fmt!("%?/%?", id, cx.node_types.len()));
-    match cx.node_types.find(&(id as uint)) {
-       Some(&t) => t,
+    match oldsmallintmap::find(*cx.node_types, id as uint) {
+       Some(t) => t,
        None => cx.sess.bug(
            fmt!("node_id_to_type: no type for node `%s`",
                 ast_map::node_id_to_str(cx.items, id,
@@ -3180,8 +3179,8 @@ pub fn expr_kind(tcx: ctxt,
         }
 
         ast::expr_cast(*) => {
-            match tcx.node_types.find(&(expr.id as uint)) {
-                Some(&t) => {
+            match oldsmallintmap::find(*tcx.node_types, expr.id as uint) {
+                Some(t) => {
                     if ty::type_is_immediate(t) {
                         RvalueDatumExpr
                     } else {
@@ -4290,8 +4289,7 @@ pub fn eval_repeat_count(tcx: ctxt,
                          count_expr: @ast::expr,
                          span: span)
                       -> uint {
-    match const_eval::eval_const_expr_partial(tcx, count_expr) {
-      Ok(ref const_val) => match *const_val {
+    match const_eval::eval_const_expr(tcx, count_expr) {
         const_eval::const_int(count) => return count as uint,
         const_eval::const_uint(count) => return count as uint,
         const_eval::const_float(count) => {
@@ -4312,13 +4310,7 @@ pub fn eval_repeat_count(tcx: ctxt,
                                 repeat count but found boolean");
             return 0;
         }
-      },
-      Err(*) => {
-        tcx.sess.span_err(span,
-                          ~"expected constant integer for repeat count \
-                            but found variable");
-        return 0;
-      }
+
     }
 }
 

branches/snap-stage3/src/librustc/middle/typeck/infer/mod.rs

@@ -277,7 +277,7 @@ use core::result;
 use core::vec;
 use std::list::Nil;
 use std::oldmap::HashMap;
-use std::smallintmap::SmallIntMap;
+use std::oldsmallintmap;
 use syntax::ast::{ret_style, purity};
 use syntax::ast::{m_const, m_imm, m_mutbl};
 use syntax::ast::{unsafe_fn, impure_fn, pure_fn, extern_fn};
@@ -353,7 +353,7 @@ pub fn fixup_err_to_str(f: fixup_err) -> ~str {
 
 fn new_ValsAndBindings<V:Copy,T:Copy>() -> ValsAndBindings<V, T> {
     ValsAndBindings {
-        vals: @mut SmallIntMap::new(),
+        vals: oldsmallintmap::mk(),
         bindings: ~[]
     }
 }

branches/snap-stage3/src/librustc/middle/typeck/infer/unify.rs

@@ -10,7 +10,7 @@
 
 use core::prelude::*;
 use core::result;
-use std::smallintmap::SmallIntMap;
+use std::oldsmallintmap::SmallIntMap;
 
 use middle::ty::{Vid, expected_found, IntVarValue};
 use middle::ty;
@@ -27,7 +27,7 @@ pub enum VarValue<V, T> {
 }
 
 pub struct ValsAndBindings<V, T> {
-    vals: @mut SmallIntMap<VarValue<V, T>>,
+    vals: SmallIntMap<VarValue<V, T>>,
     bindings: ~[(V, VarValue<V, T>)],
 }
 
@@ -64,12 +64,12 @@ pub impl InferCtxt {
             vid: V) -> Node<V, T>
         {
             let vid_u = vid.to_uint();
-            match vb.vals.find(&vid_u) {
+            match vb.vals.find(vid_u) {
                 None => {
                     tcx.sess.bug(fmt!(
                         "failed lookup of vid `%u`", vid_u));
                 }
-                Some(var_val) => {
+                Some(ref var_val) => {
                     match *var_val {
                         Redirect(vid) => {
                             let node: Node<V,T> = helper(tcx, vb, vid);
@@ -103,8 +103,8 @@ pub impl InferCtxt {
 
         { // FIXME(#4903)---borrow checker is not flow sensitive
             let vb = UnifyVid::appropriate_vals_and_bindings(self);
-            let old_v = vb.vals.get(&vid.to_uint());
-            vb.bindings.push((vid, *old_v));
+            let old_v = vb.vals.get(vid.to_uint());
+            vb.bindings.push((vid, old_v));
             vb.vals.insert(vid.to_uint(), new_v);
         }
     }

branches/snap-stage3/src/librustc/middle/typeck/mod.rs

@@ -69,6 +69,7 @@ use std::list::{List, Nil, Cons};
 use std::list;
 use std::oldmap::HashMap;
 use std::oldmap;
+use std::oldsmallintmap;
 use syntax::ast::{provided, required};
 use syntax::ast_map::node_id_to_str;
 use syntax::ast_util::{local_def, split_trait_methods};