Author: juraj
Date: 2008-02-08 11:29:03 -0500 (Fri, 08 Feb 2008)
New Revision: 95272
Modified:
trunk/mcs/class/corlib/System/ChangeLog
trunk/mcs/class/corlib/System/Random.cs
Log:
2008-02-08 Juraj Skripsky <[EMAIL PROTECTED]>
* Random.cs (Next): Use rounding (via implicit conversion) only on
positive values. Fixes bug 359918.
Modified: trunk/mcs/class/corlib/System/ChangeLog
===================================================================
--- trunk/mcs/class/corlib/System/ChangeLog 2008-02-08 16:06:32 UTC (rev
95271)
+++ trunk/mcs/class/corlib/System/ChangeLog 2008-02-08 16:29:03 UTC (rev
95272)
@@ -1,3 +1,8 @@
+2008-02-08 Juraj Skripsky <[EMAIL PROTECTED]>
+
+ * Random.cs (Next): Use rounding (via implicit conversion) only on
+ positive values. Fixes bug 359918.
+
2008-02-03 Sebastien Pouliot <[EMAIL PROTECTED]>
* Environment.cs: Remove unused method found with Gendarme.
Modified: trunk/mcs/class/corlib/System/Random.cs
===================================================================
--- trunk/mcs/class/corlib/System/Random.cs 2008-02-08 16:06:32 UTC (rev
95271)
+++ trunk/mcs/class/corlib/System/Random.cs 2008-02-08 16:29:03 UTC (rev
95272)
@@ -122,8 +122,7 @@
if (diff == 0)
return minValue;
- int result = (int)(Sample () * diff + minValue);
- return ((result != maxValue) ? result : (result - 1));
+ return (int)(Sample () * diff) + minValue;
}
public virtual void NextBytes (byte [] buffer)
_______________________________________________
Mono-patches maillist - [email protected]
http://lists.ximian.com/mailman/listinfo/mono-patches
