-
-
Notifications
You must be signed in to change notification settings - Fork 670
Reimplement strtod and parseFloat / strtod + Fix String.fromCodePoint + improve strtol #768
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
Merged
Merged
Changes from 66 commits
Commits
Show all changes
74 commits
Select commit
Hold shift + click to select a range
4b3c01a
init
MaxGraey 0cd4c72
wip
MaxGraey 80992cf
wip
MaxGraey 315c9a7
wip
MaxGraey e2acae8
Merge branch 'master' into improve-strtod
MaxGraey 6830551
fix String.fromCodePoint
MaxGraey a88033d
add some tests
MaxGraey 4c66030
add assert wich discard neg units for String.fromCahrCode
MaxGraey f24e88c
Merge branch 'master' into improve-strtod
MaxGraey df8ee28
update build
MaxGraey 3a5ace2
update (wip)
MaxGraey 9324691
update (wip)
MaxGraey 374dc5b
update (wip)
MaxGraey 5f992e1
update (wip)
MaxGraey 0ebe332
update (wip)
MaxGraey af4170d
update (wip)
MaxGraey f270c55
fix
MaxGraey 53787a9
fix
MaxGraey 8fc79e3
fixes
MaxGraey e2858d2
update (wip)
MaxGraey bf15b9b
update tests (wip)
MaxGraey 920d703
update (wip)
MaxGraey 0d4944c
update (wip)
MaxGraey ba13bf5
fixes
MaxGraey 24dba43
more tests (wip)
MaxGraey 08fe09d
fix sign
MaxGraey 4fd2f4f
cleanup
MaxGraey 0d09667
improve stdtol
MaxGraey 7ba7ea6
more tests
MaxGraey a3423ce
more tests (wip)
MaxGraey 14a349c
add more tests
MaxGraey 0036547
fix one edge case. Add rint
MaxGraey d21aa12
rint -> nearest
MaxGraey 6771a0f
update (wip)
MaxGraey 9a9addd
finish implementation
MaxGraey 00a4f78
clenaup (wip)
MaxGraey 6f0133e
add comments and hints
MaxGraey c2d5577
simplify fixmul
MaxGraey f933857
more tests
MaxGraey a21dfa0
more tests
MaxGraey 838aaed
fix
MaxGraey 1b8d239
minor optimize for fast path
MaxGraey 4d46e54
more tests
MaxGraey 04b6ef0
simplify test
MaxGraey 94818ab
more tests
MaxGraey 3d3b04f
Fix rest parsing issues
MaxGraey bddd863
more tests
MaxGraey 3c6d546
minor improvment
MaxGraey dc9027b
more tests
MaxGraey b63d877
improve fast path
MaxGraey 407f0ad
improve isSpace
MaxGraey 901a867
finalise isSpace
MaxGraey 1b63560
Merge branch 'master' into improve-strtod
MaxGraey 47772d6
improve isSpace more
MaxGraey 3567be3
Merge branch 'master' into improve-strtod
MaxGraey d53e988
rebuild tests
MaxGraey 25631c9
fixes according review
MaxGraey 64808ce
Merge branch 'master' into improve-strtod
MaxGraey 4c3d937
rebuild tests
MaxGraey 277fdaf
Fixes which to address the comments
MaxGraey aae9aee
add some test262 tests. Fix pow10
MaxGraey 396e806
more test262 tests. Minor fix
MaxGraey 396268b
last test from test262 set
MaxGraey 9b5c3e9
minor refactoring
MaxGraey b3a14aa
simplify pow10
MaxGraey 6c44c8b
update comment
MaxGraey d83309a
add comments about failure cases
MaxGraey b516f21
Merge branch 'master' into improve-strtod
MaxGraey fc2c425
rebuild tests
MaxGraey a9a6e7c
Merge branch 'master' into improve-strtod
MaxGraey 36733b9
rebuild
MaxGraey 23c28cd
refactor
MaxGraey 6bbcf3c
Merge branch 'master' into improve-strtod
MaxGraey 06dda3b
rebuild
MaxGraey File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this useful to a user as a public API?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess yes. It's has different name in C/C++ as
ldexp