Fix MEMMOVE in rb_darray_insert

peterzhu2118 · peterzhu2118 · commit 07e89bde4693 · 2024-12-24T09:40:12.000-05:00
Ruby's MEMMOVE takes in the element data type for the third argument, not
the size of the element. What this did was do sizeof(sizeof( ... )) which
always returned sizeof(size_t).
diff --git a/darray.h b/darray.h
@@ -58,7 +58,7 @@
     MEMMOVE( \
         rb_darray_ref(*(ptr_to_ary), idx + 1), \
         rb_darray_ref(*(ptr_to_ary), idx), \
-        sizeof((*(ptr_to_ary))->data[0]), \
+        (*(ptr_to_ary))->data[0], \
         rb_darray_size(*(ptr_to_ary)) - idx); \
     rb_darray_set(*(ptr_to_ary), idx, element); \
     (*(ptr_to_ary))->meta.size++; \
