From ba01ea3730025028affbf4ce56f29621861779c6 Mon Sep 17 00:00:00 2001 From: Jorge Aparicio Date: Thu, 27 Nov 2014 19:36:41 -0500 Subject: [PATCH] libserialize: remove unnecessary `to_string()` calls --- src/libserialize/base64.rs | 24 ++++----- src/libserialize/hex.rs | 2 +- src/libserialize/json.rs | 100 ++++++++++++++++++------------------- 3 files changed, 63 insertions(+), 63 deletions(-) diff --git a/src/libserialize/base64.rs b/src/libserialize/base64.rs index 8efb8e5b78d..c8ec1700a1d 100644 --- a/src/libserialize/base64.rs +++ b/src/libserialize/base64.rs @@ -286,13 +286,13 @@ mod tests { #[test] fn test_to_base64_basic() { - assert_eq!("".as_bytes().to_base64(STANDARD), "".to_string()); - assert_eq!("f".as_bytes().to_base64(STANDARD), "Zg==".to_string()); - assert_eq!("fo".as_bytes().to_base64(STANDARD), "Zm8=".to_string()); - assert_eq!("foo".as_bytes().to_base64(STANDARD), "Zm9v".to_string()); - assert_eq!("foob".as_bytes().to_base64(STANDARD), "Zm9vYg==".to_string()); - assert_eq!("fooba".as_bytes().to_base64(STANDARD), "Zm9vYmE=".to_string()); - assert_eq!("foobar".as_bytes().to_base64(STANDARD), "Zm9vYmFy".to_string()); + assert_eq!("".as_bytes().to_base64(STANDARD), ""); + assert_eq!("f".as_bytes().to_base64(STANDARD), "Zg=="); + assert_eq!("fo".as_bytes().to_base64(STANDARD), "Zm8="); + assert_eq!("foo".as_bytes().to_base64(STANDARD), "Zm9v"); + assert_eq!("foob".as_bytes().to_base64(STANDARD), "Zm9vYg=="); + assert_eq!("fooba".as_bytes().to_base64(STANDARD), "Zm9vYmE="); + assert_eq!("foobar".as_bytes().to_base64(STANDARD), "Zm9vYmFy"); } #[test] @@ -301,19 +301,19 @@ mod tests { .contains("\r\n")); assert_eq!("foobar".as_bytes().to_base64(Config {line_length: Some(4), ..STANDARD}), - "Zm9v\r\nYmFy".to_string()); + "Zm9v\r\nYmFy"); } #[test] fn test_to_base64_padding() { - assert_eq!("f".as_bytes().to_base64(Config {pad: false, ..STANDARD}), "Zg".to_string()); - assert_eq!("fo".as_bytes().to_base64(Config {pad: false, ..STANDARD}), "Zm8".to_string()); + assert_eq!("f".as_bytes().to_base64(Config {pad: false, ..STANDARD}), "Zg"); + assert_eq!("fo".as_bytes().to_base64(Config {pad: false, ..STANDARD}), "Zm8"); } #[test] fn test_to_base64_url_safe() { - assert_eq!([251, 255].to_base64(URL_SAFE), "-_8".to_string()); - assert_eq!([251, 255].to_base64(STANDARD), "+/8=".to_string()); + assert_eq!([251, 255].to_base64(URL_SAFE), "-_8"); + assert_eq!([251, 255].to_base64(STANDARD), "+/8="); } #[test] diff --git a/src/libserialize/hex.rs b/src/libserialize/hex.rs index 443a31f7493..4c20f72cac5 100644 --- a/src/libserialize/hex.rs +++ b/src/libserialize/hex.rs @@ -158,7 +158,7 @@ mod tests { #[test] pub fn test_to_hex() { - assert_eq!("foobar".as_bytes().to_hex(), "666f6f626172".to_string()); + assert_eq!("foobar".as_bytes().to_hex(), "666f6f626172"); } #[test] diff --git a/src/libserialize/json.rs b/src/libserialize/json.rs index abf2341a76c..248d78236ad 100644 --- a/src/libserialize/json.rs +++ b/src/libserialize/json.rs @@ -2486,76 +2486,76 @@ mod tests { #[test] fn test_write_null() { - assert_eq!(Null.to_string().into_string(), "null".to_string()); - assert_eq!(Null.to_pretty_str().into_string(), "null".to_string()); + assert_eq!(Null.to_string().into_string(), "null"); + assert_eq!(Null.to_pretty_str().into_string(), "null"); } #[test] fn test_write_i64() { - assert_eq!(U64(0).to_string().into_string(), "0".to_string()); - assert_eq!(U64(0).to_pretty_str().into_string(), "0".to_string()); + assert_eq!(U64(0).to_string().into_string(), "0"); + assert_eq!(U64(0).to_pretty_str().into_string(), "0"); - assert_eq!(U64(1234).to_string().into_string(), "1234".to_string()); - assert_eq!(U64(1234).to_pretty_str().into_string(), "1234".to_string()); + assert_eq!(U64(1234).to_string().into_string(), "1234"); + assert_eq!(U64(1234).to_pretty_str().into_string(), "1234"); - assert_eq!(I64(-5678).to_string().into_string(), "-5678".to_string()); - assert_eq!(I64(-5678).to_pretty_str().into_string(), "-5678".to_string()); + assert_eq!(I64(-5678).to_string().into_string(), "-5678"); + assert_eq!(I64(-5678).to_pretty_str().into_string(), "-5678"); } #[test] fn test_write_f64() { - assert_eq!(F64(3.0).to_string().into_string(), "3".to_string()); - assert_eq!(F64(3.0).to_pretty_str().into_string(), "3".to_string()); + assert_eq!(F64(3.0).to_string().into_string(), "3"); + assert_eq!(F64(3.0).to_pretty_str().into_string(), "3"); - assert_eq!(F64(3.1).to_string().into_string(), "3.1".to_string()); - assert_eq!(F64(3.1).to_pretty_str().into_string(), "3.1".to_string()); + assert_eq!(F64(3.1).to_string().into_string(), "3.1"); + assert_eq!(F64(3.1).to_pretty_str().into_string(), "3.1"); - assert_eq!(F64(-1.5).to_string().into_string(), "-1.5".to_string()); - assert_eq!(F64(-1.5).to_pretty_str().into_string(), "-1.5".to_string()); + assert_eq!(F64(-1.5).to_string().into_string(), "-1.5"); + assert_eq!(F64(-1.5).to_pretty_str().into_string(), "-1.5"); - assert_eq!(F64(0.5).to_string().into_string(), "0.5".to_string()); - assert_eq!(F64(0.5).to_pretty_str().into_string(), "0.5".to_string()); + assert_eq!(F64(0.5).to_string().into_string(), "0.5"); + assert_eq!(F64(0.5).to_pretty_str().into_string(), "0.5"); - assert_eq!(F64(f64::NAN).to_string().into_string(), "null".to_string()); - assert_eq!(F64(f64::NAN).to_pretty_str().into_string(), "null".to_string()); + assert_eq!(F64(f64::NAN).to_string().into_string(), "null"); + assert_eq!(F64(f64::NAN).to_pretty_str().into_string(), "null"); - assert_eq!(F64(f64::INFINITY).to_string().into_string(), "null".to_string()); - assert_eq!(F64(f64::INFINITY).to_pretty_str().into_string(), "null".to_string()); + assert_eq!(F64(f64::INFINITY).to_string().into_string(), "null"); + assert_eq!(F64(f64::INFINITY).to_pretty_str().into_string(), "null"); - assert_eq!(F64(f64::NEG_INFINITY).to_string().into_string(), "null".to_string()); - assert_eq!(F64(f64::NEG_INFINITY).to_pretty_str().into_string(), "null".to_string()); + assert_eq!(F64(f64::NEG_INFINITY).to_string().into_string(), "null"); + assert_eq!(F64(f64::NEG_INFINITY).to_pretty_str().into_string(), "null"); } #[test] fn test_write_str() { - assert_eq!(String("".to_string()).to_string().into_string(), "\"\"".to_string()); - assert_eq!(String("".to_string()).to_pretty_str().into_string(), "\"\"".to_string()); + assert_eq!(String("".to_string()).to_string().into_string(), "\"\""); + assert_eq!(String("".to_string()).to_pretty_str().into_string(), "\"\""); - assert_eq!(String("foo".to_string()).to_string().into_string(), "\"foo\"".to_string()); - assert_eq!(String("foo".to_string()).to_pretty_str().into_string(), "\"foo\"".to_string()); + assert_eq!(String("foo".to_string()).to_string().into_string(), "\"foo\""); + assert_eq!(String("foo".to_string()).to_pretty_str().into_string(), "\"foo\""); } #[test] fn test_write_bool() { - assert_eq!(Boolean(true).to_string().into_string(), "true".to_string()); - assert_eq!(Boolean(true).to_pretty_str().into_string(), "true".to_string()); + assert_eq!(Boolean(true).to_string().into_string(), "true"); + assert_eq!(Boolean(true).to_pretty_str().into_string(), "true"); - assert_eq!(Boolean(false).to_string().into_string(), "false".to_string()); - assert_eq!(Boolean(false).to_pretty_str().into_string(), "false".to_string()); + assert_eq!(Boolean(false).to_string().into_string(), "false"); + assert_eq!(Boolean(false).to_pretty_str().into_string(), "false"); } #[test] fn test_write_array() { - assert_eq!(Array(vec![]).to_string().into_string(), "[]".to_string()); - assert_eq!(Array(vec![]).to_pretty_str().into_string(), "[]".to_string()); + assert_eq!(Array(vec![]).to_string().into_string(), "[]"); + assert_eq!(Array(vec![]).to_pretty_str().into_string(), "[]"); - assert_eq!(Array(vec![Boolean(true)]).to_string().into_string(), "[true]".to_string()); + assert_eq!(Array(vec![Boolean(true)]).to_string().into_string(), "[true]"); assert_eq!( Array(vec![Boolean(true)]).to_pretty_str().into_string(), "\ [\n \ true\n\ - ]".to_string() + ]" ); let long_test_array = Array(vec![ @@ -2564,7 +2564,7 @@ mod tests { Array(vec![String("foo\nbar".to_string()), F64(3.5)])]); assert_eq!(long_test_array.to_string().into_string(), - "[false,null,[\"foo\\nbar\",3.5]]".to_string()); + "[false,null,[\"foo\\nbar\",3.5]]"); assert_eq!( long_test_array.to_pretty_str().into_string(), "\ @@ -2575,27 +2575,27 @@ mod tests { \"foo\\nbar\",\n \ 3.5\n \ ]\n\ - ]".to_string() + ]" ); } #[test] fn test_write_object() { - assert_eq!(mk_object(&[]).to_string().into_string(), "{}".to_string()); - assert_eq!(mk_object(&[]).to_pretty_str().into_string(), "{}".to_string()); + assert_eq!(mk_object(&[]).to_string().into_string(), "{}"); + assert_eq!(mk_object(&[]).to_pretty_str().into_string(), "{}"); assert_eq!( mk_object(&[ ("a".to_string(), Boolean(true)) ]).to_string().into_string(), - "{\"a\":true}".to_string() + "{\"a\":true}" ); assert_eq!( mk_object(&[("a".to_string(), Boolean(true))]).to_pretty_str(), "\ {\n \ \"a\": true\n\ - }".to_string() + }" ); let complex_obj = mk_object(&[ @@ -2612,7 +2612,7 @@ mod tests { {\"c\":\"\\f\\r\"},\ {\"d\":\"\"}\ ]\ - }".to_string() + }" ); assert_eq!( complex_obj.to_pretty_str().into_string(), @@ -2626,7 +2626,7 @@ mod tests { \"d\": \"\"\n \ }\n \ ]\n\ - }".to_string() + }" ); let a = mk_object(&[ @@ -2660,14 +2660,14 @@ mod tests { let mut encoder = Encoder::new(writer); animal.encode(&mut encoder).unwrap(); }), - "\"Dog\"".to_string() + "\"Dog\"" ); assert_eq!( with_str_writer(|writer| { let mut encoder = PrettyEncoder::new(writer); animal.encode(&mut encoder).unwrap(); }), - "\"Dog\"".to_string() + "\"Dog\"" ); let animal = Frog("Henry".to_string(), 349); @@ -2676,7 +2676,7 @@ mod tests { let mut encoder = Encoder::new(writer); animal.encode(&mut encoder).unwrap(); }), - "{\"variant\":\"Frog\",\"fields\":[\"Henry\",349]}".to_string() + "{\"variant\":\"Frog\",\"fields\":[\"Henry\",349]}" ); assert_eq!( with_str_writer(|writer| { @@ -2689,7 +2689,7 @@ mod tests { \"Henry\",\n \ 349\n \ ]\n\ - }".to_string() + }" ); } @@ -2700,14 +2700,14 @@ mod tests { let mut encoder = Encoder::new(writer); value.encode(&mut encoder).unwrap(); }); - assert_eq!(s, "\"jodhpurs\"".to_string()); + assert_eq!(s, "\"jodhpurs\""); let value = Some("jodhpurs".to_string()); let s = with_str_writer(|writer| { let mut encoder = PrettyEncoder::new(writer); value.encode(&mut encoder).unwrap(); }); - assert_eq!(s, "\"jodhpurs\"".to_string()); + assert_eq!(s, "\"jodhpurs\""); } #[test] @@ -2717,13 +2717,13 @@ mod tests { let mut encoder = Encoder::new(writer); value.encode(&mut encoder).unwrap(); }); - assert_eq!(s, "null".to_string()); + assert_eq!(s, "null"); let s = with_str_writer(|writer| { let mut encoder = Encoder::new(writer); value.encode(&mut encoder).unwrap(); }); - assert_eq!(s, "null".to_string()); + assert_eq!(s, "null"); } #[test]