Bug #2452

bigdecimal/math のレビュー

Added by Sho Hashimoto about 2 years ago. Updated 10 months ago.

[ruby-reference-manual:1893]
Status:Closed Start date:12/07/2009
Priority:Normal Due date:
Assignee:- % Done:

0%

Category:doc:review
Target version:-
reporter: ruby_version:

Description

bigdecimal/math のレビューを行う。

やり方は、 http://doc.loveruby.net/wiki/Phase3ReviewProcess1.html を参照してください。

Associated revisions

Revision 3910
Added by sho-h about 2 years ago

apply a patch from 5.5 (see #2452)

History

Updated by 5 5 about 2 years ago

  • Priority changed from Normal to Low
library bigdecimal/math の「要約」で,
 E (prec) == exp(1.0, prec)
という記述がありますが,exp の第一引数は 1.0 を受け付けません。
(BigDecimal オブジェクトを与えるべき)
また,E の定義で内部的に exp を使っているわけでもないようです。
したがって,「== exp(1.0, prec)」の部分は無いほうがよいと思います。
※もともと bigdecimal/math.rb のコメントがこうなっているのですが…。

また,BigMath#E の説明で「自然対数 e」とあるのは「自然対数の底 e」の誤りでしょう。


Index: refm/api/src/bigdecimal/math.rd
===================================================================
--- refm/api/src/bigdecimal/math.rd	(revision 3909)
+++ refm/api/src/bigdecimal/math.rd	(working copy)
@@ -11,7 +11,7 @@
  * exp (x, prec)
  * log (x, prec)
  * PI  (prec)
- * E   (prec) == exp(1.0, prec)
+ * E   (prec)

 引数:

@@ -247,7 +247,7 @@

 --- E(prec) -> BigDecimal

-自然対数 e を prec で指定した精度で計算します。
+自然対数の底 e を prec で指定した精度で計算します。

 @param prec 計算結果の精度。

Updated by 5 5 about 2 years ago

  • Priority changed from Low to Normal
すみません,誤って優先度を変更してしまいました。
元に戻します。

Updated by Sho Hashimoto about 2 years ago

ありがとうございます。そのまま適用しました。

Updated by okkez _ over 1 year ago

  • Status changed from Open to Feedback
レビューしました。
特に問題ないと思います。

Updated by okkez _ over 1 year ago

  • Status changed from Feedback to Closed
close します。

Also available in: Atom PDF