Thread overview
[Issue 416] New: regression: unittest failure in phobos, math(501)
Oct 09, 2006
d-bugmail
Oct 10, 2006
Don Clugston
[Issue 416] mistake in unittest std/math.d line 501
Oct 10, 2006
d-bugmail
Oct 18, 2006
d-bugmail
October 09, 2006
http://d.puremagic.com/issues/show_bug.cgi?id=416

           Summary: regression: unittest failure in phobos, math(501)
           Product: D
           Version: 0.169
          Platform: PC
        OS/Version: Windows
            Status: NEW
          Severity: normal
          Priority: P2
         Component: DMD
        AssignedTo: bugzilla@digitalmars.com
        ReportedBy: lio@lunesu.com


Phobos build flags (win32.mak):
CFLAGS=-g -mn -6 -r
DFLAGS=-unittest -g -w

unittest
Error: AssertError Failure math(501)

The offending line in std/math.d:
assert(isNegZero(atanh(-real.infinity)));

The assert was there in 0.168 but did not fail then.


-- 

October 10, 2006
d-bugmail@puremagic.com wrote:
> http://d.puremagic.com/issues/show_bug.cgi?id=416
> 
>            Summary: regression: unittest failure in phobos, math(501)
>            Product: D
>            Version: 0.169
>           Platform: PC
>         OS/Version: Windows
>             Status: NEW
>           Severity: normal
>           Priority: P2
>          Component: DMD
>         AssignedTo: bugzilla@digitalmars.com
>         ReportedBy: lio@lunesu.com
> 
> 
> Phobos build flags (win32.mak):
> CFLAGS=-g -mn -6 -r
> DFLAGS=-unittest -g -w
> 
> unittest
> Error: AssertError Failure math(501)
> 
> The offending line in std/math.d:
> assert(isNegZero(atanh(-real.infinity))); 
> 
> The assert was there in 0.168 but did not fail then.

I don't know how it passed before. The new compiler is correct!
That line should be changed to:
assert(isnan(atanh(-real.infinity)));

It was my mistake in the original code that I submitted.


October 10, 2006
http://d.puremagic.com/issues/show_bug.cgi?id=416


lio@lunesu.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|regression: unittest failure|mistake in unittest
                   |in phobos, math(501)        |std/math.d line 501




------- Comment #2 from lio@lunesu.com  2006-10-10 06:37 -------
From the Don:

I don't know how it passed before. The new compiler is correct! That line should be changed to:

assert(isnan(atanh(-real.infinity)));

It was my mistake in the original code that I submitted.


-- 

October 18, 2006
http://d.puremagic.com/issues/show_bug.cgi?id=416


bugzilla@digitalmars.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED




------- Comment #3 from bugzilla@digitalmars.com  2006-10-18 13:26 -------
Fixed DMD 0.170


--