Skip to content

Conversation

dbatyai
Copy link
Member

@dbatyai dbatyai commented May 21, 2015

JerryScript-DCO-1.0-Signed-off-by: Dániel Bátyai [email protected]

@ILyoan ILyoan mentioned this pull request May 21, 2015
25 tasks
@galpeter galpeter added this to the ECMA builtins milestone May 21, 2015
@dbatyai dbatyai added the ecma builtins Related to ECMA built-in routines label May 22, 2015
@galpeter
Copy link
Contributor

lgtm

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is part of step 3, isn't it?

@ruben-ayrapetyan ruben-ayrapetyan self-assigned this May 26, 2015
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please, change the assertion to ecma_is_completion_value_normal_true (put_comp_value).

@dbatyai
Copy link
Member Author

dbatyai commented May 28, 2015

I've updated the patch.

@egavrin
Copy link
Contributor

egavrin commented May 28, 2015

/jerry-core/ecma/builtin-objects/ecma-builtin-array-prototype.cpp: In function ‘ecma_completion_value_t ecma_builtin_array_prototype_object_filter(ecma_value_t, ecma_value_t, ecma_value_t)’:
/jerry-core/ecma/builtin-objects/ecma-builtin-array-prototype.cpp:794:49: error: ‘to_object_comp’ was not declared in this scope
     func_object_p = ecma_get_object_from_value (to_object_comp);
                                                 ^
compilation terminated due to -Wfatal-errors.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the to_object_comp? Why should it be freed?

@dbatyai
Copy link
Member Author

dbatyai commented May 28, 2015

@egavrin, @ruben-ayrapetyan sorry, fixed

JerryScript-DCO-1.0-Signed-off-by: Dániel Bátyai [email protected]
@ruben-ayrapetyan
Copy link
Contributor

make push

@egavrin
Copy link
Contributor

egavrin commented May 28, 2015

@dbatyai @ruben-ayrapetyan Nope ^_^, wrong email

+ make precommit -j4
�[1;33m Signed-off-by message is incorrect. The following line should be at the end of the ed47d2279a9758ee3e82f7dac87f3dccdd9dc566 commit's message: 'JerryScript-DCO-1.0-Signed-off-by: Dániel Bátyai [email protected]'.

@ruben-ayrapetyan
Copy link
Contributor

@egavrin , ed47d2279a9758ee3e82f7dac87f3dccdd9dc566 seems to be another commit.
The commit in the pull request is a80282a.

In other, already merged, commits the [email protected] e-mail is used.

@egavrin
Copy link
Contributor

egavrin commented May 28, 2015

@ruben-ayrapetyan ah, my bad.
make push

@dbatyai
Copy link
Member Author

dbatyai commented May 28, 2015

merged: 3329583

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ecma builtins Related to ECMA built-in routines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants