alloc: fix deprecated warnings

This commit is contained in:
Jorge Aparicio 2018-08-19 21:56:49 +02:00
parent 96572cb5f8
commit 7c37c6d33e
2 changed files with 21 additions and 20 deletions

View File

@ -42,7 +42,7 @@
// This implies that even an empty internal node has at least one edge. // This implies that even an empty internal node has at least one edge.
use core::marker::PhantomData; use core::marker::PhantomData;
use core::mem; use core::mem::{self, MaybeUninit};
use core::ptr::{self, Unique, NonNull}; use core::ptr::{self, Unique, NonNull};
use core::slice; use core::slice;
@ -73,7 +73,7 @@ struct LeafNode<K, V> {
/// This node's index into the parent node's `edges` array. /// This node's index into the parent node's `edges` array.
/// `*node.parent.edges[node.parent_idx]` should be the same thing as `node`. /// `*node.parent.edges[node.parent_idx]` should be the same thing as `node`.
/// This is only guaranteed to be initialized when `parent` is nonnull. /// This is only guaranteed to be initialized when `parent` is nonnull.
parent_idx: u16, parent_idx: MaybeUninit<u16>,
/// The number of keys and values this node stores. /// The number of keys and values this node stores.
/// ///
@ -83,8 +83,8 @@ struct LeafNode<K, V> {
/// The arrays storing the actual data of the node. Only the first `len` elements of each /// The arrays storing the actual data of the node. Only the first `len` elements of each
/// array are initialized and valid. /// array are initialized and valid.
keys: [K; CAPACITY], keys: MaybeUninit<[K; CAPACITY]>,
vals: [V; CAPACITY], vals: MaybeUninit<[V; CAPACITY]>,
} }
impl<K, V> LeafNode<K, V> { impl<K, V> LeafNode<K, V> {
@ -94,10 +94,10 @@ impl<K, V> LeafNode<K, V> {
LeafNode { LeafNode {
// As a general policy, we leave fields uninitialized if they can be, as this should // As a general policy, we leave fields uninitialized if they can be, as this should
// be both slightly faster and easier to track in Valgrind. // be both slightly faster and easier to track in Valgrind.
keys: mem::uninitialized(), keys: MaybeUninit::uninitialized(),
vals: mem::uninitialized(), vals: MaybeUninit::uninitialized(),
parent: ptr::null(), parent: ptr::null(),
parent_idx: mem::uninitialized(), parent_idx: MaybeUninit::uninitialized(),
len: 0 len: 0
} }
} }
@ -115,10 +115,10 @@ unsafe impl Sync for LeafNode<(), ()> {}
// ever take a pointer past the first key. // ever take a pointer past the first key.
static EMPTY_ROOT_NODE: LeafNode<(), ()> = LeafNode { static EMPTY_ROOT_NODE: LeafNode<(), ()> = LeafNode {
parent: ptr::null(), parent: ptr::null(),
parent_idx: 0, parent_idx: MaybeUninit::uninitialized(),
len: 0, len: 0,
keys: [(); CAPACITY], keys: MaybeUninit::uninitialized(),
vals: [(); CAPACITY], vals: MaybeUninit::uninitialized(),
}; };
/// The underlying representation of internal nodes. As with `LeafNode`s, these should be hidden /// The underlying representation of internal nodes. As with `LeafNode`s, these should be hidden
@ -430,7 +430,7 @@ impl<BorrowType, K, V, Type> NodeRef<BorrowType, K, V, Type> {
root: self.root, root: self.root,
_marker: PhantomData _marker: PhantomData
}, },
idx: self.as_leaf().parent_idx as usize, idx: unsafe { usize::from(*self.as_leaf().parent_idx.get_ref()) },
_marker: PhantomData _marker: PhantomData
}) })
} else { } else {
@ -567,7 +567,7 @@ impl<'a, K: 'a, V: 'a, Type> NodeRef<marker::Immut<'a>, K, V, Type> {
// the node, which is allowed by LLVM. // the node, which is allowed by LLVM.
unsafe { unsafe {
slice::from_raw_parts( slice::from_raw_parts(
self.as_leaf().keys.as_ptr(), self.as_leaf().keys.get_ref().as_ptr(),
self.len() self.len()
) )
} }
@ -578,7 +578,7 @@ impl<'a, K: 'a, V: 'a, Type> NodeRef<marker::Immut<'a>, K, V, Type> {
debug_assert!(!self.is_shared_root()); debug_assert!(!self.is_shared_root());
unsafe { unsafe {
slice::from_raw_parts( slice::from_raw_parts(
self.as_leaf().vals.as_ptr(), self.as_leaf().vals.get_ref().as_ptr(),
self.len() self.len()
) )
} }
@ -605,7 +605,7 @@ impl<'a, K: 'a, V: 'a, Type> NodeRef<marker::Mut<'a>, K, V, Type> {
} else { } else {
unsafe { unsafe {
slice::from_raw_parts_mut( slice::from_raw_parts_mut(
&mut self.as_leaf_mut().keys as *mut [K] as *mut K, self.as_leaf_mut().keys.get_mut() as *mut [K] as *mut K,
self.len() self.len()
) )
} }
@ -616,7 +616,7 @@ impl<'a, K: 'a, V: 'a, Type> NodeRef<marker::Mut<'a>, K, V, Type> {
debug_assert!(!self.is_shared_root()); debug_assert!(!self.is_shared_root());
unsafe { unsafe {
slice::from_raw_parts_mut( slice::from_raw_parts_mut(
&mut self.as_leaf_mut().vals as *mut [V] as *mut V, self.as_leaf_mut().vals.get_mut() as *mut [V] as *mut V,
self.len() self.len()
) )
} }
@ -1013,7 +1013,7 @@ impl<'a, K, V> Handle<NodeRef<marker::Mut<'a>, K, V, marker::Internal>, marker::
let ptr = self.node.as_internal_mut() as *mut _; let ptr = self.node.as_internal_mut() as *mut _;
let mut child = self.descend(); let mut child = self.descend();
child.as_leaf_mut().parent = ptr; child.as_leaf_mut().parent = ptr;
child.as_leaf_mut().parent_idx = idx; child.as_leaf_mut().parent_idx.set(idx);
} }
/// Unsafely asserts to the compiler some static information about whether the underlying /// Unsafely asserts to the compiler some static information about whether the underlying
@ -1152,12 +1152,12 @@ impl<'a, K, V> Handle<NodeRef<marker::Mut<'a>, K, V, marker::Leaf>, marker::KV>
ptr::copy_nonoverlapping( ptr::copy_nonoverlapping(
self.node.keys().as_ptr().add(self.idx + 1), self.node.keys().as_ptr().add(self.idx + 1),
new_node.keys.as_mut_ptr(), new_node.keys.get_mut().as_mut_ptr(),
new_len new_len
); );
ptr::copy_nonoverlapping( ptr::copy_nonoverlapping(
self.node.vals().as_ptr().add(self.idx + 1), self.node.vals().as_ptr().add(self.idx + 1),
new_node.vals.as_mut_ptr(), new_node.vals.get_mut().as_mut_ptr(),
new_len new_len
); );
@ -1210,12 +1210,12 @@ impl<'a, K, V> Handle<NodeRef<marker::Mut<'a>, K, V, marker::Internal>, marker::
ptr::copy_nonoverlapping( ptr::copy_nonoverlapping(
self.node.keys().as_ptr().add(self.idx + 1), self.node.keys().as_ptr().add(self.idx + 1),
new_node.data.keys.as_mut_ptr(), new_node.data.keys.get_mut().as_mut_ptr(),
new_len new_len
); );
ptr::copy_nonoverlapping( ptr::copy_nonoverlapping(
self.node.vals().as_ptr().add(self.idx + 1), self.node.vals().as_ptr().add(self.idx + 1),
new_node.data.vals.as_mut_ptr(), new_node.data.vals.get_mut().as_mut_ptr(),
new_len new_len
); );
ptr::copy_nonoverlapping( ptr::copy_nonoverlapping(

View File

@ -119,6 +119,7 @@
#![feature(exact_chunks)] #![feature(exact_chunks)]
#![feature(rustc_const_unstable)] #![feature(rustc_const_unstable)]
#![feature(const_vec_new)] #![feature(const_vec_new)]
#![feature(maybe_uninit)]
// Allow testing this library // Allow testing this library