All implementations of incrementHolesCount are the same.

Author: woess

graal-js/src/com.oracle.truffle.js/src/com/oracle/truffle/js/runtime/array/dyn/AbstractWritableArray.java

@@ -199,8 +199,9 @@ protected final int prepareSupportedHoles(JSDynamicObject object, int index, Pro
     }
 
     @SuppressWarnings("unused")
-    protected void incrementHolesCount(JSDynamicObject object, int offset) {
-        throw Errors.shouldNotReachHere();
+    protected final void incrementHolesCount(JSDynamicObject object, int offset) {
+        assert isHolesType();
+        arraySetHoleCount(object, arrayGetHoleCount(object) + offset);
     }
 
     protected abstract void setHoleValue(JSDynamicObject object, int index);

graal-js/src/com.oracle.truffle.js/src/com/oracle/truffle/js/runtime/array/dyn/HolesDoubleArray.java

@@ -142,11 +142,6 @@ public AbstractDoubleArray toNonHoles(JSDynamicObject object, long index, Object
         return newArray;
     }
 
-    @Override
-    protected void incrementHolesCount(JSDynamicObject object, int offset) {
-        arraySetHoleCount(object, arrayGetHoleCount(object) + offset);
-    }
-
     @Override
     public boolean isSupported(JSDynamicObject object, long index) {
         return isSupportedHoles(object, index);

graal-js/src/com.oracle.truffle.js/src/com/oracle/truffle/js/runtime/array/dyn/HolesIntArray.java

@@ -160,11 +160,6 @@ public Object getInBoundsFast(JSDynamicObject object, int index) {
         return value;
     }
 
-    @Override
-    protected void incrementHolesCount(JSDynamicObject object, int offset) {
-        arraySetHoleCount(object, arrayGetHoleCount(object) + offset);
-    }
-
     @Override
     public HolesIntArray toHoles(JSDynamicObject object, long index, Object value) {
         return this;

graal-js/src/com.oracle.truffle.js/src/com/oracle/truffle/js/runtime/array/dyn/HolesJSObjectArray.java

@@ -153,11 +153,6 @@ public AbstractWritableArray toObject(JSDynamicObject object, long index, Object
         return newArray;
     }
 
-    @Override
-    protected void incrementHolesCount(JSDynamicObject object, int offset) {
-        arraySetHoleCount(object, arrayGetHoleCount(object) + offset);
-    }
-
     @Override
     public int prepareInBounds(JSDynamicObject object, int index, ProfileHolder profile) {
         return prepareInBoundsHoles(object, index, profile);

graal-js/src/com.oracle.truffle.js/src/com/oracle/truffle/js/runtime/array/dyn/HolesObjectArray.java

@@ -135,11 +135,6 @@ public AbstractObjectArray toNonHoles(JSDynamicObject object, long index, Object
         return newArray;
     }
 
-    @Override
-    protected void incrementHolesCount(JSDynamicObject object, int offset) {
-        arraySetHoleCount(object, arrayGetHoleCount(object) + offset);
-    }
-
     @Override
     public int prepareInBounds(JSDynamicObject object, int index, ProfileHolder profile) {
         return prepareInBoundsHoles(object, index, profile);