---

yaml
---
r: 78813
b: refs/heads/try
c: 4de9bca
h: refs/heads/master
i:
  78811: c7e48eb
v: v3

Author: blake2-ppc

[refs]

@@ -2,7 +2,7 @@
 refs/heads/master: 25ed29a0edb3d48fef843a0b818ee68faf2252da
 refs/heads/snap-stage1: e33de59e47c5076a89eadeb38f4934f58a3618a6
 refs/heads/snap-stage3: 60fba4d7d677ec098e6a43014132fe99f7547363
-refs/heads/try: 2f82d89b2869adb4b1b17d19303676fa635d5df3
+refs/heads/try: 4de9bca4d813b88f06f7c5f64d2c452c80611f6b
 refs/tags/release-0.1: 1f5c5126e96c79d22cb7862f75304136e204f105
 refs/heads/ndm: f3868061cd7988080c30d6d5bf352a5a5fe2460b
 refs/heads/try2: 147ecfdd8221e4a4d4e090486829a06da1e0ca3c

branches/try/src/libextra/getopts.rs

@@ -689,9 +689,9 @@ pub mod groups {
                 }
             }
 
-            // FIXME: #5516 should be graphemes not codepoints
+            // FIXME: #5516
             // here we just need to indent the start of the description
-            let rowlen = row.char_len();
+            let rowlen = row.len();
             if rowlen < 24 {
                 do (24 - rowlen).times {
                     row.push_char(' ')
@@ -707,14 +707,14 @@ pub mod groups {
                 desc_normalized_whitespace.push_char(' ');
             }
 
-            // FIXME: #5516 should be graphemes not codepoints
+            // FIXME: #5516
             let mut desc_rows = ~[];
             do each_split_within(desc_normalized_whitespace, 54) |substr| {
                 desc_rows.push(substr.to_owned());
                 true
             };
 
-            // FIXME: #5516 should be graphemes not codepoints
+            // FIXME: #5516
             // wrapped description
             row.push_str(desc_rows.connect(desc_sep));
 
@@ -798,7 +798,7 @@ pub mod groups {
             cont
         };
 
-        ss.char_offset_iter().advance(|x| machine(x));
+        ss.iter().enumerate().advance(|x| machine(x));
 
         // Let the automaton 'run out' by supplying trailing whitespace
         while cont && match state { B | C => true, A => false } {
@@ -1580,31 +1580,4 @@ Options:
         debug!("generated: <<%s>>", usage);
         assert!(usage == expected)
     }
-
-    #[test]
-    fn test_groups_usage_description_multibyte_handling() {
-        let optgroups = ~[
-            groups::optflag("k", "k\u2013w\u2013",
-                "The word kiwi is normally spelled with two i's"),
-            groups::optflag("a", "apple",
-                "This \u201Cdescription\u201D has some characters that could \
-confuse the line wrapping; an apple costs 0.51€ in some parts of Europe."),
-        ];
-
-        let expected =
-~"Usage: fruits
-
-Options:
-    -k --k–w–           The word kiwi is normally spelled with two i's
-    -a --apple          This “description” has some characters that could
-                        confuse the line wrapping; an apple costs 0.51€ in
-                        some parts of Europe.
-";
-
-        let usage = groups::usage("Usage: fruits", optgroups);
-
-        debug!("expected: <<%s>>", expected);
-        debug!("generated: <<%s>>", usage);
-        assert!(usage == expected)
-    }
 }

branches/try/src/librustc/back/arm.rs

@@ -13,7 +13,7 @@ use driver::session::sess_os_to_meta_os;
 use driver::session;
 use metadata::loader::meta_section_name;
 
-pub fn get_target_strs(target_triple: ~str, target_os: session::os) -> target_strs::t {
+pub fn get_target_strs(target_os: session::os) -> target_strs::t {
     return target_strs::t {
         module_asm: ~"",
 
@@ -61,7 +61,13 @@ pub fn get_target_strs(target_triple: ~str, target_os: session::os) -> target_st
           }
         },
 
-        target_triple: target_triple,
+        target_triple: match target_os {
+          session::os_macos => ~"arm-apple-darwin",
+          session::os_win32 => ~"arm-pc-mingw32",
+          session::os_linux => ~"arm-unknown-linux-gnueabihf",
+          session::os_android => ~"arm-linux-androideabi",
+          session::os_freebsd => ~"arm-unknown-freebsd"
+        },
 
         cc_args: ~[~"-marm"]
     };

branches/try/src/librustc/back/mips.rs

@@ -13,7 +13,7 @@ use driver::session;
 use driver::session::sess_os_to_meta_os;
 use metadata::loader::meta_section_name;
 
-pub fn get_target_strs(target_triple: ~str, target_os: session::os) -> target_strs::t {
+pub fn get_target_strs(target_os: session::os) -> target_strs::t {
     return target_strs::t {
         module_asm: ~"",
 
@@ -61,7 +61,13 @@ pub fn get_target_strs(target_triple: ~str, target_os: session::os) -> target_st
           }
         },
 
-        target_triple: target_triple,
+        target_triple: match target_os {
+          session::os_macos => ~"mips-apple-darwin",
+          session::os_win32 => ~"mips-pc-mingw32",
+          session::os_linux => ~"mips-unknown-linux-gnu",
+          session::os_android => ~"mips-unknown-android-gnu",
+          session::os_freebsd => ~"mips-unknown-freebsd"
+        },
 
         cc_args: ~[]
     };

branches/try/src/librustc/back/x86.rs

@@ -14,7 +14,7 @@ use driver::session::sess_os_to_meta_os;
 use driver::session;
 use metadata::loader::meta_section_name;
 
-pub fn get_target_strs(target_triple: ~str, target_os: session::os) -> target_strs::t {
+pub fn get_target_strs(target_os: session::os) -> target_strs::t {
     return target_strs::t {
         module_asm: ~"",
 
@@ -44,7 +44,13 @@ pub fn get_target_strs(target_triple: ~str, target_os: session::os) -> target_st
           }
         },
 
-        target_triple: target_triple,
+        target_triple: match target_os {
+          session::os_macos => ~"i686-apple-darwin",
+          session::os_win32 => ~"i686-pc-mingw32",
+          session::os_linux => ~"i686-unknown-linux-gnu",
+          session::os_android => ~"i686-unknown-android-gnu",
+          session::os_freebsd => ~"i686-unknown-freebsd"
+        },
 
         cc_args: ~[~"-m32"]
     };

branches/try/src/librustc/back/x86_64.rs

@@ -14,7 +14,7 @@ use driver::session::sess_os_to_meta_os;
 use driver::session;
 use metadata::loader::meta_section_name;
 
-pub fn get_target_strs(target_triple: ~str, target_os: session::os) -> target_strs::t {
+pub fn get_target_strs(target_os: session::os) -> target_strs::t {
     return target_strs::t {
         module_asm: ~"",
 
@@ -52,7 +52,13 @@ pub fn get_target_strs(target_triple: ~str, target_os: session::os) -> target_st
           }
         },
 
-        target_triple: target_triple,
+        target_triple: match target_os {
+          session::os_macos => ~"x86_64-apple-darwin",
+          session::os_win32 => ~"x86_64-pc-mingw32",
+          session::os_linux => ~"x86_64-unknown-linux-gnu",
+          session::os_android => ~"x86_64-unknown-android-gnu",
+          session::os_freebsd => ~"x86_64-unknown-freebsd",
+        },
 
         cc_args: ~[~"-m64"]
     };

branches/try/src/librustc/driver/driver.rs

@@ -570,12 +570,11 @@ pub fn build_target_config(sopts: @session::options,
       abi::Arm => (ast::ty_i32, ast::ty_u32, ast::ty_f64),
       abi::Mips => (ast::ty_i32, ast::ty_u32, ast::ty_f64)
     };
-    let target_triple = sopts.target_triple.clone();
     let target_strs = match arch {
-      abi::X86 => x86::get_target_strs(target_triple, os),
-      abi::X86_64 => x86_64::get_target_strs(target_triple, os),
-      abi::Arm => arm::get_target_strs(target_triple, os),
-      abi::Mips => mips::get_target_strs(target_triple, os)
+      abi::X86 => x86::get_target_strs(os),
+      abi::X86_64 => x86_64::get_target_strs(os),
+      abi::Arm => arm::get_target_strs(os),
+      abi::Mips => mips::get_target_strs(os)
     };
     let target_cfg = @session::config {
         os: os,

branches/try/src/librustc/metadata/creader.rs

@@ -25,7 +25,7 @@ use syntax::codemap::{span, dummy_sp};
 use syntax::diagnostic::span_handler;
 use syntax::parse::token;
 use syntax::parse::token::ident_interner;
-use syntax::visit;
+use syntax::oldvisit;
 
 // Traverses an AST, reading all the information about use'd crates and extern
 // libraries necessary for later resolving, typechecking, linking, etc.
@@ -46,25 +46,17 @@ pub fn read_crates(diag: @mut span_handler,
         next_crate_num: 1,
         intr: intr
     };
-    let mut v = ReadCrateVisitor{ e:e };
+    let v =
+        oldvisit::mk_simple_visitor(@oldvisit::SimpleVisitor {
+            visit_view_item: |a| visit_view_item(e, a),
+            visit_item: |a| visit_item(e, a),
+            .. *oldvisit::default_simple_visitor()});
     visit_crate(e, crate);
-    visit::walk_crate(&mut v, crate, ());
+    oldvisit::visit_crate(crate, ((), v));
     dump_crates(*e.crate_cache);
     warn_if_multiple_versions(e, diag, *e.crate_cache);
 }
 
-struct ReadCrateVisitor { e:@mut Env }
-impl visit::Visitor<()> for ReadCrateVisitor {
-    fn visit_view_item(&mut self, a:&ast::view_item, _:()) {
-        visit_view_item(self.e, a);
-        visit::walk_view_item(self, a, ());
-    }
-    fn visit_item(&mut self, a:@ast::item, _:()) {
-        visit_item(self.e, a);
-        visit::walk_item(self, a, ());
-    }
-}
-
 #[deriving(Clone)]
 struct cache_entry {
     cnum: int,

branches/try/src/librustc/middle/lint.rs

@@ -56,7 +56,7 @@ use syntax::visit::Visitor;
  * lint attributes.
  *
  * At each node of the ast which can modify lint attributes, all known lint
- * passes are also applied.  Each lint pass is a visit::Visitor implementator.
+ * passes are also applied.  Each lint pass is an oldvisit::vt<()> structure.
  * The visitors are constructed via the lint_*() functions below. There are
  * also some lint checks which operate directly on ast nodes (such as
  * @ast::item), and those are organized as check_item_*(). Each visitor added
@@ -508,7 +508,7 @@ impl Context {
         }
     }
 
-    fn add_old_lint(&mut self, v: @mut OuterLint) {
+    fn add_oldvisit_lint(&mut self, v: @mut OuterLint) {
         self.visitors.push(OldVisitor(v, v.inner_variant()));
     }
 
@@ -547,7 +547,7 @@ impl Context {
                     }
                 }
             }
