Rework HeapTupleHeader macros to reuse itemptr.h
authorAlvaro Herrera <alvherre@alvh.no-ip.org>
Mon, 22 Feb 2021 20:21:22 +0000 (17:21 -0300)
committerAlvaro Herrera <alvherre@alvh.no-ip.org>
Thu, 25 Mar 2021 18:47:48 +0000 (15:47 -0300)
The original definitions pointlessly disregarded existing ItemPointer
macros that do the same thing.

Reported-by: Michael Paquier <michael@paquier.xyz>
Discussion: https://postgr.es/m/20210222201557.GA32655@alvherre.pgsql

src/include/access/htup_details.h

index 9a4f43c1ff155c5dbdf13c8c3caf39f1f0bb38cb..960772f76b999d2921232977eefd2d561d46ee8d 100644 (file)
@@ -443,11 +443,10 @@ do { \
 )
 
 #define HeapTupleHeaderIndicatesMovedPartitions(tup) \
-       (ItemPointerGetOffsetNumber(&(tup)->t_ctid) == MovedPartitionsOffsetNumber && \
-        ItemPointerGetBlockNumberNoCheck(&(tup)->t_ctid) == MovedPartitionsBlockNumber)
+       ItemPointerIndicatesMovedPartitions(&(tup)->t_ctid)
 
 #define HeapTupleHeaderSetMovedPartitions(tup) \
-       ItemPointerSet(&(tup)->t_ctid, MovedPartitionsBlockNumber, MovedPartitionsOffsetNumber)
+       ItemPointerSetMovedPartitions(&(tup)->t_ctid)
 
 #define HeapTupleHeaderGetDatumLength(tup) \
        VARSIZE(tup)