YourWishes/jerryscript
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Remove 'useless' variable from function.bind (#2327)

Browse Source 
The `args_length` variable in the `ecma_builtin_function_prototype_object_bind`
method is not needed, the `arguments_number` variable can be
directly used.

JerryScript-DCO-1.0-Signed-off-by: Peter Gal pgal.u-szeged@partner.samsung.com
This commit is contained in:
Péter Gál
2018-05-15 02:03:40 +02:00
committed by yichoi
parent 86111acbc4
commit 38f0dbf065
1 changed files with 2 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
jerry-core/ecma/builtin-objects/ecma-builtin-function-prototype.c
+2 -5
View File
@@ -250,15 +250,12 @@ ecma_builtin_function_prototype_object_bind (ecma_value_t this_arg, /**< this ar
/* 4. 11. 18. */ /* 4. 11. 18. */
ecma_object_t *prototype_obj_p = ecma_builtin_get (ECMA_BUILTIN_ID_FUNCTION_PROTOTYPE); ecma_object_t *prototype_obj_p = ecma_builtin_get (ECMA_BUILTIN_ID_FUNCTION_PROTOTYPE);
ecma_length_t args_length = arguments_number;
ecma_object_t *function_p; ecma_object_t *function_p;
ecma_extended_object_t *ext_function_p; ecma_extended_object_t *ext_function_p;
if (arguments_number == 0 if (arguments_number == 0
|| (arguments_number == 1 && !ecma_is_value_integer_number (arguments_list_p[0]))) || (arguments_number == 1 && !ecma_is_value_integer_number (arguments_list_p[0])))
{ {
args_length = 1;
function_p = ecma_create_object (prototype_obj_p, function_p = ecma_create_object (prototype_obj_p,
sizeof (ecma_extended_object_t), sizeof (ecma_extended_object_t),
ECMA_OBJECT_TYPE_BOUND_FUNCTION); ECMA_OBJECT_TYPE_BOUND_FUNCTION);
@@ -280,7 +277,7 @@ ecma_builtin_function_prototype_object_bind (ecma_value_t this_arg, /**< this ar
{ {
JERRY_ASSERT (arguments_number > 0); JERRY_ASSERT (arguments_number > 0);
size_t obj_size = sizeof (ecma_extended_object_t) + (args_length * sizeof (ecma_value_t)); size_t obj_size = sizeof (ecma_extended_object_t) + (arguments_number * sizeof (ecma_value_t));
function_p = ecma_create_object (prototype_obj_p, function_p = ecma_create_object (prototype_obj_p,
obj_size, obj_size,
@@ -302,7 +299,7 @@ ecma_builtin_function_prototype_object_bind (ecma_value_t this_arg, /**< this ar
*args_p++ = ecma_copy_value_if_not_object (arguments_list_p[i]); *args_p++ = ecma_copy_value_if_not_object (arguments_list_p[i]);
} }
ecma_value_t args_len_or_this = ecma_make_integer_value ((ecma_integer_value_t) args_length); ecma_value_t args_len_or_this = ecma_make_integer_value ((ecma_integer_value_t) arguments_number);
ext_function_p->u.bound_function.args_len_or_this = args_len_or_this; ext_function_p->u.bound_function.args_len_or_this = args_len_or_this;
} }