diff options
| author | Peter Eisentraut <peter@eisentraut.org> | 2025-12-03 08:40:33 +0100 |
|---|---|---|
| committer | Peter Eisentraut <peter@eisentraut.org> | 2025-12-03 08:40:33 +0100 |
| commit | 623801b3bdb4e16df39f945b6aa5f109744c7b6b (patch) | |
| tree | aca6681a91d929d42747c6695717a621fad18c8d | |
| parent | c252d37d8ca8f254fa715ec24fcb23a035a88993 (diff) | |
Remove useless casts to Pointer
in arguments of memcpy() and memmove() calls
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
| -rw-r--r-- | contrib/bloom/blutils.c | 2 | ||||
| -rw-r--r-- | contrib/bloom/blvacuum.c | 3 | ||||
| -rw-r--r-- | src/backend/access/gin/ginxlog.c | 2 | ||||
| -rw-r--r-- | src/backend/utils/adt/multirangetypes.c | 2 |
4 files changed, 4 insertions, 5 deletions
diff --git a/contrib/bloom/blutils.c b/contrib/bloom/blutils.c index 2c0e71eedc6..bf50037a71a 100644 --- a/contrib/bloom/blutils.c +++ b/contrib/bloom/blutils.c @@ -336,7 +336,7 @@ BloomPageAddItem(BloomState *state, Page page, BloomTuple *tuple) /* Copy new tuple to the end of page */ opaque = BloomPageGetOpaque(page); itup = BloomPageGetTuple(state, page, opaque->maxoff + 1); - memcpy((Pointer) itup, (Pointer) tuple, state->sizeOfBloomTuple); + memcpy(itup, tuple, state->sizeOfBloomTuple); /* Adjust maxoff and pd_lower */ opaque->maxoff++; diff --git a/contrib/bloom/blvacuum.c b/contrib/bloom/blvacuum.c index 9e5f0574fad..920884ca245 100644 --- a/contrib/bloom/blvacuum.c +++ b/contrib/bloom/blvacuum.c @@ -94,8 +94,7 @@ blbulkdelete(IndexVacuumInfo *info, IndexBulkDeleteResult *stats, { /* No; copy it to itupPtr++, but skip copy if not needed */ if (itupPtr != itup) - memmove((Pointer) itupPtr, (Pointer) itup, - state.sizeOfBloomTuple); + memmove(itupPtr, itup, state.sizeOfBloomTuple); itupPtr = BloomPageGetNextTuple(&state, itupPtr); } diff --git a/src/backend/access/gin/ginxlog.c b/src/backend/access/gin/ginxlog.c index 75df3d7a680..606741fa396 100644 --- a/src/backend/access/gin/ginxlog.c +++ b/src/backend/access/gin/ginxlog.c @@ -212,7 +212,7 @@ ginRedoRecompress(Page page, ginxlogRecompressDataLeaf *data) if (tailCopy) { Assert(writePtr + segsize < PageGetSpecialPointer(page)); - memcpy(writePtr, (Pointer) oldseg, segsize); + memcpy(writePtr, oldseg, segsize); } writePtr += segsize; oldseg = GinNextPostingListSegment(oldseg); diff --git a/src/backend/utils/adt/multirangetypes.c b/src/backend/utils/adt/multirangetypes.c index 5273b97f7fe..55e0b4fdc31 100644 --- a/src/backend/utils/adt/multirangetypes.c +++ b/src/backend/utils/adt/multirangetypes.c @@ -629,7 +629,7 @@ write_multirange_data(MultirangeType *multirange, TypeCacheEntry *rangetyp, } flags[i] = *((Pointer) ranges[i] + VARSIZE(ranges[i]) - sizeof(char)); len = VARSIZE(ranges[i]) - sizeof(RangeType) - sizeof(char); - memcpy(ptr, (Pointer) (ranges[i] + 1), len); + memcpy(ptr, ranges[i] + 1, len); ptr += att_align_nominal(len, elemalign); } } |