Re: uintptr_t for Datum

Поиск
Список
Период
Сортировка
От Magnus Hagander
Тема Re: uintptr_t for Datum
Дата
Msg-id 9837222c0912310846u7845cac9v55a534c953c1766b@mail.gmail.com
обсуждение исходный текст
Ответ на Re: uintptr_t for Datum  (Tom Lane <tgl@sss.pgh.pa.us>)
Ответы Re: uintptr_t for Datum  (Tom Lane <tgl@sss.pgh.pa.us>)
Список 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 по дате отправления:

Предыдущее
От: Tom Lane
Дата:
Сообщение: Re: uintptr_t for Datum
Следующее
От: Robert Haas
Дата:
Сообщение: Re: point_ops for GiST