[prev in list] [next in list] [prev in thread] [next in thread] 

List:       jakarta-commons-dev
Subject:    [jira] Resolved: (MATH-309) nextExponential parameter check bug -
From:       "Phil Steitz (JIRA)" <jira () apache ! org>
Date:       2009-10-31 2:34:59
Message-ID: 642911247.1256956499369.JavaMail.jira () brutus
[Download RAW message or body]


     [ https://issues.apache.org/jira/browse/MATH-309?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel \
]

Phil Steitz resolved MATH-309.
------------------------------

    Resolution: Fixed

Fixed in r831510.  Thanks for reporting this.

> nextExponential parameter check bug - patch supplied
> ----------------------------------------------------
> 
> Key: MATH-309
> URL: https://issues.apache.org/jira/browse/MATH-309
> Project: Commons Math
> Issue Type: Bug
> Affects Versions: 1.0, 1.1, 1.2, 2.0
> Environment: Ubuntu 9.04
> Reporter: Mikkel Meyer Andersen
> Priority: Minor
> Fix For: 2.1
> 
> Attachments: patch_random_exp
> 
> Original Estimate: 0.08h
> Remaining Estimate: 0.08h
> 
> Index: src/main/java/org/apache/commons/math/random/RandomDataImpl.java
> ===================================================================
> --- src/main/java/org/apache/commons/math/random/RandomDataImpl.java	(revision \
>                 830102)
> +++ src/main/java/org/apache/commons/math/random/RandomDataImpl.java	(working copy)
> @@ -462,7 +462,7 @@
> * @return the random Exponential value
> */
> public double nextExponential(double mean) {
> -        if (mean < 0.0) {
> +        if (mean <= 0.0) {
> throw MathRuntimeException.createIllegalArgumentException(
> "mean must be positive ({0})", mean);
> }

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[prev in list] [next in list] [prev in thread] [next in thread]