Fix up Map definition

There's no need to define the key type as &'static str when &'a str will
do and be more flexible

phf.rs

@@ -15,21 +15,17 @@ impl<T> Container for PhfMap<T> {
     }
 }
 
-impl<T> Map<&'static str, T> for PhfMap<T> {
+impl<'a, T> Map<&'a str, T> for PhfMap<T> {
     #[inline]
-    fn find<'a>(&'a self, key: & &'static str) -> Option<&'a T> {
-        self.find_str(key)
-    }
-}
-
-impl<T> PhfMap<T> {
-    #[inline]
-    pub fn find_str<'a, S: Str>(&'a self, key: &S) -> Option<&'a T> {
-        self.entries.bsearch(|&(val, _)| val.cmp(&key.as_slice())).map(|idx| {
+    fn find<'a>(&'a self, key: & &str) -> Option<&'a T> {
+        self.entries.bsearch(|&(val, _)| val.cmp(key)).map(|idx| {
             let (_, ref val) = self.entries[idx];
             val
         })
     }
+}
+
+impl<T> PhfMap<T> {
 
     #[inline]
     pub fn entries<'a>(&'a self) -> PhfMapEntries<'a, T> {

phf_mac.rs

@@ -19,6 +19,7 @@ use syntax::parse::token;
 use syntax::parse::token::{COMMA, EOF, FAT_ARROW};
 
 #[macro_registrar]
+#[doc(hidden)]
 pub fn macro_registrar(register: |Name, SyntaxExtension|) {
     register(token::intern("phf_map"),
              NormalTT(~SyntaxExpanderTT {

test.rs

@@ -80,14 +80,3 @@ fn test_values() {
     assert!(hash.contains(&11));
     assert_eq!(2, hash.len());
 }
-
-#[test]
-fn test_find_str() {
-    static map: PhfMap<int> = phf_map!(
-        "foo" => 10,
-        "bar" => 11,
-    );
-    assert!(map.find_str(&~"foo") == Some(&10));
-    assert!(map.find_str(&~"bar") == Some(&11));
-    assert!(map.find_str(&~"baz") == None);
-}