Cleanup to structure and decoder.

Author: SpaceManiac

src/bson.rs

@@ -22,7 +22,6 @@
 //! BSON definition
 
 use std::collections::BTreeMap;
-use std::string;
 
 use chrono::{DateTime, UTC};
 use rustc_serialize::json;
@@ -34,13 +33,13 @@ use spec::BinarySubtype;
 pub enum Bson {
     FloatingPoint(f64),
     String(String),
-    Array(self::Array),
-    Document(self::Document),
+    Array(Array),
+    Document(Document),
     Boolean(bool),
     Null,
-    RegExp(string::String, string::String),
-    JavaScriptCode(string::String),
-    JavaScriptCodeWithScope(string::String, self::Document),
+    RegExp(String, String),
+    JavaScriptCode(String),
+    JavaScriptCodeWithScope(String, Document),
     Deprecated,
     I32(i32),
     I64(i64),
@@ -66,32 +65,32 @@ impl Bson {
             &Bson::Null => json::Json::Null,
             &Bson::RegExp(ref pat, ref opt) => {
                 let mut re = json::Object::new();
-                re.insert("pattern".to_string(), json::Json::String(pat.clone()));
-                re.insert("options".to_string(), json::Json::String(opt.clone()));
+                re.insert("pattern".to_owned(), json::Json::String(pat.clone()));
+                re.insert("options".to_owned(), json::Json::String(opt.clone()));
 
                 json::Json::Object(re)
             },
             &Bson::JavaScriptCode(ref code) => json::Json::String(code.clone()),
             &Bson::JavaScriptCodeWithScope(ref code, ref scope) => {
                 let mut obj = json::Object::new();
-                obj.insert("code".to_string(), json::Json::String(code.clone()));
+                obj.insert("code".to_owned(), json::Json::String(code.clone()));
 
                 let scope_obj =
                     scope.iter().map(|(k, v)| (k.clone(), v.to_json())).collect();
 
-                obj.insert("scope".to_string(), json::Json::Object(scope_obj));
+                obj.insert("scope".to_owned(), json::Json::Object(scope_obj));
 
                 json::Json::Object(obj)
             },
-            &Bson::Deprecated => json::Json::String("deprecated".to_string()),
+            &Bson::Deprecated => json::Json::String("deprecated".to_owned()),
             &Bson::I32(v) => json::Json::I64(v as i64),
             &Bson::I64(v) => json::Json::I64(v),
             &Bson::TimeStamp(v) => json::Json::I64(v),
             &Bson::Binary(t, ref v) => {
                 let mut obj = json::Object::new();
                 let tval: u8 = From::from(t);
-                obj.insert("type".to_string(), json::Json::I64(tval as i64));
-                obj.insert("data".to_string(), json::Json::String(v[..].to_hex()));
+                obj.insert("type".to_owned(), json::Json::I64(tval as i64));
+                obj.insert("data".to_owned(), json::Json::String(v[..].to_hex()));
 
                 json::Json::Object(obj)
             },
@@ -107,7 +106,7 @@ impl Bson {
             &json::Json::F64(x) => Bson::FloatingPoint(x),
             &json::Json::String(ref x) => Bson::String(x.clone()),
             &json::Json::Boolean(x) => Bson::Boolean(x),
-            &json::Json::Array(ref x) => Bson::Array(x.iter().map(|x| Bson::from_json(x)).collect()),
+            &json::Json::Array(ref x) => Bson::Array(x.iter().map(Bson::from_json).collect()),
             &json::Json::Object(ref x) => Bson::Document(x.iter().map(|(k, v)| (k.clone(), Bson::from_json(v))).collect()),
             &json::Json::Null => Bson::Null,
         }
@@ -120,7 +119,7 @@ pub trait ToBson {
 
 impl ToBson for str {
     fn to_bson(&self) -> Bson {
-        Bson::String(self.to_string())
+        Bson::String(self.to_owned())
     }
 }