Re: uintptr_t for Datum - Mailing list pgsql-hackers

From Magnus Hagander
Subject Re: uintptr_t for Datum
Date
Msg-id 9837222c0912310846u7845cac9v55a534c953c1766b@mail.gmail.com
Whole thread Raw
In response to Re: uintptr_t for Datum  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: uintptr_t for Datum  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
2009/12/31 Tom Lane <tgl@sss.pgh.pa.us>:
> Magnus Hagander <magnus@hagander.net> writes:
>> Attached patch is the part of the win64 patch that changes Datum to be
>> uintptr_t, and associated changes, with only very minor changes from
>> me. It also includes autoconf tests that I tricked Bruce into fixing
>> for me :-)
>
>> Comments?
>
> This is a joke no?

Hey, it got your attention ;)


>  Where's the logic to provide a definition of
> intptr_t if the platform fails to?  The lack of attention to updating

autoconf does that. This is exactly what broke on Bruce's platform,
and autoconf fixed it in the way that is included in the patch.


> the comments about Datum doesn't give me a warm feeling either.

Will look over that.


> BTW, it looks like the patch is showing a manual change to
> pg_config.h.in.  Don't do that.  Run autoheader.

That also came out of Bruce's patch. Bruce, can you look at doing
that? I don't have a machine easily accessible with the right autoconf
version ATM :(

-- Magnus HaganderMe: http://www.hagander.net/Work: http://www.redpill-linpro.com/


pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: uintptr_t for Datum
Next
From: Robert Haas
Date:
Subject: Re: point_ops for GiST