[m-rev.] for review: mercury implementation of string__format

Fergus Henderson fjh at cs.mu.OZ.AU
Fri Nov 15 00:16:09 AEDT 2002


On 14-Nov-2002, Peter Ross <pro at missioncriticalit.com> wrote:
> petdr wrote:
> > fjh wrote:
> > > On 13-Nov-2002, Peter Ross <pro at missioncriticalit.com> wrote:
> > > BTW, can you explain why the -max*max tests added in your previous
> commit
> > > (for which the relative diff was not posted for review) were coming out
> > > as NaN rather than -Inf?  Isn't that a bug?
> > >
> > No I can't explain it.  It looks like a bug to me, but it comes from using
> > the C sprintf function.
> so it is not clear to me how it could be fixed.

Well, the first thing to do is to analyze whether the bug is occurring
in the C sprintf function, or whether the numbers are being computed
incorrectly before sprintf was called.  

If parts of the test can't easily be fixed, then those parts should
be moved to to a separate test which is disabled and marked with an
XXX comment in the Mmakefile.

-- 
Fergus Henderson <fjh at cs.mu.oz.au>  |  "I have always known that the pursuit
The University of Melbourne         |  of excellence is a lethal habit"
WWW: <http://www.cs.mu.oz.au/~fjh>  |     -- the last words of T. S. Garp.
--------------------------------------------------------------------------
mercury-reviews mailing list
post:  mercury-reviews at cs.mu.oz.au
administrative address: owner-mercury-reviews at cs.mu.oz.au
unsubscribe: Address: mercury-reviews-request at cs.mu.oz.au Message: unsubscribe
subscribe:   Address: mercury-reviews-request at cs.mu.oz.au Message: subscribe
--------------------------------------------------------------------------



More information about the reviews mailing list