Re: [9front] APE stddef.h ptrdiff_t should it be 8?

[jamos@oboj.net]

Others on this list of course know these things
much better than me, but this is my understanding:

The header file, as it is different for different
architectures should be in /$objtype/inlcude/ape _instead_
of in the current location. You'd have to have one
header file each objtype to, but it would probably only
be the definition of ptrdiff_t that would differ,
depending on 32 or 64 bit addresses for the arch.
In Plan 9, include files does in general not include other
include files either.

And it should be 'long long' in ape, as vlong isn't defined.

/Jonas

On 2020-03-08 00:14, Trevor Higgins wrote:

> On 03/08/2020 04:55 AM, ori@eigenstate.org wrote: I ran into a 
> fustercluck trying to get tcl to run on amd64.
  just random memory corruptions as would be expected.

>> Can you be more specific? What failed?
>> 
>> Turns out it is due to APE ptrdiff_t being defined as long and not 
>> long
>> long (which it is on *nix).
>> 
>> What is the correct definition of ptrdiff_t on amd64?
> The definition should be a vlong, since that's what subtracting
> two pointers yields. If you want to submit a patch, the system
> dependent location for this is:
> 
> /$objtype/include/ape/stdint.h
  Not wishing to be picky, but vlong would not be APE , correct or not? 
it would be typedef long long.
Also this change still requires patching /sys/include/ape/stddef.h to 
prevent attempt to duplicately define the structure
and Probably for good measure in both files:
#ifndef _PTRDIFF_T
#define _PTRDIFF_T
typedef  <arch specific>  ptrdiff_t   /*** long long in amd64 and just 
long in sys/include/ape/stddef.h **/
#endif

Or maybe , ptrdiff_t should be removed from stddef.h