Noratrieb/datastructures
1
0
Fork 0
mirror of https://github.com/Noratrieb/datastructures.git synced 2026-01-14 17:35:02 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

insert_after on cursor

Browse source
This commit is contained in:
nora 2021-08-13 22:13:11 +02:00
parent 5c4deeadc0
commit 3c7ddc8340
2 changed files with 19 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

2
src/linked_list/mod.rs
View file

@ -399,7 +399,7 @@ impl<T> Node<T> {
fn allocate_nonnull<T>(element: T) -> NonNull<T> {
let boxed = Box::new(element);
// SAFETY: box is always non-null
unsafe { NonNull::new_unchecked(Box::leak(boxed)) }
unsafe { NonNull::new_unchecked(Box::into_raw(boxed)) }
}
/// The iterator over the linked list

22
src/packed_linked_list/mod.rs
View file

@ -12,7 +12,7 @@ use std::ptr::NonNull;
fn allocate_nonnull<T>(element: T) -> NonNull<T> {
// SAFETY: box is always non-null
unsafe { NonNull::new_unchecked(Box::leak(Box::new(element))) }
unsafe { NonNull::new_unchecked(Box::into_raw(Box::new(element))) }
}
///
@ -363,6 +363,7 @@ impl<T, const COUNT: usize> Node<T, COUNT> {
// copy all values up
if COUNT > 1 {
std::ptr::copy(
// todo miri error here
&self.values[index] as *const _,
&mut self.values[index + 1] as *mut _,
self.size - index,
@ -506,8 +507,21 @@ impl<'a, T, const COUNT: usize> CursorMut<'a, T, COUNT> {
// SAFETY: the node is not full and the index is not out of bounds
(false, false) => unsafe { current.insert(element, self.index + 1) },
(false, true) => {
// check whether the next node is full. if it is not full, insert it at the start
// if it is full or the next node doesn't exist, allocate a new node inbetween
// we need to copy some values to the next node, always allocate a new one to avoid needing to copy too many values
// nodes that are not very full will make insertions faster later, so we prefer them
unsafe {
let mut next = self.allocate_new_node_after();
let next = next.as_mut();
let to_copy = next.size - self.index;
std::ptr::copy_nonoverlapping(
&current.values[self.index] as *const _,
&mut next.values[0] as *mut _,
to_copy,
);
current.values[self.index] = MaybeUninit::new(element);
next.size = to_copy;
current.size = self.index + 1;
}
}
}
}
@ -656,7 +670,7 @@ mod iter {
node: list.first.map(|nn| unsafe { Box::from_raw(nn.as_ptr()) }),
index: 0,
};
// do not drop the list
// do not drop the list, the iterator has taken 'ownership'
mem::forget(list);
iter
}