Cast fat raw pointers to thin raw pointers without going through refs (…

…#9)
maciejhirsz · Mar 14, 2020 · 64b7812 · 64b7812
1 parent a167927
commit 64b7812
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 7 deletions.
diff --git a/Cargo.toml b/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "beef"
-version = "0.2.0"
+version = "0.2.1"
 authors = ["Maciej Hirsz <hello@maciej.codes>"]
 edition = "2018"
 description = "More compact Cow"

diff --git a/src/lib.rs b/src/lib.rs
@@ -80,10 +80,12 @@ unsafe impl Beef for str {
     }
 
     #[inline]
-    unsafe fn owned_from_parts(mut ptr: NonNull<Self>, capacity: NonZeroUsize) -> String {
+    unsafe fn owned_from_parts(ptr: NonNull<Self>, capacity: NonZeroUsize) -> String {
+        let len = ptr.as_ref().len();
+
         String::from_utf8_unchecked(Vec::from_raw_parts(
-            ptr.as_mut().as_mut_ptr(),
-            ptr.as_mut().len(),
+            ptr.cast().as_ptr(),
+            len,
             capacity.get(),
         ))
     }
@@ -106,10 +108,12 @@ unsafe impl<T: Clone> Beef for [T] {
     }
 
     #[inline]
-    unsafe fn owned_from_parts(mut ptr: NonNull<Self>, capacity: NonZeroUsize) -> Vec<T> {
+    unsafe fn owned_from_parts(ptr: NonNull<Self>, capacity: NonZeroUsize) -> Vec<T> {
+        let len = ptr.as_ref().len();
+
         Vec::from_raw_parts(
-            ptr.as_mut().as_mut_ptr(),
-            ptr.as_mut().len(),
+            ptr.cast().as_ptr(),
+            len,
             capacity.get(),
         )
     }