From: Yusuke ENDOH <mame@...>
Date: 2010-02-09T23:09:30+09:00
Subject: [ruby-dev:40375] Re: メンテナ確認プロセス終了。メンテナ募集

遠藤です。

2010年2月9日15:07 Shigeo Kobayashi <shigeo@tinyforest.gr.jp>:
> 小林＠仕事中
> です。
> ----- Original Message ----- From: "Shigeo Kobayashi"
> <shigeo@tinyforest.gr.jp>
> Subject: [ruby-dev:40355] Re: メンテナ確認プロセス終了。メンテナ募集
>
>> あと、BigDecimal('1').divmod(BigDecimal('3E-9'))の件が話題になっ
>> ていますが、先月投稿したパッチ（[ruby-dev:40105] ）はダメ？
>
> 早速指摘が...
>
> +    mx = a->Prec+abs(a->exponent);
> +    if(mx<b->Prec+abs(b->exponent)) mx = b->Prec+abs(b->exponent);
>
> ですね。失礼しました。


おおお、このパッチ＋加減算の前の precision の修正で rubyspec を
パスしました (ただし rubyspec 側にも一箇所修正あり) 。

ありがとうございます。とりあえずはこれをコミットさせて頂きます。


diff --git a/ext/bigdecimal/bigdecimal.c b/ext/bigdecimal/bigdecimal.c
index 8e85fdf..67622ea 100644
--- a/ext/bigdecimal/bigdecimal.c
+++ b/ext/bigdecimal/bigdecimal.c
@@ -970,8 +970,8 @@ BigDecimal_DoDivmod(VALUE self, VALUE r, Real
**div, Real **mod)
        return Qtrue;
     }

-    mx = a->Prec;
-    if(mx<b->Prec) mx = b->Prec;
+    mx = a->Prec+abs(a->exponent);
+    if(mx<b->Prec+abs(b->exponent)) mx = b->Prec+abs(b->exponent);
     mx =(mx + 1) * VpBaseFig();
     GUARD_OBJ(c,VpCreateRbObject(mx, "0"));
     GUARD_OBJ(res,VpCreateRbObject((mx+1) * 2 +(VpBaseFig() + 1), "#0"));
@@ -983,6 +983,7 @@ BigDecimal_DoDivmod(VALUE self, VALUE r, Real
**div, Real **mod)
     VpAddSub(c,a,res,-1);
     if(!VpIsZero(c) && (VpGetSign(a)*VpGetSign(b)<0)) {
         VpAddSub(res,d,VpOne(),-1);
+	GUARD_OBJ(d,VpCreateRbObject(GetAddSubPrec(c, b)*(VpBaseFig() + 1), "0"));
         VpAddSub(d  ,c,b,       1);
         *div = res;
         *mod = d;

-- 
Yusuke ENDOH <mame@tsg.ne.jp>