chicken-hackers
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Chicken-hackers] pending patches


From: Christian Kellermann
Subject: Re: [Chicken-hackers] pending patches
Date: Fri, 16 Mar 2012 12:04:19 +0100
User-agent: Mutt/1.5.21 (2010-09-15)

* Christian Kellermann <address@hidden> [120316 09:53]:
> * felix winkelmann <address@hidden> [120316 08:33]:
> > >> > - [PATCH] Bugfix for #791 and unpack flonums correctly for integer?
> > >> 
> > >> Can't remember.
> > > 
> > > http://lists.nongnu.org/archive/html/chicken-hackers/2012-03/msg00010.html
> > > 
> > 
> > Signed off and pushed.
> 
> This patch breaks the tests for me. This is due to library tests
> being compiled with specialisation.
> 
> The test in question is:
> 
> (assert (not (integer? +inf.)))
> 
> 
> This works fine in csi:
> 
> #;1> (not (integer? +inf.))
> #t
> 
> Also the assert works fine.
> 
> Did make check pass for you with the compiled library tests? I am
> testing on a 64 bit linux.

I have attached a patch to fix the specialisation for the C_u_i_fpintegerp.

BUT while this patch looks ok, it segfaults for me when doing the library tests.
Could it be that the type guessing here is wrong in a particular case?

The patch looks fine and is meant to compile ;)

Cheers,

Christian

-- 
9 out of 10 voices in my head say, that I am crazy,
one is humming.

Attachment: 0001-Check-for-isnan-isinf-in-C_u_i_fpintegerp.patch
Description: Text document


reply via email to

[Prev in Thread] Current Thread [Next in Thread]