Skip to content

toNumber operation should handle binary and octal literals#3471

Merged
rerobika merged 1 commit intojerryscript-project:masterfrom
orkvi:number1
Jan 10, 2020
Merged

toNumber operation should handle binary and octal literals#3471
rerobika merged 1 commit intojerryscript-project:masterfrom
orkvi:number1

Conversation

@orkvi
Copy link
Copy Markdown
Contributor

@orkvi orkvi commented Dec 20, 2019

I changed the hexadecimal converter (string to number) function to allow conversion into other bases.

JerryScript-DCO-1.0-Signed-off-by: Virag Orkenyi orkvi@inf.u-szeged.hu

@orkvi orkvi force-pushed the number1 branch 2 times, most recently from c3d8806 to 0416eb6 Compare December 20, 2019 07:25
Comment thread jerry-core/ecma/base/ecma-helpers-conversion.c Outdated
Comment thread jerry-core/ecma/base/ecma-helpers-conversion.c Outdated
Comment thread jerry-core/ecma/base/ecma-helpers-conversion.c Outdated
Comment thread jerry-core/ecma/base/ecma-helpers-conversion.c Outdated
Comment thread jerry-core/ecma/base/ecma-helpers-conversion.c Outdated
Comment thread jerry-core/ecma/base/ecma-helpers-conversion.c Outdated
Comment thread jerry-core/ecma/base/ecma-helpers-conversion.c
@rerobika rerobika added the ES2015 Related to ES2015 features label Dec 20, 2019
@rerobika rerobika changed the title String to Number toNumber operation should handle binary and octal literals Jan 6, 2020
JerryScript-DCO-1.0-Signed-off-by: Virag Orkenyi orkvi@inf.u-szeged.hu
@rerobika rerobika requested review from galpeter and zherczeg January 7, 2020 13:26
@rerobika rerobika dismissed their stale review January 7, 2020 13:26

I dismiss the review, since I've become a co-author.

Copy link
Copy Markdown
Contributor

@galpeter galpeter left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Copy Markdown
Contributor

@robertsipka robertsipka left a comment

Choose a reason for hiding this comment

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

lgtm

@rerobika rerobika merged commit b6c2e6e into jerryscript-project:master Jan 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ES2015 Related to ES2015 features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants