Re: svn commit: r654100 - in /commons/proper/math/trunk/src: java/org/apache/commons/math/analysis/BrentSolver.java site/xdoc/changes.xml test/org/apache/commons/math/analysis/BrentSolverTest.java

["J.Pietschmann" <j3...@yahoo.de>]

brentworden@apache.org wrote:
> +        if (sign>  0) {
> +            // check if either value is close to a zero

I consider this a dangerous move, because this may indicate even
for a strictly positive (or negative) function that it has a
zero.
Note that the situation is different once we get into solving,
because we *know* in this case that a root must be somewhere in
the interval, it is just that the root value may be somewhat off.

J.Pietschmann


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org