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

implements toLowerCase, toUpperCase, substring, substr and valueOf St… #91

Merged
merged 1 commit into from
Sep 25, 2019

Conversation

arbroween
Copy link
Contributor

…ring prototype methods

Hey there !
I implemented toLowerCase, toUpperCase, substring, substr and valueOf, to get more familiar with the code base as suggested on the project Discord. They are part of #13.

This is my first PR and also I am not an expert in Rust, but I tried to implement the algorithms in the tc39 specification as much as I could and I also followed the style of the already implemented String prototype methods.

I don't know if the implementation semantics are exactly the ones expected, but they should be close.

Any feedback is very welcomed!
Tell me if there is something I can improve

@jasonwilliams
Copy link
Member

Hey @arbroween looks good, are you able to rebase?

@jasonwilliams
Copy link
Member

ping @arbroween

@jasonwilliams jasonwilliams force-pushed the string-prototype-methods branch from ab47eed to 6950ea4 Compare September 25, 2019 14:45
@jasonwilliams jasonwilliams merged commit 83616ef into boa-dev:master Sep 25, 2019
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.

2 participants