Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement Array.prototype.join #38

Merged
merged 1 commit into from
May 21, 2015

Conversation

galpeter
Copy link
Contributor

JerryScript-DCO-1.0-Signed-off-by: Peter Gal [email protected]

@ILyoan ILyoan mentioned this pull request May 12, 2015
25 tasks
@galpeter galpeter force-pushed the array_prototype_join branch 2 times, most recently from e620036 to c63c4f7 Compare May 13, 2015 12:33
@egavrin
Copy link
Contributor

egavrin commented May 14, 2015

Could you, please, rework this patch corresponding to implementation style of other builtins, like #22.

@galpeter galpeter added this to the ECMA builtins milestone May 20, 2015
@galpeter galpeter force-pushed the array_prototype_join branch 2 times, most recently from cc21cc3 to 0bfd6a9 Compare May 20, 2015 16:30
@galpeter
Copy link
Contributor Author

@egavrin , I've updated the pull request.

@galpeter
Copy link
Contributor Author

@dbatyai , could you also check the code?

@galpeter galpeter force-pushed the array_prototype_join branch from 0bfd6a9 to 5a08e40 Compare May 21, 2015 07:28
@egavrin
Copy link
Contributor

egavrin commented May 21, 2015

OK for me, make push

@dbatyai
Copy link
Member

dbatyai commented May 21, 2015

@galpeter, LGTM

JerryScript-DCO-1.0-Signed-off-by: Peter Gal [email protected]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants