Skip to content

Commit f39f6b5

Browse files
committed
Seems that the childXids list would be better based on Oid lists than
integer lists.
1 parent 0007490 commit f39f6b5

File tree

1 file changed

+5
-5
lines changed
  • src/backend/access/transam

1 file changed

+5
-5
lines changed

src/backend/access/transam/xact.c

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
*
1111
*
1212
* IDENTIFICATION
13-
* $PostgreSQL: pgsql/src/backend/access/transam/xact.c,v 1.213 2005/08/17 22:14:33 tgl Exp $
13+
* $PostgreSQL: pgsql/src/backend/access/transam/xact.c,v 1.214 2005/08/20 23:45:08 tgl Exp $
1414
*
1515
*-------------------------------------------------------------------------
1616
*/
@@ -129,12 +129,12 @@ typedef struct TransactionStateData
129129
typedef TransactionStateData *TransactionState;
130130

131131
/*
132-
* childXids is currently implemented as an integer List, relying on the
133-
* assumption that TransactionIds are no wider than int. We use these
132+
* childXids is currently implemented as an Oid List, relying on the
133+
* assumption that TransactionIds are no wider than Oid. We use these
134134
* macros to provide some isolation in case that changes in the future.
135135
*/
136-
#define lfirst_xid(lc) ((TransactionId) lfirst_int(lc))
137-
#define lappend_xid(list, datum) lappend_int(list, (int) (datum))
136+
#define lfirst_xid(lc) ((TransactionId) lfirst_oid(lc))
137+
#define lappend_xid(list, datum) lappend_oid(list, (Oid) (datum))
138138

139139
/*
140140
* CurrentTransactionState always points to the current transaction state

0 commit comments

Comments
 (0)