This reverts commit
479d756c992af132c6ba45248c65dc230c5986cf.
There’s nothing wrong with this patch, but I had never pulled this
commit to my local repository and had completely forgotten about it, and
today removed the unsafe code in a *different* direction that I like
better (`bytes.try_into().map(|bytes| u64::from_ne_bytes(bytes))`), so
reverting it so I can cleanly rebase is just easier for me!
#[cfg(test)]
use std::mem;
use std::ops::{Index, IndexMut};
#[cfg(test)]
use std::mem;
use std::ops::{Index, IndexMut};
use any::{Any, UncheckedAnyExt};
use any::{Any, UncheckedAnyExt};
#[inline]
fn write(&mut self, bytes: &[u8]) {
// This expects to receive one and exactly one 64-bit value
#[inline]
fn write(&mut self, bytes: &[u8]) {
// This expects to receive one and exactly one 64-bit value
- assert!(bytes.len() == 8);
- self.value = u64::from(bytes[0]) | u64::from(bytes[1]) << 8 |
- 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;
+ debug_assert!(bytes.len() == 8);
+ unsafe {
+ ptr::copy_nonoverlapping(&bytes[0] as *const u8 as *const u64, &mut self.value, 1)
+ }