diff options
| author | Peter Eisentraut <peter@eisentraut.org> | 2025-12-03 09:54:15 +0100 |
|---|---|---|
| committer | Peter Eisentraut <peter@eisentraut.org> | 2025-12-03 09:54:15 +0100 |
| commit | 756a43689324b473ee07549a6eb7a53a203df5ad (patch) | |
| tree | b72ab16ee77de3e82288c76c7b9493b93d378216 /src/backend/access/rmgrdesc/genericdesc.c | |
| parent | 8c6bbd674ed810df9af5ec42f6b38c205e3ad365 (diff) | |
Don't rely on pointer arithmetic with Pointer type
The comment for the Pointer type says 'XXX Pointer arithmetic is done
with this, so it can't be void * under "true" ANSI compilers.'. This
fixes that. Change from Pointer to use char * explicitly where
pointer arithmetic is needed. This makes the meaning of the code
clearer locally and removes a dependency on the actual definition of
the Pointer type. (The definition of the Pointer type is not changed
in this commit.)
Reviewed-by: Chao Li <li.evan.chao@gmail.com>
Reviewed-by: Bertrand Drouvot <bertranddrouvot.pg@gmail.com>
Discussion: https://www.postgresql.org/message-id/4154950a-47ae-4223-bd01-1235cc50e933%40eisentraut.org
Diffstat (limited to 'src/backend/access/rmgrdesc/genericdesc.c')
| -rw-r--r-- | src/backend/access/rmgrdesc/genericdesc.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/backend/access/rmgrdesc/genericdesc.c b/src/backend/access/rmgrdesc/genericdesc.c index 75dc4108b9a..29a4c9e894b 100644 --- a/src/backend/access/rmgrdesc/genericdesc.c +++ b/src/backend/access/rmgrdesc/genericdesc.c @@ -23,8 +23,8 @@ void generic_desc(StringInfo buf, XLogReaderState *record) { - Pointer ptr = XLogRecGetData(record), - end = ptr + XLogRecGetDataLen(record); + const char *ptr = XLogRecGetData(record); + const char *end = ptr + XLogRecGetDataLen(record); while (ptr < end) { |