Re: Portability bug with UINTPTR_MAX in Solaris/Forte

[Kevin Ryde]

Matthias Koeppe <address@hidden> writes:
>
> (I still think that tricky
> things like this should be checked at `configure' time, rather than
> during compilation, though.)

Oh, well, no need to add to the configure script if a cpp conditional
can do it cleanly and portably.

> +/* On Solaris 7 and 8, /usr/include/sys/int_limits.h defines
> +   INTPTR_MAX and UINTPTR_MAX to empty, INTPTR_MIN is not defined.

That's a typo there is it?  Only UINTPTR_MAX defined to empty.

> +#if SCM_SIZEOF_INTPTR_T != 0 && defined(INTPTR_MAX) && defined(INTPTR_MIN) 
> && INTPTR_MAX-0 != 0 && INTPTR_MIN-0 != 0

While you're at it you might like to merge the tests so scm_t_bits and
scm_t_signed_bits are both based on "intptr" stuff, or both on "long",
rather than having separate conditionals.  Wouldn't want them to come
out different.