Refactor to avoid a spurious compatibility warning
[anymap] / src / raw.rs
index 7d537832e424fef9862570295188644663260743..aa3d6dc7dd83b1d3e159339c5abd5c2abdee86d2 100644 (file)
@@ -7,6 +7,7 @@ use std::borrow::Borrow;
 use std::collections::hash_map::{self, HashMap};
 use std::hash::Hash;
 use std::hash::{Hasher, BuildHasherDefault};
+#[cfg(test)]
 use std::mem;
 use std::ops::{Index, IndexMut};
 use std::ptr;
@@ -24,7 +25,7 @@ impl Hasher for TypeIdHasher {
         // This expects to receive one and exactly one 64-bit value
         debug_assert!(bytes.len() == 8);
         unsafe {
-            ptr::copy_nonoverlapping(mem::transmute(&bytes[0]), &mut self.value, 1)
+            ptr::copy_nonoverlapping(&bytes[0] as *const u8 as *const u64, &mut self.value, 1)
         }
     }
 
@@ -55,7 +56,7 @@ fn type_id_hasher() {
 /// contents of an `Map`. However, because you will then be dealing with `Any` trait objects, it
 /// doesn’t tend to be so very useful. Still, if you need it, it’s here.
 #[derive(Debug)]
-pub struct RawMap<A: ?Sized + UncheckedAnyExt = Any> {
+pub struct RawMap<A: ?Sized + UncheckedAnyExt = dyn Any> {
     inner: HashMap<TypeId, Box<A>, BuildHasherDefault<TypeIdHasher>>,
 }
 
@@ -69,13 +70,6 @@ impl<A: ?Sized + UncheckedAnyExt> Clone for RawMap<A> where Box<A>: Clone {
     }
 }
 
-impl<A: ?Sized + UncheckedAnyExt> Default for RawMap<A> {
-    #[inline]
-    fn default() -> RawMap<A> {
-        RawMap::new()
-    }
-}
-
 impl_common_methods! {
     field: RawMap.inner;
     new() => HashMap::with_hasher(Default::default());