Re: Guile FFI wrapper, t_scm_word ambiguity

Andreas Rottmann <[email protected]> writes:

>> The non-reps parts of "real" Pika (e.g., libscm/hash-values.[ch]
>> should _not_ be using t_scm_word as an integer type.   Those are
>> bugs.   They should be using t_scm_fixnum and should be reviewed for
>> sign issues and overflow issues.
>>
> Funny how things line up together: The next problem for me was
> hashing; the unit tests run OK so far, up to unit-vector-hash, which
> fails due to bogus hash value calculation. I guess I'll now pass the
> ball, waiting for someone else fix this bug (I'm not feeling familiar
> with the guts of Pika yet), and then have a look at the vector hashes
> again. 
>
I take that back (with using t_scm_fixnum, some pieces became much
more obvious to me); if you are comfortable with waiting a few days,
I'll sort the fixes for Pika out of my code and will provide them as a
changeset.

Andy
-- 
Andreas Rottmann         | [email protected]      | 118634484@ICQ | [email protected]
http://yi.org/rotty      | GnuPG Key: http://yi.org/rotty/gpg.asc
Fingerprint              | DFB4 4EB4 78A4 5EEE 6219  F228 F92F CFC5 01FD 5B62

Make free software, not war!