]> git.lizzy.rs Git - rust.git/commitdiff
Change to_str().to_string() to just to_str()
authorAdolfo Ochagavía <aochagavia92@gmail.com>
Thu, 5 Jun 2014 07:15:19 +0000 (09:15 +0200)
committerAdolfo Ochagavía <aochagavia92@gmail.com>
Fri, 6 Jun 2014 07:56:59 +0000 (09:56 +0200)
31 files changed:
src/compiletest/errors.rs
src/compiletest/util.rs
src/doc/complement-cheatsheet.md
src/doc/guide-tasks.md
src/doc/rust.md
src/libgetopts/lib.rs
src/libnum/complex.rs
src/libnum/rational.rs
src/libregex_macros/lib.rs
src/librustc/driver/driver.rs
src/librustc/middle/liveness.rs
src/librustc/middle/mem_categorization.rs
src/librustc/middle/trans/base.rs
src/librustc/middle/typeck/infer/to_str.rs
src/librustc/util/ppaux.rs
src/librustdoc/clean/inline.rs
src/librustdoc/clean/mod.rs
src/librustdoc/html/format.rs
src/librustdoc/html/markdown.rs
src/librustdoc/lib.rs
src/libsyntax/ext/source_util.rs
src/libsyntax/ext/tt/macro_rules.rs
src/libsyntax/parse/token.rs
src/libtest/lib.rs
src/libtime/lib.rs
src/liburl/lib.rs
src/test/bench/core-set.rs
src/test/run-pass/monad.rs
src/test/run-pass/static-impl.rs
src/test/run-pass/trait-cast.rs
src/test/run-pass/trait-to-str.rs

