chicken-hackers
[Top][All Lists]
Advanced

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

Re: [Chicken-hackers] [PATCH] Re: make check failing


From: Felix
Subject: Re: [Chicken-hackers] [PATCH] Re: make check failing
Date: Sat, 05 Jan 2013 20:22:31 +0100 (CET)

From: Peter Bex <address@hidden>
Subject: Re: [Chicken-hackers] [PATCH] Re: make check failing
Date: Sat, 5 Jan 2013 00:24:52 +0100

> On Sat, Jan 05, 2013 at 12:16:21AM +0100, Felix wrote:
>> > Actually, if you want to write portable code you must, and you must also
>> > wrap it in inexact->exact.  However, since the FFI isn't portable anyway
>> > it won't matter that much, except when if decide to switch to supporting
>> > bignums in core.  When that happens, all code that doesn't round and
>> > convert to exact will break unless we decide to keep this for backwards
>> > compat for a while.  But eventually it'll break.
>> 
>> I don't write portable code, and a switch to supporting bignums in the
>> core-system is not decided on, yet.
> 
> True, but it could happen!  Maybe not this year, but who knows what'll
> happen the next 10 years :)
> 
>> > (vector-ref (vector 1 2 3 4) 1.5) shouldn't
>> > work, why then should ((foreign-lambda void do-something int) 1.5),
>> > especially considering vector-ref could reasonably be defined as this:
>> > 
>> > (lambda (v i) 
>> >   (check-type 'vector v)
>> >   ((foreign-lambda scheme-object C_block_item scheme-object int) v i))?
>> 
>> Because these are two completely different things, and you know that.
> 
> Well, yes.  But still, I don't see why it has to be different.  If you
> really feel strongly about it I'll push your change as-is; I don't feel
> as strongly about it; I just have this faint hunch that it'll help
> prevent some small class of bugs.  Just say the word.

I have pushed your amended patch. There is no convincing argument that
I could give.


cheers,
felix



reply via email to

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