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 wrap around issues on large numerics #543

Closed
wants to merge 4 commits into from
Closed

Fix wrap around issues on large numerics #543

wants to merge 4 commits into from

Commits on May 27, 2022

  1. Fix wrap around issues on large numerics 

    There seemed to be a mistake on LLONG_MAX vs ULLONG_MAX (the latter is
the maximum value for an object of type `unsigned long long int` which
JSUINT64 should be). I fixed that.

I am unsure about how much performance is lost on the check for overflow
on digit check...
    Joachim Lusiardi committed May 27, 2022
    Configuration menu
    Copy the full SHA
    cabb84a View commit details
    Browse the repository at this point in the history

  2. [pre-commit.ci] auto fixes from pre-commit.com hooks 

    for more information, see https://pre-commit.ci
    @pre-commit-ci
    pre-commit-ci[bot] committed May 27, 2022
    Configuration menu
    Copy the full SHA
    06b2e63 View commit details
    Browse the repository at this point in the history

  3. remove exclamation marks

    Joachim Lusiardi committed May 27, 2022
    Configuration menu
    Copy the full SHA
    15f3635 View commit details
    Browse the repository at this point in the history

  4. Merge branch 'issue_440' of github.com:jlusiardi/ultrajson into issue… 

    …_440
    Joachim Lusiardi committed May 27, 2022
    Configuration menu
    Copy the full SHA
    c98d5cb View commit details
    Browse the repository at this point in the history