Skip to content

Conversation

bzsolt
Copy link
Member

@bzsolt bzsolt commented Jun 4, 2015

JerryScript-DCO-1.0-Signed-off-by: Zsolt Borbély [email protected]

Copy link
Contributor

Choose a reason for hiding this comment

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

Could you, please, use ret_value like in the implementation of getOwnPropertyNames routine below?

Copy link
Member Author

Choose a reason for hiding this comment

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

Of course, I've updated it.

Copy link
Contributor

Choose a reason for hiding this comment

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

Thank you.

@egavrin egavrin added this to the ECMA builtins milestone Jun 4, 2015
@egavrin egavrin added the ecma builtins Related to ECMA built-in routines label Jun 4, 2015
@ruben-ayrapetyan
Copy link
Contributor

Looks good to me

Copy link
Contributor

Choose a reason for hiding this comment

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

This test does not call the getPrototypeOf at all, the reason for the exception would be that the x is not defined/assigned a value.

Copy link
Member Author

Choose a reason for hiding this comment

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

Hm, you're right, I will remove it. Thanks.

JerryScript-DCO-1.0-Signed-off-by: Zsolt Borbély [email protected]
@egavrin
Copy link
Contributor

egavrin commented Jun 5, 2015

Good to me. make push

@galpeter
Copy link
Contributor

galpeter commented Jun 8, 2015

Rebased & merged: cefeea0

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.

4 participants