coffee/fast_float
1
0
Fork 0
mirror of https://github.com/fastfloat/fast_float.git synced 2025-12-09 10:16:48 +08:00
Code Issues Packages Projects Releases Wiki Activity

Trying alternate handling of overflows.

Browse Source
This commit is contained in:
Daniel Lemire 2020-12-31 11:14:48 -05:00
parent 47ffc1303b
commit 51b27a3324
1 changed files with 12 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
include/fast_float/ascii_number.h
View File

@ -107,10 +107,10 @@ parsed_number_string parse_number_string(const char *p, const char *pend, chars_
if ((start_digits == p) || ((start_digits == p - 1) && (*start_digits == '.') )) {
return answer;
}
// digit_count is the exact number of digits.
int32_t digit_count =
int32_t(p - start_digits - 1); // used later to guard against overflows
int32_t(p - start_digits); // used later to guard against overflows
if(exponent > 0) {digit_count--;}
if ((fmt & chars_format::scientific) && (p != pend) && (('e' == *p) || ('E' == *p))) {
const char * location_of_e = p;
int64_t exp_number = 0; // exponential part
@ -149,16 +149,21 @@ parsed_number_string parse_number_string(const char *p, const char *pend, chars_
// If we frequently had to deal with long strings of digits,
// we could extend our code by using a 128-bit integer instead
// of a 64-bit integer. However, this is uncommon.
if (((digit_count >= 19))) { // this is uncommon
//
// We can deal with up to 19 digits.
if (((digit_count > 19))) { // this is uncommon
// It is possible that the integer had an overflow.
// We have to handle the case where we have 0.0000somenumber.
// We need to be mindful of the case where we only have zeroes...
// E.g., 0.000000000...000.
const char *start = start_digits;
while (*start == '0' || (*start == '.')) {
while ((start != pend) && (*start == '0' || *start == '.')) {
if(*start == '.') { digit_count++; } // We will subtract it again later.
start++;
}
// we over-decrement by one when there is a decimal separator
// We over-decrement by one when there is a decimal separator
digit_count -= int(start - start_digits);
if (digit_count >= 19) {
if (digit_count > 19) {
answer.mantissa = 0xFFFFFFFFFFFFFFFF; // important: we don't want the mantissa to be used in a fast path uninitialized.
answer.too_many_digits = true;
return answer;