.toSpliced should throw a TypeError instead of RangeError

tc39/proposal-change-array-by-copy#70
petamoriken · Jun 26, 2022 · c34d852 · c34d852
1 parent 7019893
commit c34d852
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/src/Float16Array.mjs b/src/Float16Array.mjs
@@ -17,6 +17,7 @@ import {
   CANNOT_MIX_BIGINT_AND_OTHER_TYPES,
   DERIVED_CONSTRUCTOR_CREATED_TYPEDARRAY_OBJECT_WHICH_WAS_TOO_SMALL_LENGTH,
   ITERATOR_PROPERTY_IS_NOT_CALLABLE,
+  MAXIMUM_ALLOWED_LENGTH_EXCEEDED,
   OFFSET_IS_OUT_OF_BOUNDS,
   REDUCE_OF_EMPTY_ARRAY_WITH_NO_INITIAL_VALUE,
   SPECIES_CONSTRUCTOR_DIDNT_RETURN_TYPEDARRAY_OBJECT,
@@ -29,6 +30,7 @@ import {
   ArrayPrototypePush,
   ArrayPrototypeSlice,
   ArrayPrototypeToLocaleString,
+  MAX_SAFE_INTEGER,
   NativeArrayBuffer,
   NativeObject,
   NativeProxy,
@@ -1067,6 +1069,9 @@ export class Float16Array {
 
     // don't use SpeciesConstructor
     const newLength = length + insertCount - actualDeleteCount;
+    if (newLength > MAX_SAFE_INTEGER) {
+      throw NativeTypeError(MAXIMUM_ALLOWED_LENGTH_EXCEEDED);
+    }
     const proxy = new Float16Array(newLength);
     const array = getFloat16BitsArray(proxy);
 

diff --git a/src/_util/messages.mjs b/src/_util/messages.mjs
@@ -17,4 +17,5 @@ export const CANNOT_MIX_BIGINT_AND_OTHER_TYPES =
 export const ITERATOR_PROPERTY_IS_NOT_CALLABLE = "@@iterator property is not callable";
 export const REDUCE_OF_EMPTY_ARRAY_WITH_NO_INITIAL_VALUE =
   "Reduce of empty array with no initial value";
+export const MAXIMUM_ALLOWED_LENGTH_EXCEEDED = "Maximum allowed length exceeded";
 export const OFFSET_IS_OUT_OF_BOUNDS = "Offset is out of bounds";