[ruby-dev:23486] Re: [BUG] numeric.c (flo_to_s) broken?

山本です。

HEADがビルドできるようになったので、HEADに移植しました。

バッファサイズをどうするか悩んだのですが、DBL_DIG * 2 はあまり
安全でないので、下のようにしました。

sizeof(DBL_DIG) * 8 というのは、DBL_DIG のとりうる桁数の代わりです。
（とりうる桁数より常に大きい）

# "%.15g" に見覚えがあったのにどこのスレッドかわからなかったのですが、
# [ruby-talk:97891] でした。

Index: numeric.c
===================================================================
RCS file: /ruby/ruby/numeric.c,v
retrieving revision 1.107
diff -u -w -b -p -r1.107 numeric.c
--- numeric.c	7 May 2004 08:44:15 -0000	1.107
+++ numeric.c	8 May 2004 11:17:24 -0000
@@ -489,10 +489,10 @@ static VALUE
 flo_to_s(flt)
     VALUE flt;
 {
-    char buf[32];
-    char *fmt = "%.15f";
+    char fmt[sizeof(DBL_DIG) * 8 + 10];
+    char buf[DBL_DIG + (sizeof(FLT_MAX_EXP) + sizeof(FLT_MIN_EXP)) * 8 + 10];
     double value = RFLOAT(flt)->value;
-    double avalue, d1, d2;
+    int exp;
     char *p, *e;
 
     if (isinf(value))
@@ -500,13 +500,25 @@ flo_to_s(flt)
     else if(isnan(value))
 	return rb_str_new2("NaN");
 
-    avalue = fabs(value);
-    if (avalue < 1.0e-7 || avalue >= 1.0e15) {
-	fmt = "%.16e";
-    }    
+    sprintf(fmt, "%%.%de", DBL_DIG - 1);
     sprintf(buf, fmt, value);
-    if (!(e = strchr(buf, 'e'))) {
-	e = buf + strlen(buf);
+    e = strchr(buf, 'e');
+    exp = atoi(e+1);
+    if (-7 <= exp && exp < DBL_DIG - 1) {
+	char *d = strchr(buf, '.');
+	if (exp > 0) {
+	    memmove(d, d+1, exp);
+	    d[exp] = '.';
+	}
+	if (exp < 0) {
+	    int n = -exp;
+	    d[0] = d[-1];
+	    memmove(d+n, d, strlen(d)+1);
+	    e += n;
+	    memset(buf, '0', n+1);
+	    buf[1] = '.';
+	}
+	*e = '\0';
     }
     p = e;
     while (*--p=='0')