removed unsafe code in favor of explicit assert

This commit is contained in:
Marcel Hellwig 2018-11-13 08:33:44 +01:00
parent 0850f5ec36
commit 479d756c99

View file

@ -10,7 +10,6 @@ use std::hash::{Hasher, BuildHasherDefault};
#[cfg(test)] #[cfg(test)]
use std::mem; use std::mem;
use std::ops::{Index, IndexMut}; use std::ops::{Index, IndexMut};
use std::ptr;
use any::{Any, UncheckedAnyExt}; use any::{Any, UncheckedAnyExt};
@ -23,10 +22,11 @@ impl Hasher for TypeIdHasher {
#[inline] #[inline]
fn write(&mut self, bytes: &[u8]) { fn write(&mut self, bytes: &[u8]) {
// This expects to receive one and exactly one 64-bit value // This expects to receive one and exactly one 64-bit value
debug_assert!(bytes.len() == 8); assert!(bytes.len() == 8);
unsafe { self.value = u64::from(bytes[0]) | u64::from(bytes[1]) << 8 |
ptr::copy_nonoverlapping(&bytes[0] as *const u8 as *const u64, &mut self.value, 1) u64::from(bytes[2]) << 16 | u64::from(bytes[3]) << 24 |
} u64::from(bytes[4]) << 32 | u64::from(bytes[5]) << 40 |
u64::from(bytes[6]) << 48 | u64::from(bytes[7]) << 56;
} }
#[inline] #[inline]