From 338b7cc1eff1959e518e95fd31ae4bb20f3121e5 Mon Sep 17 00:00:00 2001
From: Nilstrieb <48135649+Nilstrieb@users.noreply.github.com>
Date: Tue, 21 Dec 2021 13:22:51 +0100
Subject: [PATCH] SOLVED: more alignment fun

---
 src/lib.rs  | 3 ++-
 src/test.rs | 2 +-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/lib.rs b/src/lib.rs
index afbe2d3..b850698 100644
--- a/src/lib.rs
+++ b/src/lib.rs
@@ -117,7 +117,7 @@ impl<T: ?Sized> Vechonk<T> {
     pub fn push(&mut self, element: Box<T>) {
         let elem_size = mem::size_of_val(element.as_ref());
 
-        let elem_align = mem::align_of_val(&element);
+        let elem_align = mem::align_of_val(element.as_ref());
         let elem_ptr = Box::into_raw(element);
         let meta = ptr::metadata(elem_ptr);
 
@@ -155,6 +155,7 @@ impl<T: ?Sized> Vechonk<T> {
         // SAFETY: `elem_ptr` comes from `Box`, and is therefore valid to read from for the size
         //         We have made sure above that we have more than `elem_size` bytes free
         //         The two allocations cannot overlap, since the `Box` owned its contents, and so do we
+        //         `dest_ptr` has been aligned above
         unsafe {
             ptr::copy_nonoverlapping::<u8>(elem_ptr as _, dest_ptr, elem_size);
         }
diff --git a/src/test.rs b/src/test.rs
index 2490ac2..9448bb3 100644
--- a/src/test.rs
+++ b/src/test.rs
@@ -133,8 +133,8 @@ fn push_alignment() {
 
     let mut chonk = Vechonk::<dyn Any>::with_capacity(4096);
 
-    chonk.push(Box::new(0_u8));
     chonk.push(Box::new(BigAlign(5)));
+    chonk.push(Box::new(0_u8));
     chonk.push(Box::new(1_u64));
 
     let _ = chonk[0];