Re: get rid of Pointer type, mostly

> On Nov 24, 2025, at 18:20, Peter Eisentraut <peter@eisentraut.org> wrote:
>
> In a previous thread[0], the question was asked, 'Why do we bother with a "Pointer" type?'.  So I looked into get rid
ofit. 
>
> There are two stages to this.  One is changing all code that wants to do pointer arithmetic to use char * instead of
relyingon Pointer being char *.  Then we can change Pointer to be void * and remove a bunch of casts. 
>
> The second is getting rid of uses of Pointer for variables where you might as well use void * directly.  These are
actuallynot that many. 
>
> This gets rid of all uses, except in the GIN code, which is full of Pointer use, and it's part of the documented API.
I'm not touching that, not least because this kind of code 
>
>    Pointer   **extra_data = (Pointer **) PG_GETARG_POINTER(4);
>
> needs more brain-bending to understand that I'm prepared to spend.  So as far as I'm concerned, the pointer type can
continueto exist as a curiosity of the GIN API, but in all other places, it wasn't really doing much of anything
anyway.
>
>
> [0]:
https://www.postgresql.org/message-id/CA%2BhUKG%2BNFKnr%3DK4oybwDvT35dW%3DVAjAAfiuLxp%2B5JeZSOV3nBg%40mail.gmail.com<0001-Remove-useless-casts-to-Pointer.patch><0002-Use-better-DatumGet-function.patch><0003-Don-t-rely-on-pointer-arithmetic-with-Pointer-type.patch><0004-Change-Pointer-to-void.patch><0005-Remove-no-longer-needed-casts-to-Pointer.patch><0006-Remove-some-uses-of-the-Pointer-type.patch>

0001 - Removed type cast from memcpy(), which should be safe, as memcpy doesn’t care about types of source and dest
pointersreferring to, type casting is redundant. 

0002 - Changed DatumGetPointer() to DatumGetCString() for %s. Basically, DatumGetPointer() returns a void * pointer and
DatumGetCString()returns a char * pointers, but they return the same addresses, thus DatumGetCString() better fits %s. 

0003 - Changed type casting from Pointer to char *. Now, Pointer is a typedef of char *, so the replacement is safe. In
generic_desc(),Pointer is replaced with const char *, which should be safe also. 

0004 - Changed typedef of Pointer from char * to void *. I guess the purpose is to let compiler alter for missed usages
ofPointer. 

0005/0006 - Removed all usages of Pointer expect in Gin code.

All look good. Only things is that, as Pointer is changed from char * to void *, and Gin code are still using Pointer,
sothese is a change for Gin code. But I don’t think that would impact runtime, as long as build passes, that should be
fine.Build passed on my MacBook M4. If there is a breakage, build farm should be able to catch the error. 

Best regards,
--
Chao Li (Evan)
HighGo Software Co., Ltd.
https://www.highgo.com/