Skip to content

Conversation

@kkristof
Copy link
Contributor

JerryScript-DCO-1.0-Signed-off-by: Kristof Kosztyo kkosztyo.u-szeged@partner.samsung.com

@kkristof kkristof added ecma builtins Related to ECMA built-in routines development Feature implementation labels Jun 18, 2015
@kkristof kkristof added this to the ECMA builtins milestone Jun 18, 2015
@ILyoan ILyoan mentioned this pull request Jun 18, 2015
9 tasks
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this intentional?

@kkristof
Copy link
Contributor Author

Hi, I have updated the patch.

Copy link
Contributor

Choose a reason for hiding this comment

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

Please align the * to the function name.

@galpeter
Copy link
Contributor

aside from the minor style comments it looks good to me.

Choose a reason for hiding this comment

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

Could you, please, move the comment about [[Class]] below to ecma_op_create_object_object_noarg_and_set_prototype?

JerryScript-DCO-1.0-Signed-off-by: Kristof Kosztyo kkosztyo.u-szeged@partner.samsung.com
@kkristof
Copy link
Contributor Author

Hi, I have updated the patch.

@ruben-ayrapetyan
Copy link

Looks good to me.

@egavrin
Copy link
Contributor

egavrin commented Jun 22, 2015

make push

@galpeter
Copy link
Contributor

Rebased & merged: 61ab205

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

Labels

development Feature implementation ecma builtins Related to ECMA built-in routines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants