Re: SPI bug.

Thomas Hallgren wrote:
> Tom Lane wrote:
>> Furthermore, we have never promised ABI-level compatibility across
>> versions inside the backend, and we are quite unlikely to make such
>> a promise in the foreseeable future.
>>
> I know that no promises has been made but PostgreSQL is improved every
> day and this would be a very easy promise to make.

Binary compatibility of backend APIs is by no means a "very easy promise
to make," nor would it be a good idea in any case.

> Also, the interpretation of the definition vary between compiler
> vendors. On Windows Itanium, the int is 32 bit. On Unix it's 64.

`int' is 32 bit on most modern platforms I can think of. Perhaps you're
thinking of `long', which is indeed 64-bit on many 64-bit Unixen but
32-bit on 64-bit Windows (BTW, this likely means that Postgres is
completely broken on 64-bit Windows: sizeof(Datum) == sizeof(unsigned
long) == 4, sizeof(void *) == 8).

> The 1998 revision of C declares the following types for a good reason:
>
> int8_t , int16_t, int32_t int64_t,
> uint8_t, uint16_t, uint32_t, uint64_t.

We don't currently depend on C99, and not all platforms have a 64-bit
datatype. In any case, I'm still unconvinced that using `int' and `long'
in backend APIs is a problem.

-Neil