robin/bitbuffer
1
0
Fork 0
mirror of https://codeberg.org/icewind/bitbuffer.git synced 2026-06-03 08:34:07 +02:00
Code Issues Releases Wiki Activity

dont save pointer to data

Browse source 
this doesn't actually make a performance difference
This commit is contained in:
Robin Appelman 2019-07-24 12:32:18 +02:00
commit 211de49a54
2 changed files with 2 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

1
.gitignore vendored
View file

@ -2,3 +2,4 @@
**/*.rs.bk **/*.rs.bk
Cargo.lock Cargo.lock
*.iml *.iml
*.bench

7
src/buffer.rs
View file

@ -42,7 +42,6 @@ where
bit_len: usize, bit_len: usize,
byte_len: usize, byte_len: usize,
endianness: PhantomData<E>, endianness: PhantomData<E>,
ptr: *const u8,
} }
impl<E> BitBuffer<E> impl<E> BitBuffer<E>
@ -65,7 +64,6 @@ where
pub fn new(bytes: Vec<u8>, _endianness: E) -> Self { pub fn new(bytes: Vec<u8>, _endianness: E) -> Self {
let byte_len = bytes.len(); let byte_len = bytes.len();
BitBuffer { BitBuffer {
ptr: bytes.as_ptr(),
bytes: Rc::new(bytes), bytes: Rc::new(bytes),
byte_len, byte_len,
bit_len: byte_len * 8, bit_len: byte_len * 8,
@ -102,7 +100,7 @@ where
let usize_bit_size = size_of::<usize>() * 8; let usize_bit_size = size_of::<usize>() * 8;
let raw_container: &usize = unsafe { let raw_container: &usize = unsafe {
// this is safe here because we already have checks that we don't read past the slice // this is safe here because we already have checks that we don't read past the slice
let ptr = self.ptr.add(byte_index); let ptr = self.bytes.as_ptr().add(byte_index);
transmute(ptr) transmute(ptr)
}; };
let container = if E::is_le() { let container = if E::is_le() {
@ -574,7 +572,6 @@ where
} }
Ok(BitBuffer { Ok(BitBuffer {
ptr: self.ptr,
bytes: Rc::clone(&self.bytes), bytes: Rc::clone(&self.bytes),
byte_len: bit_len / 8, byte_len: bit_len / 8,
bit_len, bit_len,
@ -587,7 +584,6 @@ impl<E: Endianness> From<Vec<u8>> for BitBuffer<E> {
fn from(bytes: Vec<u8>) -> Self { fn from(bytes: Vec<u8>) -> Self {
let byte_len = bytes.len(); let byte_len = bytes.len();
BitBuffer { BitBuffer {
ptr: bytes.as_ptr(),
bytes: Rc::new(bytes), bytes: Rc::new(bytes),
byte_len, byte_len,
bit_len: byte_len * 8, bit_len: byte_len * 8,
@ -599,7 +595,6 @@ impl<E: Endianness> From<Vec<u8>> for BitBuffer<E> {
impl<E: Endianness> Clone for BitBuffer<E> { impl<E: Endianness> Clone for BitBuffer<E> {
fn clone(&self) -> Self { fn clone(&self) -> Self {
BitBuffer { BitBuffer {
ptr: self.ptr,
bytes: Rc::clone(&self.bytes), bytes: Rc::clone(&self.bytes),
byte_len: self.byte_len, byte_len: self.byte_len,
bit_len: self.bit_len, bit_len: self.bit_len,