Remove Unique

It is only used for AlignedBox which can just directly implement Send
and Sync. Additionally make the Send and Sync impls more restrictive by
requiring the inner type to be Send cq Sync, previously AlignedBox was
always Send and Sync, which is not sound.

src/common/aligned_box.rs

@@ -1,13 +1,15 @@
 use core::alloc::{GlobalAlloc, Layout};
 
-use crate::{common::unique::Unique, memory::Enomem};
+use crate::memory::Enomem;
 
 // Necessary because GlobalAlloc::dealloc requires the layout to be the same, and therefore Box
 // cannot be used for increased alignment directly.
-// TODO: move to common?
 pub struct AlignedBox<T: ?Sized, const ALIGN: usize> {
-    inner: Unique<T>,
+    inner: *mut T,
 }
+unsafe impl<T: Send + ?Sized, const ALIGN: usize> Send for AlignedBox<T, ALIGN> {}
+unsafe impl<T: Sync + ?Sized, const ALIGN: usize> Sync for AlignedBox<T, ALIGN> {}
+
 pub unsafe trait ValidForZero {}
 unsafe impl<const N: usize> ValidForZero for [u8; N] {}
 unsafe impl ValidForZero for u8 {}
@@ -43,9 +45,7 @@ impl<T, const ALIGN: usize> AlignedBox<T, ALIGN> {
             if ptr.is_null() {
                 return Err(Enomem);
             }
-            Self {
-                inner: Unique::new_unchecked(ptr.cast()),
-            }
+            Self { inner: ptr.cast() }
         })
     }
 }
@@ -64,7 +64,7 @@ impl<T, const ALIGN: usize> AlignedBox<[T], ALIGN> {
                 return Err(Enomem);
             }
             Self {
-                inner: Unique::new_unchecked(core::ptr::slice_from_raw_parts_mut(ptr.cast(), len)),
+                inner: core::ptr::slice_from_raw_parts_mut(ptr.cast(), len),
             }
         })
     }
@@ -75,7 +75,7 @@ impl<T: ?Sized, const ALIGN: usize> core::fmt::Debug for AlignedBox<T, ALIGN> {
         write!(
             f,
             "[aligned box at {:p}, size {} alignment {}]",
-            self.inner.as_ptr(),
+            self.inner,
             self.layout().size(),
             self.layout().align()
         )
@@ -85,8 +85,8 @@ impl<T: ?Sized, const ALIGN: usize> Drop for AlignedBox<T, ALIGN> {
     fn drop(&mut self) {
         unsafe {
             let layout = self.layout();
-            core::ptr::drop_in_place(self.inner.as_ptr());
-            crate::ALLOCATOR.dealloc(self.inner.as_ptr().cast(), layout);
+            core::ptr::drop_in_place(self.inner);
+            crate::ALLOCATOR.dealloc(self.inner.cast(), layout);
         }
     }
 }
@@ -94,12 +94,12 @@ impl<T: ?Sized, const ALIGN: usize> core::ops::Deref for AlignedBox<T, ALIGN> {
     type Target = T;
 
     fn deref(&self) -> &Self::Target {
-        unsafe { &*self.inner.as_ptr() }
+        unsafe { &*self.inner }
     }
 }
 impl<T: ?Sized, const ALIGN: usize> core::ops::DerefMut for AlignedBox<T, ALIGN> {
     fn deref_mut(&mut self) -> &mut Self::Target {
-        unsafe { &mut *self.inner.as_ptr() }
+        unsafe { &mut *self.inner }
     }
 }
 impl<T: Clone + ValidForZero, const ALIGN: usize> Clone for AlignedBox<T, ALIGN> {

src/common/mod.rs

@@ -1,7 +1,6 @@
 pub mod aligned_box;
 #[macro_use]
 pub mod int_like;
-pub mod unique;
 
 /// Debug macro, lifted from the std
 #[macro_export]

src/common/unique.rs

@@ -1,30 +0,0 @@
-use core::{fmt, ptr::NonNull};
-
-/// A small wrapper around NonNull<T> that is Send + Sync, which is
-/// only correct if the pointer is never accessed from multiple
-/// locations across threads. Which is always, if the pointer is
-/// unique.
-pub struct Unique<T: ?Sized>(NonNull<T>);
-
-impl<T: ?Sized> Copy for Unique<T> {}
-impl<T: ?Sized> Clone for Unique<T> {
-    fn clone(&self) -> Self {
-        *self
-    }
-}
-unsafe impl<T: ?Sized> Send for Unique<T> {}
-unsafe impl<T: ?Sized> Sync for Unique<T> {}
-
-impl<T: ?Sized> Unique<T> {
-    pub unsafe fn new_unchecked(ptr: *mut T) -> Self {
-        Self(NonNull::new_unchecked(ptr))
-    }
-    pub fn as_ptr(self) -> *mut T {
-        self.0.as_ptr()
-    }
-}
-impl<T: ?Sized> fmt::Debug for Unique<T> {
-    fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
-        write!(f, "{:?}", self.0)
-    }
-}