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.

Sign up for GitHub

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

Jump to bottom

Fix some number serialization bugs #689

Merged
merged 3 commits into from
May 24, 2019
Merged

Fix some number serialization bugs #689

merged 3 commits into from
May 24, 2019

Commits on May 20, 2019

  1. Make SerializeVisitor._removeExponent work with non-exponent numbers 

    This allows us to avoid traversing the number string more times than
necessary.
    @nex3
    nex3 committed May 20, 2019
    Configuration menu
    Copy the full SHA
    5b9835a View commit details
    Browse the repository at this point in the history

  2. Fix a bug in SerializeVisitor._writeDecimal 

    This function didn't expect a number ending in ".0", because they're
normally filtered out by fuzzyAsInt(). However, they can make it
through for very large doubles.
    @nex3
    nex3 committed May 20, 2019
    Configuration menu
    Copy the full SHA
    fb383de View commit details
    Browse the repository at this point in the history

Commits on May 21, 2019

  1. Remove exponent notation for integers on Node.js 

    Closes #688
    @nex3
    nex3 committed May 21, 2019
    Configuration menu
    Copy the full SHA
    0fe0208 View commit details
    Browse the repository at this point in the history