On Sun, 19 Mar 2000, thi wrote:

> before the patch, `(ash 2 -1)' yielded arg-out-of-range error.
> the fix is to `SCM_UNPACK (cnt)' before comparing it w/ zero.
> this change was made to both branches of the `#ifdef SCM_BIGDIG'.

Thanks for the report.  However, we have to be careful not to confuse
SCM_UNPACK with SCM_INUM:
> !   if (SCM_UNPACK (cnt) < 0)
What is actually meant here is to try whether the number (and not the bit
structure) is less than zero.  Thus, SCM_INUM should be used rather than
SCM_UNPACK.

2000-03-20  Dirk Herrmann  <[EMAIL PROTECTED]>

        * numbers.c (scm_ash):  Fixed typing problems with the second
        parameter and added some documentation.  (Thanks Thien-Thi Nguyen
        for indicating the problem.)

And, also thank you for providing the testcases.  They made testing easy
for me!

Best regards
Dirk Herrmann

Reply via email to