diff --git a/ChangeLog b/ChangeLog index fce63ee0ad..9137bd9c6a 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,10 @@ +Tue Jul 18 10:53:37 2006 Nobuyoshi Nakada + + * object.c (rb_cstr_to_dbl): limit out-of-range message. + + * util.c (ruby_strtod): return end pointer even if ERANGE occurred. + fixed: [ruby-dev:29041] + Mon Jul 18 00:43:05 2006 Nobuyoshi Nakada * util.c (ruby_strtod): stop at dot not followed by digits. diff --git a/object.c b/object.c index 733f4f88f9..9ede440c53 100644 --- a/object.c +++ b/object.c @@ -2063,13 +2063,17 @@ rb_cstr_to_dbl(const char *p, int badcheck) const char *q; char *end; double d; + const char *ellipsis = ""; + int w; +#define OutOfRange() (((w = end - p) > 20) ? (w = 20, ellipsis = "...") : (ellipsis = "")) if (!p) return 0.0; q = p; while (ISSPACE(*p)) p++; d = strtod(p, &end); if (errno == ERANGE) { - rb_warn("Float %*s out of range", end-p, p); + OutOfRange(); + rb_warn("Float %.*s%s out of range", w, p, ellipsis); errno = 0; } if (p == end) { @@ -2100,18 +2104,20 @@ rb_cstr_to_dbl(const char *p, int badcheck) p = buf; d = strtod(p, &end); if (errno == ERANGE) { - rb_warn("Float %*s out of range", end-p, p); + OutOfRange(); + rb_warn("Float %.*s%s out of range", w, p, ellipsis); errno = 0; } if (badcheck) { - if (p == end) goto bad; + if (!end || p == end) goto bad; while (*end && ISSPACE(*end)) end++; if (*end) goto bad; } } if (errno == ERANGE) { errno = 0; - rb_raise(rb_eArgError, "Float %s out of range", q); + OutOfRange(); + rb_raise(rb_eArgError, "Float %.*s%s out of range", w, q, ellipsis); } return d; } diff --git a/util.c b/util.c index 867b14c4b0..3f85c5b13d 100644 --- a/util.c +++ b/util.c @@ -874,11 +874,13 @@ ruby_strtod( if (exp >= MDMAXEXPT) { errno = ERANGE; - return HUGE_VAL * (sign ? -1.0 : 1.0); + fraction = HUGE_VAL; + goto ret; } else if (exp < MDMINEXPT) { errno = ERANGE; - return 0.0 * (sign ? -1.0 : 1.0); + fraction = 0.0; + goto ret; } fracExp = exp; exp += 9; @@ -924,6 +926,7 @@ ruby_strtod( fraction = frac1 + frac2; } + ret: if (endPtr != NULL) { *endPtr = (char *)p; }