Skip to content

Commit 2e0334c

Browse files
committed
Returned array of Array.prototype.map() should have same length as original.
JerryScript-DCO-1.0-Signed-off-by: Dániel Bátyai dbatyai.u-szeged@partner.samsung.com
1 parent fc45b80 commit 2e0334c

File tree

2 files changed

+8
-0
lines changed

2 files changed

+8
-0
lines changed

jerry-core/ecma/builtin-objects/ecma-builtin-array-prototype.cpp

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2384,7 +2384,11 @@ ecma_builtin_array_prototype_object_map (ecma_value_t this_arg, /**< this argume
23842384

23852385
if (ecma_is_completion_value_empty (ret_value))
23862386
{
2387+
ECMA_TRY_CATCH (set_length_value,
2388+
ecma_builtin_array_prototype_helper_set_length (new_array_p, ecma_uint32_to_number (len)),
2389+
ret_value);
23872390
ret_value = new_array;
2391+
ECMA_FINALIZE (set_length_value);
23882392
}
23892393
else
23902394
{

tests/jerry/array-prototype-map.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,10 @@ assert ([1, 4, 9].map(func).equals([1,5,11]));
5252

5353
assert ([1, "X", 10].map(func).equals([1, "X1", 12]));
5454

55+
var arr = [1,2,3];
56+
arr.length = 5;
57+
assert(arr.map(func).length === arr.length);
58+
5559
var long_array = [0, 1];
5660
assert (long_array.map(func).equals([0,2]));
5761

0 commit comments

Comments
 (0)