---

yaml
---
r: 44159
b: refs/heads/snap-stage3
c: d0b5016
h: refs/heads/master
i:
  44157: 0997eea
  44155: e14fcc4
  44151: 823c949
  44143: 4f6acf8
  44127: 1ba4a69
  44095: a706eab
  44031: 3037b5c
v: v3

Author: bors

[refs]

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

branches/snap-stage3/src/libcore/vec.rs

@@ -1865,6 +1865,7 @@ pub trait OwnedVector<T> {
     fn consume(self, f: fn(uint, v: T));
     fn filter(self, f: fn(t: &T) -> bool) -> ~[T];
     fn partition(self, f: pure fn(&T) -> bool) -> (~[T], ~[T]);
+    fn grow_fn(&mut self, n: uint, op: iter::InitOp<T>);
 }
 
 impl<T> OwnedVector<T> for ~[T] {
@@ -1936,6 +1937,11 @@ impl<T> OwnedVector<T> for ~[T] {
     fn partition(self, f: fn(&T) -> bool) -> (~[T], ~[T]) {
         partition(self, f)
     }
+
+    #[inline]
+    fn grow_fn(&mut self, n: uint, op: iter::InitOp<T>) {
+        grow_fn(self, n, op);
+    }
 }
 
 impl<T> Mutable for ~[T] {
@@ -1946,7 +1952,6 @@ impl<T> Mutable for ~[T] {
 pub trait OwnedCopyableVector<T: Copy> {
     fn push_all(&mut self, rhs: &[const T]);
     fn grow(&mut self, n: uint, initval: &T);
-    fn grow_fn(&mut self, n: uint, op: iter::InitOp<T>);
     fn grow_set(&mut self, index: uint, initval: &T, val: T);
 }
 
@@ -1961,11 +1966,6 @@ impl<T: Copy> OwnedCopyableVector<T> for ~[T] {
         grow(self, n, initval);
     }
 
-    #[inline]
-    fn grow_fn(&mut self, n: uint, op: iter::InitOp<T>) {
-        grow_fn(self, n, op);
-    }
-
     #[inline]
     fn grow_set(&mut self, index: uint, initval: &T, val: T) {
         grow_set(self, index, initval, val);

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

@@ -456,6 +456,7 @@ pub fn set_glue_inlining(f: ValueRef, t: ty::t) {
 // Double-check that we never ask LLVM to declare the same symbol twice. It
 // silently mangles such symbols, breaking our linkage model.
 pub fn note_unique_llvm_symbol(ccx: @crate_ctxt, +sym: ~str) {
+    // XXX: Bad copy.
     if ccx.all_llvm_symbols.contains_key(&sym) {
         ccx.sess.bug(~"duplicate LLVM symbol: " + sym);
     }
@@ -627,10 +628,7 @@ pub fn iter_structural_ty(cx: block, av: ValueRef, t: ty::t,
             for vec::each(fn_ty.sig.inputs) |a| {
                 let llfldp_a = GEP_enum(cx, a_tup, tid, v_id,
                                         /*bad*/copy tps, j);
-                // This assumes the self type is absent (it passes
-                // None for the self_ty_opt arg of substs_tps).
-                // I think that's ok since you can't have an enum
-                // inside a trait.
+                // XXX: Is "None" right here?
                 let ty_subst = ty::subst_tps(ccx.tcx, tps, None, a.ty);
                 cx = f(cx, llfldp_a, ty_subst);
                 j += 1u;
@@ -1040,7 +1038,8 @@ pub fn load_if_immediate(cx: block, v: ValueRef, t: ty::t) -> ValueRef {
 pub fn trans_trace(bcx: block, sp_opt: Option<span>, +trace_str: ~str) {
     if !bcx.sess().trace() { return; }
     let _icx = bcx.insn_ctxt("trans_trace");
-    add_comment(bcx, trace_str);
+    // XXX: Bad copy.
+    add_comment(bcx, copy trace_str);
     let V_trace_str = C_cstr(bcx.ccx(), trace_str);
     let {V_filename, V_line} = match sp_opt {
       Some(sp) => {
@@ -1552,7 +1551,7 @@ pub fn new_fn_ctxt_w_id(ccx: @crate_ctxt,
                         llfndecl: ValueRef,
                         id: ast::node_id,
                         impl_id: Option<ast::def_id>,
-                        param_substs: Option<@param_substs>,
+                        +param_substs: Option<param_substs>,
                         sp: Option<span>) -> fn_ctxt {
     let llbbs = mk_standard_basic_blocks(llfndecl);
     return @fn_ctxt_ {
@@ -1741,7 +1740,7 @@ pub fn trans_closure(ccx: @crate_ctxt,
                      body: &ast::blk,
                      llfndecl: ValueRef,
                      ty_self: self_arg,
-                     param_substs: Option<@param_substs>,
+                     +param_substs: Option<param_substs>,
                      id: ast::node_id,
                      impl_id: Option<ast::def_id>,
                      maybe_load_env: fn(fn_ctxt),
@@ -1805,7 +1804,7 @@ pub fn trans_fn(ccx: @crate_ctxt,
                 body: &ast::blk,
                 llfndecl: ValueRef,
                 ty_self: self_arg,
-                param_substs: Option<@param_substs>,
+                +param_substs: Option<param_substs>,
                 id: ast::node_id,
                 impl_id: Option<ast::def_id>) {
     let do_time = ccx.sess.trans_stats();
@@ -1814,8 +1813,8 @@ pub fn trans_fn(ccx: @crate_ctxt,
     debug!("trans_fn(ty_self=%?)", ty_self);
     let _icx = ccx.insn_ctxt("trans_fn");
     ccx.stats.n_fns += 1;
-    let the_path_str = path_str(ccx.sess, path);
-    trans_closure(ccx, path, decl, body, llfndecl, ty_self,
+    // XXX: Bad copy of `path`.
+    trans_closure(ccx, copy path, decl, body, llfndecl, ty_self,
                   param_substs, id, impl_id,
                   |fcx| {
                       if ccx.sess.opts.extra_debuginfo {
@@ -1825,7 +1824,7 @@ pub fn trans_fn(ccx: @crate_ctxt,
                   |_bcx| { });
     if do_time {
         let end = time::get_time();
-        log_fn_time(ccx, the_path_str, start, end);
+        log_fn_time(ccx, path_str(ccx.sess, path), start, end);
     }
 }
 
@@ -1835,7 +1834,7 @@ pub fn trans_enum_variant(ccx: @crate_ctxt,
                           args: ~[ast::variant_arg],
                           disr: int,
                           is_degen: bool,
-                          param_substs: Option<@param_substs>,
+                          +param_substs: Option<param_substs>,
                           llfndecl: ValueRef) {
     let _icx = ccx.insn_ctxt("trans_enum_variant");
     // Translate variant arguments to function arguments.
@@ -1851,8 +1850,9 @@ pub fn trans_enum_variant(ccx: @crate_ctxt,
             id: varg.id,
         }
     };
+    // XXX: Bad copy of `param_substs`.
     let fcx = new_fn_ctxt_w_id(ccx, ~[], llfndecl, variant.node.id, None,
-                               param_substs, None);
+                               copy param_substs, None);
     // XXX: Bad copy.
     let raw_llargs = create_llargs_for_fn_args(fcx, no_self, copy fn_args);
     let ty_param_substs = match param_substs {
@@ -1897,7 +1897,7 @@ pub fn trans_enum_variant(ccx: @crate_ctxt,
 pub fn trans_tuple_struct(ccx: @crate_ctxt,
                           fields: ~[@ast::struct_field],
                           ctor_id: ast::node_id,
-                          param_substs: Option<@param_substs>,
+                          +param_substs: Option<param_substs>,
                           llfndecl: ValueRef) {
     let _icx = ccx.insn_ctxt("trans_tuple_struct");
 
@@ -1951,7 +1951,7 @@ pub fn trans_struct_dtor(ccx: @crate_ctxt,
                          +path: path,
                          body: &ast::blk,
                          dtor_id: ast::node_id,
-                         psubsts: Option<@param_substs>,
+                         +psubsts: Option<param_substs>,
                          hash_id: Option<mono_id>,
                          parent_id: ast::def_id)
                       -> ValueRef {
@@ -1968,7 +1968,7 @@ pub fn trans_struct_dtor(ccx: @crate_ctxt,
   let lldty = type_of_dtor(ccx, class_ty);
 
   // XXX: Bad copies.
-  let s = get_dtor_symbol(ccx, copy path, dtor_id, psubsts);
+  let s = get_dtor_symbol(ccx, copy path, dtor_id, copy psubsts);
 
   /* Register the dtor as a function. It has external linkage */
   let lldecl = decl_internal_cdecl_fn(ccx.llmod, s, lldty);
@@ -2296,7 +2296,7 @@ pub fn item_path(ccx: @crate_ctxt, i: @ast::item) -> path {
 pub fn get_dtor_symbol(ccx: @crate_ctxt,
                        +path: path,
                        id: ast::node_id,
-                       substs: Option<@param_substs>)
+                       +substs: Option<param_substs>)
                     -> ~str {
   let t = ty::node_id_to_type(ccx.tcx, id);
   match ccx.item_symbols.find(&id) {

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

@@ -834,7 +834,7 @@ pub fn add_span_comment(bcx: block, sp: span, text: ~str) {
     }
 }
 
-pub fn add_comment(bcx: block, text: &str) {
+pub fn add_comment(bcx: block, text: ~str) {
     unsafe {
         let ccx = bcx.ccx();
         if !ccx.sess.no_asm_comments() {

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

@@ -311,7 +311,7 @@ pub struct fn_ctxt_ {
 
     // If this function is being monomorphized, this contains the type
     // substitutions used.
-    param_substs: Option<@param_substs>,
+    param_substs: Option<param_substs>,
 
     // The source span and nesting context where this function comes from, for
     // error reporting and symbol generation.
@@ -1395,7 +1395,7 @@ pub fn resolve_vtable_in_fn_ctxt(fcx: fn_ctxt, +vt: typeck::vtable_origin)
         }
         typeck::vtable_param(n_param, n_bound) => {
             match fcx.param_substs {
-                Some(substs) => {
+                Some(ref substs) => {
                     find_vtable(tcx, substs, n_param, n_bound)
                 }
                 _ => {

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

@@ -524,8 +524,7 @@ pub impl Datum {
         if bcx.sess().trace() {
             trans_trace(
                 bcx, None,
-                fmt!("preserving until end of scope %d",
-                     root_info.scope));
+                fmt!("preserving until end of scope %d", root_info.scope));
         }
 
         let scratch = scratch_datum(bcx, self.ty, true);

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

@@ -332,7 +332,7 @@ pub fn trans_intrinsic(ccx: @crate_ctxt,
                        decl: ValueRef,
                        item: @ast::foreign_item,
                        +path: ast_map::path,
-                       substs: @param_substs,
+                       +substs: param_substs,
                        ref_id: Option<ast::node_id>) {
     debug!("trans_intrinsic(item.ident=%s)", ccx.sess.str_of(item.ident));
 

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

@@ -66,7 +66,7 @@ pub fn trans_impl(ccx: @crate_ctxt, +path: path, name: ast::ident,
             match self_ty {
                 None => param_substs_opt = None,
                 Some(self_ty) => {
-                    param_substs_opt = Some(@param_substs {
+                    param_substs_opt = Some(param_substs {
                         tys: ~[],
                         vtables: None,
                         bounds: @~[],
@@ -99,7 +99,7 @@ Translates a (possibly monomorphized) method body.
 pub fn trans_method(ccx: @crate_ctxt,
                     +path: path,
                     method: &ast::method,
-                    param_substs: Option<@param_substs>,
+                    +param_substs: Option<param_substs>,
                     base_self_ty: Option<ty::t>,
                     llfn: ValueRef,
                     impl_id: ast::def_id) {
@@ -118,7 +118,7 @@ pub fn trans_method(ccx: @crate_ctxt,
         }
         let self_ty = match param_substs {
             None => self_ty,
-            Some(@param_substs {tys: ref tys, _}) => {
+            Some(param_substs {tys: ref tys, _}) => {
                 ty::subst_tps(ccx.tcx, *tys, None, self_ty)
             }
         };
@@ -247,7 +247,7 @@ pub fn trans_method_callee(bcx: block,
             bound_num: b
         }) => {
             match bcx.fcx.param_substs {
-                Some(substs) => {
+                Some(ref substs) => {
                     let vtbl = find_vtable(bcx.tcx(), substs, p, b);
                     trans_monomorphized_callee(bcx, callee_id, self, mentry,
                                                trait_id, off, vtbl)

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

@@ -156,7 +156,7 @@ pub fn monomorphic_fn(ccx: @crate_ctxt,
         lldecl
     };
 
-    let psubsts = Some(@param_substs {
+    let psubsts = Some(param_substs {
         tys: substs,
         vtables: vtables,
         bounds: tpt.bounds,

branches/snap-stage3/src/libstd/deque.rs

@@ -100,7 +100,7 @@ fn grow<T>(nelts: uint, lo: uint, elts: &mut [Option<T>]) -> ~[Option<T>] {
     assert nelts == elts.len();
     let mut rv = ~[];
 
-    do vec::grow_fn(&mut rv, nelts + 1) |i| {
+    do rv.grow_fn(nelts + 1) |i| {
         let mut element = None;
         element <-> elts[(lo + i) % nelts];
         element

branches/snap-stage3/src/test/run-pass/class-impl-very-parameterized-trait.rs

@@ -127,7 +127,7 @@ priv impl<T> cat<T> {
     }
 }
 
-fn main() {
+pub fn main() {
     let mut nyan: cat<~str> = cat::new(0, 2, ~"nyan");
     for uint::range(1, 5) |_| { nyan.speak(); }
     assert(*nyan.find(&1).unwrap() == ~"nyan");

branches/snap-stage3/src/test/run-pass/const-enum-ptr.rs

@@ -11,7 +11,7 @@
 enum E { V0, V1(int) }
 const C: &static/E = &V0;
 
-fn main() {
+pub fn main() {
     match *C {
         V0 => (),
         _ => fail!()

branches/snap-stage3/src/test/run-pass/const-enum-struct.rs

@@ -12,7 +12,7 @@ enum E { V16(u16), V32(u32) }
 struct S { a: E, b: u16, c: u16 }
 const C: S = S { a: V16(0xDEAD), b: 0x600D, c: 0xBAD };
 
-fn main() {
+pub fn main() {
     let n = C.b;
     assert n != 0xBAD;
     assert n == 0x600D;

branches/snap-stage3/src/test/run-pass/const-enum-struct2.rs

@@ -12,7 +12,7 @@ enum E { V0, V16(u16) }
 struct S { a: E, b: u16, c: u16 }
 const C: S = S { a: V0, b: 0x600D, c: 0xBAD };
 
-fn main() {
+pub fn main() {
     let n = C.b;
     assert n != 0xBAD;
     assert n == 0x600D;

branches/snap-stage3/src/test/run-pass/const-enum-tuple.rs

@@ -11,7 +11,7 @@
 enum E { V16(u16), V32(u32) }
 const C: (E, u16, u16) = (V16(0xDEAD), 0x600D, 0xBAD);
 
-fn main() {
+pub fn main() {
     let (_, n, _) = C;
     assert n != 0xBAD;
     assert n == 0x600D;

branches/snap-stage3/src/test/run-pass/const-enum-tuple2.rs

@@ -11,7 +11,7 @@
 enum E { V0, V16(u16) }
 const C: (E, u16, u16) = (V0, 0x600D, 0xBAD);
 
-fn main() {
+pub fn main() {
     let (_, n, _) = C;
     assert n != 0xBAD;
     assert n == 0x600D;

branches/snap-stage3/src/test/run-pass/const-enum-tuplestruct.rs

@@ -12,7 +12,7 @@ enum E { V16(u16), V32(u32) }
 struct S(E, u16, u16);
 const C: S = S(V16(0xDEAD), 0x600D, 0xBAD);
 
-fn main() {
+pub fn main() {
     let S(_, n, _) = C;
     assert n != 0xBAD;
     assert n == 0x600D;

branches/snap-stage3/src/test/run-pass/const-enum-tuplestruct2.rs

@@ -12,7 +12,7 @@ enum E { V0, V16(u16) }
 struct S(E, u16, u16);
 const C: S = S(V0, 0x600D, 0xBAD);
 
-fn main() {
+pub fn main() {
     let S(_, n, _) = C;
     assert n != 0xBAD;
     assert n == 0x600D;

branches/snap-stage3/src/test/run-pass/const-enum-vec-index.rs

@@ -13,7 +13,7 @@ const C: &[E] = &[V0, V1(0xDEADBEE)];
 const C0: E = C[0];
 const C1: E = C[1];
 
-fn main() {
+pub fn main() {
     match C0 { 
         V0 => (),
         _ => fail!()

branches/snap-stage3/src/test/run-pass/const-enum-vec-ptr.rs

@@ -11,7 +11,7 @@
 enum E { V1(int), V0 }
 const C: &static/[E] = &[V0, V1(0xDEADBEE), V0];
 
-fn main() {
+pub fn main() {
     match C[1] {
         V1(n) => assert(n == 0xDEADBEE),
         _ => fail!()

branches/snap-stage3/src/test/run-pass/const-enum-vector.rs

@@ -11,7 +11,7 @@
 enum E { V1(int), V0 }
 const C: [E * 3] = [V0, V1(0xDEADBEE), V0];
 
-fn main() {
+pub fn main() {
     match C[1] {
         V1(n) => assert(n == 0xDEADBEE),
         _ => fail!()

branches/snap-stage3/src/test/run-pass/impl-privacy-xc-1.rs

@@ -3,7 +3,7 @@
 
 extern mod impl_privacy_xc_1;
 
-fn main() {
+pub fn main() {
     let fish = impl_privacy_xc_1::Fish { x: 1 };
     fish.swim();
 }

branches/snap-stage3/src/test/run-pass/impl-privacy-xc-2.rs

@@ -3,7 +3,7 @@
 
 extern mod impl_privacy_xc_2;
 
-fn main() {
+pub fn main() {
     let fish1 = impl_privacy_xc_2::Fish { x: 1 };
     let fish2 = impl_privacy_xc_2::Fish { x: 2 };
     io::println(if fish1.eq(&fish2) { "yes" } else { "no " });