| From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | Peter Eisentraut <peter(at)eisentraut(dot)org>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: get rid of Pointer type, mostly |
| Date: | 2025-11-24 16:19:42 |
| Message-ID: | CA+Tgmobo9ZMC0h1Z1Te9whf7q3LN7wUkzFLyuxfMZ4dyTNqKnw@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Mon, Nov 24, 2025 at 11:09 AM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> > The second is getting rid of uses of Pointer for variables where you
> > might as well use void * directly. These are actually not that many.
>
> ... but not of that. In particular, I think it's just fine if
> DatumGetPointer and PointerGetDatum take and return Pointer.
What's your objection?
(I don't have a considered opinion on this particular point, but in
general I've found that using Pointer seems to make life worse rather
than better.)
--
Robert Haas
EDB: http://www.enterprisedb.com
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Greg Burd | 2025-11-24 16:28:28 | Re: [PATCH] Fix ARM64/MSVC atomic memory ordering issues on Win11 by adding explicit DMB barriers |
| Previous Message | 河田達也 | 2025-11-24 16:17:19 | [Proposal] Adding TRIM_SPACE option to COPY |