Avoid nil-ing out VMArrays twice

It’s done in VMObject already Signed-off-by: Stefan Marr <[email protected]>
smarr · Aug 12, 2024 · 1e17f76 · 1e17f76
1 parent 76da6a0
commit 1e17f76
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 9 deletions.
diff --git a/src/vmobjects/VMArray.cpp b/src/vmobjects/VMArray.cpp
@@ -40,14 +40,6 @@
 
 const size_t VMArray::VMArrayNumberOfFields = 0;
 
-VMArray::VMArray(size_t arraySize, size_t additionalBytes)
-    : VMObject(
-          arraySize + 0 /* VMArray is not allowed to have any fields itself */,
-          additionalBytes + sizeof(VMArray)) {
-    assert(VMArrayNumberOfFields == 0);
-    nilInitializeFields();
-}
-
 VMArray* VMArray::Copy() const {
     VMArray* copy = Universe::NewArray(GetNumberOfIndexableFields());
 

diff --git a/src/vmobjects/VMArray.h b/src/vmobjects/VMArray.h
@@ -39,7 +39,12 @@ class VMArray : public VMObject {
 public:
     typedef GCArray Stored;
 
-    explicit VMArray(size_t arraySize, size_t additionalBytes);
+    explicit VMArray(size_t arraySize, size_t additionalBytes)
+        : VMObject(arraySize +
+                       0 /* VMArray is not allowed to have any fields itself */,
+                   additionalBytes + sizeof(VMArray)) {
+        assert(VMArrayNumberOfFields == 0);
+    }
 
     // VMArray doesn't need to customize `void WalkObjects(walk_heap_fn)`,
     // because it doesn't need anything special.