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

Check we don't overflow when casting down integers during parsing #4353

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from
Open

Check we don't overflow when casting down integers during parsing #4353

wants to merge 3 commits into from

Commits on Apr 19, 2024

  1. Check we don't overflow when casting down integers during parsing

    @ArnaudBienner
    ArnaudBienner committed Apr 19, 2024
    Configuration menu
    Copy the full SHA
    da5800c View commit details
    Browse the repository at this point in the history

Commits on Apr 20, 2024

  1. * Fix check 

    * Make the check optional if the target type is greater or equal to source type, or if source is floating point number
* Ignore infinity values, as they are already handled
    @ArnaudBienner
    ArnaudBienner committed Apr 20, 2024
    Configuration menu
    Copy the full SHA
    bd9d0b5 View commit details
    Browse the repository at this point in the history

  2. Use std::isinf

    @ArnaudBienner
    ArnaudBienner committed Apr 20, 2024
    Configuration menu
    Copy the full SHA
    d2f1d90 View commit details
    Browse the repository at this point in the history