Prefer `.cloned()` over `.map(|foo| foo.clone())`

components/net/storage_task.rs

@@ -106,7 +106,7 @@ impl StorageManager {
         let data = self.select_data(storage_type);
         sender.send(data.get(&origin)
                     .and_then(|entry| entry.keys().nth(index as usize))
-                    .map(|key| key.clone())).unwrap();
+                    .cloned()).unwrap();
     }
 
     fn keys(&self,

components/util/prefs.rs

@@ -176,7 +176,7 @@ pub fn reset_pref(name: &str) -> Arc<PrefValue> {
 
 pub fn reset_all_prefs() {
     let names = {
-        PREFS.lock().unwrap().keys().map(|x| x.clone()).collect::<Vec<String>>()
+        PREFS.lock().unwrap().keys().cloned().collect::<Vec<String>>()
     };
     for name in names.iter() {
         reset_pref(name);

tests/reftest.rs

@@ -100,7 +100,7 @@ fn main() {
 
     match run(test_opts,
               all_tests,
-              servo_args.iter().map(|x| x.clone()).collect()) {
+              servo_args.iter().cloned().collect()) {
         Ok(false) => process::exit(1), // tests failed
         Err(_) => process::exit(2),    // I/O-related failure
         _ => (),