[Mono-bugs] [Bug 62082][Wis] New - Different, maybe wrong, number in Random.cs implementation

bugzilla-daemon@bugzilla.ximian.com bugzilla-daemon@bugzilla.ximian.com
Thu, 29 Jul 2004 07:56:44 -0400 (EDT)

Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

Changed by azugaldia@yahoo.es.


--- shadow/62082	2004-07-29 07:56:44.000000000 -0400
+++ shadow/62082.tmp.28563	2004-07-29 07:56:44.000000000 -0400
@@ -0,0 +1,26 @@
+Bug#: 62082
+Product: Mono: Class Libraries
+Version: unspecified
+OS Details: 
+Status: NEW   
+Priority: Wishlist
+Component: CORLIB
+AssignedTo: mono-bugs@ximian.com                            
+ReportedBy: azugaldia@yahoo.es               
+QAContact: mono-bugs@ximian.com
+TargetMilestone: ---
+Summary: Different, maybe wrong, number in Random.cs implementation
+In line number 77 of [mono]/mcs/class/corlib/System/Random.cs we see      
+                  "inextp = 21" while if we check "Numerical Recipes in C
+online @ http://www.library.cornell.edu/nr/bookcpdf/c7-1.pdf" we see it
+must be "inextp = 31" because "the constant 31 is special; see Knuth".
+Although I don't know why it's needed a 31 instead of a 21 I don't see any
+reason to change the number. Maybe the Random class mus be reviewed by its
+author to check it hasn't any more subtle "bugs".