-            // Can't use visit::walk_method_helper because the
+            // Can't use oldvisit::visit_method_helper because the
             // item_stopping_visitor has overridden visit_fn(&fk_method(... ))
             // to be a no-op, so manually invoke visit_fn.
             Method(m) => {
@@ -1450,14 +1450,14 @@ pub fn check_crate(tcx: ty::ctxt, crate: @ast::Crate) {
     }
 
     // Register each of the lint passes with the context
-    cx.add_old_lint(lint_while_true());
-    cx.add_old_lint(lint_path_statement());
-    cx.add_old_lint(lint_heap());
-    cx.add_old_lint(lint_type_limits());
-    cx.add_old_lint(lint_unused_unsafe());
-    cx.add_old_lint(lint_unused_mut());
-    cx.add_old_lint(lint_unnecessary_allocations());
-    cx.add_old_lint(lint_missing_doc());
+    cx.add_oldvisit_lint(lint_while_true());
+    cx.add_oldvisit_lint(lint_path_statement());
+    cx.add_oldvisit_lint(lint_heap());
+    cx.add_oldvisit_lint(lint_type_limits());
+    cx.add_oldvisit_lint(lint_unused_unsafe());
+    cx.add_oldvisit_lint(lint_unused_mut());
+    cx.add_oldvisit_lint(lint_unnecessary_allocations());
+    cx.add_oldvisit_lint(lint_missing_doc());
     cx.add_lint(lint_session(cx));
 
     // Actually perform the lint checks (iterating the ast)