Skip to content

Conversation

@galpeter
Copy link
Contributor

JerryScript-DCO-1.0-Signed-off-by: Peter Gal pgal.u-szeged@partner.samsung.com

@galpeter galpeter added the ecma builtins Related to ECMA built-in routines label May 28, 2015
@galpeter galpeter added this to the ECMA builtins milestone May 28, 2015
@galpeter galpeter force-pushed the object_keys_props branch from 40d358b to d77440b Compare May 28, 2015 15:58
@galpeter
Copy link
Contributor Author

@dbatyai, could you take a look?

Copy link
Member

Choose a reason for hiding this comment

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

We should probably add an assert here, to make sure new_array is normal.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added check.

@galpeter galpeter force-pushed the object_keys_props branch from d77440b to 4d0e6ed Compare May 28, 2015 16:21
@galpeter
Copy link
Contributor Author

I've updated the PR.

@ILyoan ILyoan mentioned this pull request May 29, 2015
9 tasks
@egavrin egavrin self-assigned this May 29, 2015
@egavrin
Copy link
Contributor

egavrin commented May 29, 2015

make push

JerryScript-DCO-1.0-Signed-off-by: Peter Gal pgal.u-szeged@partner.samsung.com
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.

3 participants