index c96e688c2900cae3d864b04715d2ef9df5cd5b7d..8e79f58c60881d21399b2c98f0ab750f98222a73 100644 (file)
@@ -31,7 +31,7 @@ pub fn load_errors(re: &Regex, testfile: &Path) -> Vec<ExpectedError> {
 fn parse_expected(line_num: uint, line: &str, re: &Regex) -> Option<ExpectedError> {
     re.captures(line).and_then(|caps| {
         let adjusts = caps.name("adjusts").len();
-        let kind = caps.name("kind").to_ascii().to_lower().into_str().to_string();
+        let kind = caps.name("kind").to_ascii().to_lower().into_str();
         let msg = caps.name("msg").trim().to_string();
 
         debug!("line={} kind={} msg={}", line_num, kind, msg);
index 00f0689b96b660b01295edafec35d65af743d929..84bf0eb11553e3a5be977aada46ec7ff9a433240 100644 (file)
@@ -41,7 +41,7 @@ pub fn make_new_path(path: &str) -> String {
       Some(curr) => {
         format!("{}{}{}", path, path_div(), curr)
       }
-      None => path.to_str().to_string()
+      None => path.to_str()
     }
 }
 
index 5ad0e82658d1a7c7eb7b1a4468e3603cd6b16cef..9797284a65b7d6e58ad40700a438be000a030db3 100644 (file)
@@ -8,7 +8,7 @@ Use [`ToStr`](std/to_str/trait.ToStr.html).
 
 ~~~
 let x: int = 42;
-let y: String = x.to_str().to_string();
+let y: String = x.to_str();
 ~~~
 
 **String to int**
index 6b45174591462776f7cf027c5376316fb8eab2f8..957b1d6ccc671d8c93ae80bd09609c381afc9cf2 100644 (file)
@@ -467,7 +467,7 @@ fn stringifier(channel: &sync::DuplexStream<String, uint>) {
     let mut value: uint;
     loop {
         value = channel.recv();
-        channel.send(value.to_str().to_string());
+        channel.send(value.to_str());
         if value == 0 { break; }
     }
 }
@@ -492,7 +492,7 @@ extern crate sync;
 #     let mut value: uint;
 #     loop {
 #         value = channel.recv();
-#         channel.send(value.to_str().to_string());
+#         channel.send(value.to_str());
 #         if value == 0u { break; }
 #     }
 # }
index b690c2eb983474635ccc9565b417576f06e09fd5..abd5de4d10c93ef3852160cc679ae8bfe02e3b7d 100644 (file)
@@ -3579,7 +3579,7 @@ trait Printable {
 }
 
 impl Printable for int {
-  fn to_string(&self) -> String { self.to_str().to_string() }
+  fn to_string(&self) -> String { self.to_str() }
 }
 
 fn print(a: Box<Printable>) {
index 0311c3339241a5a2a9505126e2e5f7de3a2198ca..ddfe8380e09dd5c4a681f327db28f958c654a180 100644 (file)
@@ -222,7 +222,7 @@ fn from_str(nm: &str) -> Name {
 
     fn to_str(&self) -> String {
         match *self {
-            Short(ch) => ch.to_str().to_string(),
+            Short(ch) => ch.to_str(),
             Long(ref s) => s.to_string()
         }
     }
index 7febeb2661658571d0d46decd4a4b85e8241a29e..9ee80d283cf924621ac20309c075ea29d8fcf371 100644 (file)
@@ -349,7 +349,7 @@ fn test_neg() {
     #[test]
     fn test_to_str() {
         fn test(c : Complex64, s: String) {
-            assert_eq!(c.to_str().to_string(), s);
+            assert_eq!(c.to_str(), s);
         }
         test(_0_0i, "0+0i".to_string());
         test(_1_0i, "1+0i".to_string());
index 3efc359fd3fb343f600e851692b5fb96a92f9293..faf05365c04a0eef764cc89a20afdf3cc2663dda 100644 (file)
@@ -559,7 +559,7 @@ fn test_recip() {
     fn test_to_from_str() {
         fn test(r: Rational, s: String) {
             assert_eq!(FromStr::from_str(s.as_slice()), Some(r));
-            assert_eq!(r.to_str().to_string(), s);
+            assert_eq!(r.to_str(), s);
         }
         test(_1, "1/1".to_string());
         test(_0, "0/1".to_string());
index 71e3d06cf967c9e72c3a27085a4d7518eb30ccc0..89d6620f1278243fc9923a3ac20fd7de6d3a2e6b 100644 (file)
@@ -611,7 +611,7 @@ fn parse(cx: &mut ExtCtxt, tts: &[ast::TokenTree]) -> Option<String> {
     let regex = match entry.node {
         ast::ExprLit(lit) => {
             match lit.node {
-                ast::LitStr(ref s, _) => s.to_str().to_string(),
+                ast::LitStr(ref s, _) => s.to_str(),
                 _ => {
                     cx.span_err(entry.span, format!(
                         "expected string literal but got `{}`",
index d04df996186dde7c68086c6c60d352648755626e..89c0a381cf9f295afa4e38ab1b797eba50de5978 100644 (file)
@@ -533,7 +533,7 @@ fn post(&self,
         match node {
             pprust::NodeItem(item) => {
                 try!(pp::space(&mut s.s));
-                s.synth_comment(item.id.to_str().to_string())
+                s.synth_comment(item.id.to_str())
             }
             pprust::NodeBlock(blk) => {
                 try!(pp::space(&mut s.s));
@@ -541,7 +541,7 @@ fn post(&self,
             }
             pprust::NodeExpr(expr) => {
                 try!(pp::space(&mut s.s));
-                try!(s.synth_comment(expr.id.to_str().to_string()));
+                try!(s.synth_comment(expr.id.to_str()));
                 s.pclose()
             }
             pprust::NodePat(pat) => {
index ce02243403ff2ea4306e1d182da1345d5c5e4c52..b5cb0f8e5aac8b12b47e4d7075695ffcd4790f94 100644 (file)
@@ -324,7 +324,7 @@ fn variable(&self, node_id: NodeId, span: Span) -> Variable {
     fn variable_name(&self, var: Variable) -> String {
         match self.var_kinds.get(var.get()) {
             &Local(LocalInfo { ident: nm, .. }) | &Arg(_, nm) => {
-                token::get_ident(nm).get().to_str().to_string()
+                token::get_ident(nm).get().to_str()
             },
             &ImplicitRet => "<implicit-ret>".to_string()
         }
index e6b48f024833452ddfc87be1df10a9ae0cd02ded..1d2ed72f70ea0283f8c53c5430571279985f0c3d 100644 (file)
@@ -1303,7 +1303,7 @@ impl Repr for InteriorKind {
     fn repr(&self, _tcx: &ty::ctxt) -> String {
         match *self {
             InteriorField(NamedField(fld)) => {
-                token::get_name(fld).get().to_str().to_string()
+                token::get_name(fld).get().to_str()
             }
             InteriorField(PositionalField(i)) => format!("\\#{:?}", i),
             InteriorElement(_) => "[]".to_string(),
index 464e8cff0fa67b9b5a80dcb3d7421bf4101789db..6096060f975e9eab53456b797f3e5d4ceec3cc17 100644 (file)
@@ -1927,7 +1927,7 @@ fn exported_name(ccx: &CrateContext, id: ast::NodeId,
         _ => ccx.tcx.map.with_path(id, |mut path| {
             if attr::contains_name(attrs, "no_mangle") {
                 // Don't mangle
-                path.last().unwrap().to_str().to_string()
+                path.last().unwrap().to_str()
             } else {
                 match weak_lang_items::link_name(attrs) {
                     Some(name) => name.get().to_string(),
index f03ecd9aee158322f0c81d1332a8383851fd4f1f..59d73c7fb1f2d917c94e03055688aea29d08385c 100644 (file)
@@ -79,13 +79,13 @@ fn inf_str(&self, cx: &InferCtxt) -> String {
 
 impl InferStr for IntVarValue {
     fn inf_str(&self, _cx: &InferCtxt) -> String {
-        self.to_str().to_string()
+        self.to_str()
     }
 }
 
 impl InferStr for ast::FloatTy {
     fn inf_str(&self, _cx: &InferCtxt) -> String {
-        self.to_str().to_string()
+        self.to_str()
     }
 }
 
index f73666bebf6d7c3ec58cc4c025f608b29f3c8e34..e14fd89fc74d72ce6bc7a1bd3dd4fdc9b424fbcb 100644 (file)
@@ -366,7 +366,7 @@ fn push_sig_to_str(cx: &ctxt,
       ty_bare_fn(ref f) => {
           bare_fn_to_str(cx, f.fn_style, f.abi, None, &f.sig)
       }
-      ty_infer(infer_ty) => infer_ty.to_str().to_string(),
+      ty_infer(infer_ty) => infer_ty.to_str(),
       ty_err => "[type error]".to_string(),
       ty_param(param_ty {idx: id, def_id: did}) => {
           let ident = match cx.ty_param_defs.borrow().find(&did.node) {
@@ -753,7 +753,10 @@ fn repr(&self, tcx: &ctxt) -> String {
 
 impl Repr for ty::Variance {
     fn repr(&self, _: &ctxt) -> String {
-        self.to_str().to_string()
+        // The first `.to_str()` returns a &'static str (it is not an implementation
+        // of the ToStr trait). Because of that, we need to call `.to_str()` again
+        // if we want to have a `String`.
+        self.to_str().to_str()
     }
 }
 
@@ -950,13 +953,13 @@ fn user_string(&self, _tcx: &ctxt) -> String {
 
 impl Repr for abi::Abi {
     fn repr(&self, _tcx: &ctxt) -> String {
-        self.to_str().to_string()
+        self.to_str()
     }
 }
 
 impl UserString for abi::Abi {
     fn user_string(&self, _tcx: &ctxt) -> String {
-        self.to_str().to_string()
+        self.to_str()
     }
 }
 
index 575dd057867fb24b7e7df3d5ce3058c5aa7ba5f2..397476856fccb750fce2a452fd99ee471ca91d66 100644 (file)
@@ -93,7 +93,7 @@ fn try_inline_def(cx: &core::DocContext,
     cx.inlined.borrow_mut().get_mut_ref().insert(did);
     ret.push(clean::Item {
         source: clean::Span::empty(),
-        name: Some(fqn.last().unwrap().to_str().to_string()),
+        name: Some(fqn.last().unwrap().to_str()),
         attrs: load_attrs(tcx, did),
         inner: inner,
         visibility: Some(ast::Public),
index 1992c102e474e5f748a47942adbaedc85c9e1f6d..4614d7cee3a600504076e3b352bc9fa28181e187 100644 (file)
@@ -526,7 +526,7 @@ fn clean(&self) -> TyParamBound {
                  external_path("Share", &empty)),
         };
         let fqn = csearch::get_item_path(tcx, did);
-        let fqn = fqn.move_iter().map(|i| i.to_str().to_string()).collect();
+        let fqn = fqn.move_iter().map(|i| i.to_str()).collect();
         cx.external_paths.borrow_mut().get_mut_ref().insert(did,
                                                             (fqn, TypeTrait));
         TraitBound(ResolvedPath {
@@ -545,7 +545,7 @@ fn clean(&self) -> TyParamBound {
             core::NotTyped(_) => return RegionBound,
         };
         let fqn = csearch::get_item_path(tcx, self.def_id);
-        let fqn = fqn.move_iter().map(|i| i.to_str().to_string())
+        let fqn = fqn.move_iter().map(|i| i.to_str())
                      .collect::<Vec<String>>();
         let path = external_path(fqn.last().unwrap().as_slice(),
                                  &self.substs);
@@ -1239,7 +1239,7 @@ fn clean(&self) -> Type {
                 };
                 let fqn = csearch::get_item_path(tcx, did);
                 let fqn: Vec<String> = fqn.move_iter().map(|i| {
-                    i.to_str().to_string()
+                    i.to_str()
                 }).collect();
                 let kind = match ty::get(*self).sty {
                     ty::ty_struct(..) => TypeStruct,
@@ -1617,7 +1617,7 @@ fn clean(&self) -> BareFunctionDecl {
                 type_params: Vec::new(),
             },
             decl: self.decl.clean(),
-            abi: self.abi.to_str().to_string(),
+            abi: self.abi.to_str(),
         }
     }
 }
@@ -1891,12 +1891,12 @@ fn lit_to_str(lit: &ast::Lit) -> String {
         ast::LitStr(ref st, _) => st.get().to_string(),
         ast::LitBinary(ref data) => format!("{:?}", data.as_slice()),
         ast::LitChar(c) => format!("'{}'", c),
-        ast::LitInt(i, _t) => i.to_str().to_string(),
-        ast::LitUint(u, _t) => u.to_str().to_string(),
-        ast::LitIntUnsuffixed(i) => i.to_str().to_string(),
+        ast::LitInt(i, _t) => i.to_str(),
+        ast::LitUint(u, _t) => u.to_str(),
+        ast::LitIntUnsuffixed(i) => i.to_str(),
         ast::LitFloat(ref f, _t) => f.get().to_string(),
         ast::LitFloatUnsuffixed(ref f) => f.get().to_string(),
-        ast::LitBool(b) => b.to_str().to_string(),
+        ast::LitBool(b) => b.to_str(),
         ast::LitNil => "".to_string(),
     }
 }
index 41d84deea6f4105661c96cdf66cfb742ef8064a7..51d2a67d6cbf410f9148a1ca8d074254aa4590d6 100644 (file)
@@ -401,7 +401,7 @@ fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
                        } else {
                            let mut m = decl.bounds
                                            .iter()
-                                           .map(|s| s.to_str().to_string());
+                                           .map(|s| s.to_str());
                            format!(
                                ": {}",
                                m.collect::<Vec<String>>().connect(" + "))
index 406bdc48af3b24c0222c898247db1f5fed84acb5..e0a8a459a6acba616ded543c7003c7b506fe202b 100644 (file)
@@ -208,7 +208,7 @@ pub fn render(w: &mut fmt::Formatter, s: &str, print_toc: bool) -> fmt::Result {
         // Transform the contents of the header into a hyphenated string
         let id = (s.as_slice().words().map(|s| {
             match s.to_ascii_opt() {
-                Some(s) => s.to_lower().into_str().to_string(),
+                Some(s) => s.to_lower().into_str(),
                 None => s.to_string()
             }
         }).collect::<Vec<String>>().connect("-")).to_string();
index fe53439703adb39116a8c1bc876197ad621707a3..58afd13f1a4a5703142cdeed4c2cfd3ee620f64d 100644 (file)
@@ -360,7 +360,7 @@ fn json_input(input: &str) -> Result<Output, String> {
         }
     };
     match json::from_reader(&mut input) {
-        Err(s) => Err(s.to_str().to_string()),
+        Err(s) => Err(s.to_str()),
         Ok(json::Object(obj)) => {
             let mut obj = obj;
             // Make sure the schema is what we expect
index d2e689b5934f050881fd1462a5f0027ed6d71b5a..93b66ede2671f3a87d895c2d976c96c93f44f3c7 100644 (file)
@@ -125,7 +125,7 @@ pub fn expand_include_str(cx: &mut ExtCtxt, sp: Span, tts: &[ast::TokenTree])
         Some(src) => {
             // Add this input file to the code map to make it available as
             // dependency information
-            let filename = file.display().to_str().to_string();
+            let filename = file.display().to_str();
             let interned = token::intern_and_get_ident(src);
             cx.codemap().new_filemap(filename, src.to_string());
 
index 0622bf76ab92529966dd2a8dfcc309ab6ed55d64..c78d4d258f6f5ef317363c3844f5b3b71c69f962 100644 (file)
@@ -252,7 +252,7 @@ fn ms(m: Matcher_) -> Matcher {
 
     box MacroRulesDefiner {
         def: RefCell::new(Some(MacroDef {
-            name: token::get_ident(name).to_str().to_string(),
+            name: token::get_ident(name).to_str(),
             ext: NormalTT(exp, Some(sp))
         }))
     } as Box<MacResult>
index 42319eeb371d6f66751979de099826282797786e..129ea5fdf6d0406d8e98dd6a8bf4c54f90d174ec 100644 (file)
@@ -205,7 +205,7 @@ pub fn to_str(t: &Token) -> String {
                                                ast_util::ForceSuffix),
       LIT_UINT(u, t) => ast_util::uint_ty_to_str(t, Some(u),
                                                  ast_util::ForceSuffix),
-      LIT_INT_UNSUFFIXED(i) => { (i as u64).to_str().to_string() }
+      LIT_INT_UNSUFFIXED(i) => { (i as u64).to_str() }
       LIT_FLOAT(s, t) => {
         let mut body = String::from_str(get_ident(s).get());
         if body.as_slice().ends_with(".") {
index 4d408864dc513548c975a1acf9e559d86855cf43..c1aa2c77df1d2ee86f195e7128809c48bc98ff82 100644 (file)
@@ -1503,7 +1503,7 @@ pub fn filter_for_ignored_option() {
         let filtered = filter_tests(&opts, tests);
 
         assert_eq!(filtered.len(), 1);
-        assert_eq!(filtered.get(0).desc.name.to_str().to_string(),
+        assert_eq!(filtered.get(0).desc.name.to_str(),
                    "1".to_string());
         assert!(filtered.get(0).desc.ignore == false);
     }
@@ -1554,7 +1554,7 @@ fn testfn() { }
                  "test::sort_tests".to_string());
 
         for (a, b) in expected.iter().zip(filtered.iter()) {
-            assert!(*a == b.desc.name.to_str().to_string());
+            assert!(*a == b.desc.name.to_str());
         }
     }
 
index a478aa931d444cd6d338b371f0f14d33463b2748..90af1da78a457d096655d5ff8e6ac30318c63d31 100644 (file)
@@ -1020,7 +1020,7 @@ fn parse_type(ch: char, tm: &Tm) -> String {
           'U' => format!("{:02d}", (tm.tm_yday - tm.tm_wday + 7) / 7),
           'u' => {
             let i = tm.tm_wday as int;
-            (if i == 0 { 7 } else { i }).to_str().to_string()
+            (if i == 0 { 7 } else { i }).to_str()
           }
           'V' => iso_week('V', tm),
           'v' => {
@@ -1033,8 +1033,8 @@ fn parse_type(ch: char, tm: &Tm) -> String {
               format!("{:02d}",
                              (tm.tm_yday - (tm.tm_wday - 1 + 7) % 7 + 7) / 7)
           }
-          'w' => (tm.tm_wday as int).to_str().to_string(),
-          'Y' => (tm.tm_year as int + 1900).to_str().to_string(),
+          'w' => (tm.tm_wday as int).to_str(),
+          'Y' => (tm.tm_year as int + 1900).to_str(),
           'y' => format!("{:02d}", (tm.tm_year as int + 1900) % 100),
           'Z' => "".to_string(),    // FIXME(pcwalton): Implement this.
           'z' => {
index 5da6c5afe4278d7bb3acbf1609ff509ffde88c1b..af8db85a5c8d072f849beb44a4043bcb85502869 100644 (file)
@@ -527,7 +527,7 @@ fn get_authority(rawurl: &str) ->
     Result<(Option<UserInfo>, String, Option<String>, String), String> {
     if !rawurl.starts_with("//") {
         // there is no authority.
-        return Ok((None, "".to_string(), None, rawurl.to_str().to_string()));
+        return Ok((None, "".to_string(), None, rawurl.to_str()));
     }
 
     enum State {
index ab2f9b0020624ba6b0c0791a6bcd0bf275534d51..d229d958014cd842c7637a56f8c429398de9c202 100644 (file)
@@ -90,11 +90,11 @@ pub fn bench_str<T:MutableSet<String>,
             let mut set = f();
             timed(&mut self.sequential_strings, || {
                 for i in range(0u, num_keys) {
-                    set.insert(i.to_str().to_string());
+                    set.insert(i.to_str());
                 }
 
                 for i in range(0u, num_keys) {
-                    assert!(set.contains(&i.to_str().to_string()));
+                    assert!(set.contains(&i.to_str()));
                 }
             })
         }
@@ -103,7 +103,7 @@ pub fn bench_str<T:MutableSet<String>,
             let mut set = f();
             timed(&mut self.random_strings, || {
                 for _ in range(0, num_keys) {
-                    let s = rng.gen::<uint>().to_str().to_string();
+                    let s = rng.gen::<uint>().to_str();
                     set.insert(s);
                 }
             })
@@ -112,11 +112,11 @@ pub fn bench_str<T:MutableSet<String>,
         {
             let mut set = f();
             for i in range(0u, num_keys) {
-                set.insert(i.to_str().to_string());
+                set.insert(i.to_str());
             }
             timed(&mut self.delete_strings, || {
                 for i in range(0u, num_keys) {
-                    assert!(set.remove(&i.to_str().to_string()));
+                    assert!(set.remove(&i.to_str()));
                 }
             })
         }
index 3ad5da317013fd05f7422798338c6d5bfd8da4a3..2b67ef09c59db5152d610b4ee9741dcba2488888 100644 (file)
@@ -38,7 +38,7 @@ fn bind<B>(&self, f: |&A| -> Option<B>) -> Option<B> {
 }
 
 fn transform(x: Option<int>) -> Option<String> {
-    x.bind(|n| Some(*n + 1) ).bind(|n| Some(n.to_str().to_string()) )
+    x.bind(|n| Some(*n + 1) ).bind(|n| Some(n.to_str()) )
 }
 
 pub fn main() {
index 1545de93786eeea2a26977d95abe27c1750d02f3..f6c314b20dd14fab9a0f18dc13338dbd812d5f08 100644 (file)
@@ -31,7 +31,7 @@ trait uint_utils {
 
 impl uint_utils for uint {
     fn str(&self) -> String {
-        self.to_str().to_string()
+        self.to_str()
     }
     fn multi(&self, f: |uint|) {
         let mut c = 0u;
index 454cf4c8eda21294b382ddd8089c181e6391ba3a..45c70bc2d945e48f73b0fe30a0944480a6a1b887 100644 (file)
@@ -36,7 +36,7 @@ fn to_str_(&self) -> String {
 
 impl to_str for int {
     fn to_str_(&self) -> String {
-        self.to_str().to_string()
+        self.to_str()
     }
 }
 
index ea5e0a9ee910b7698e6c80ffbca72356d362aab6..cd7c5c6f8f7d48ae6e94950e43cfe4d3fbf8b8cd 100644 (file)
@@ -15,7 +15,7 @@ trait to_str {
 }
 
 impl to_str for int {
-    fn to_string(&self) -> String { self.to_str().to_string() }
+    fn to_string(&self) -> String { self.to_str() }
 }
 
 impl<T:to_str> to_str for Vec<T> {