Changes from the review of the @str PR.

Author: huonw

src/librustc/metadata/cstore.rs

@@ -215,15 +215,7 @@ impl CStore {
             debug!("  hash[{}]: {}", x.name, x.hash);
         }
 
-        let mut hashes = ~[];
-        for ch in result.move_iter() {
-            let crate_hash {
-                hash,
-                ..
-            } = ch;
-            hashes.push(hash)
-        }
-        hashes
+        result.move_iter().map(|crate_hash { hash, ..}| hash).collect()
     }
 }
 

src/librustdoc/clean.rs

@@ -878,16 +878,13 @@ fn path_to_str(p: &ast::Path) -> ~str {
 
     let mut s = ~"";
     let mut first = true;
-    for i in p.segments.iter().map(|x| {
-            let string = token::get_ident(x.identifier.name);
-            string.get().to_str()
-    }) {
+    for i in p.segments.iter().map(|x| token::get_ident(x.identifier.name)) {
         if !first || p.global {
             s.push_str("::");
         } else {
             first = false;
         }
-        s.push_str(i);
+        s.push_str(i.get());
     }
     s
 }

src/libsyntax/parse/token.rs

@@ -554,13 +554,6 @@ pub struct InternedString {
     priv string: RcStr,
 }
 
-#[unsafe_destructor]
-impl Drop for InternedString {
-    fn drop(&mut self) {
-        // No-op just to make this not implicitly copyable.
-    }
-}
-
 impl InternedString {
     #[inline]
     pub fn new(string: &'static str) -> InternedString {