[Ecls-list] Fwd: Re: Re: latest cvs update fails on cygwin

Larry Clapp larry at theclapp.org
Sun Oct 22 19:23:59 UTC 2006


On Sun, Oct 22, 2006 at 09:04:02PM +0200, Juan Jose Garcia-Ripoll wrote:
> 2006/10/22, Larry Clapp <larry at theclapp.org>:
> >   static double
> >   round_double(double d)
> >   {
> >           if (d > 0) {
> >                   double q = floor(q + 0.5);
> 
> Ooops. Stupid typo. Funny that on four platforms it gave the right result :-/

Weeeell, it didn't outright crash.  Dunno if that counts as "right".
:)  In any case, I suspect somebody needs to add to their test cases
(or change the default compiler options!) if that kind of error can
slip through.

Note btw that round_long_double() has the same error.

-- L





More information about the ecl-devel mailing list