Skip to content

seq: add a lower bound for exponents#7145

Merged
sylvestre merged 1 commit intouutils:mainfrom
alexs-sh:seq-fix-min-exponent
Jan 16, 2025
Merged

seq: add a lower bound for exponents#7145
sylvestre merged 1 commit intouutils:mainfrom
alexs-sh:seq-fix-min-exponent

Conversation

@alexs-sh
Copy link
Contributor

@alexs-sh alexs-sh commented Jan 16, 2025

About

This PR adds a lower bound for the minimum exponent value to prevent overflow when parsing BigDecimal numbers.

Before

cargo run -- seq 1e-9223372036854775808
seq: invalid floating point argument: '1e-9223372036854775808'

After

cargo run -- seq 1e-9223372036854775808

Expected

seq 1e-9223372036854775808

Thank you

Add bounds for exponents to avoid overflow issues for inputs like 'seq
1e-9223372036854775808'
@github-actions
Copy link

GNU testsuite comparison:

GNU test failed: tests/misc/stdbuf. tests/misc/stdbuf is passing on 'main'. Maybe you have to rebase?
Skip an intermittent issue tests/tail/inotify-dir-recreate (fails in this run but passes in the 'main' branch)
Congrats! The gnu test tests/seq/seq-precision is no longer failing!

@sylvestre
Copy link
Contributor

Congrats! The gnu test tests/seq/seq-precision is no longer failing!

Excellent!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants