From f39f6b500f69074de6a9cbbdea90104dfd1c685d Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Sat, 20 Aug 2005 23:45:08 +0000 Subject: [PATCH] Seems that the childXids list would be better based on Oid lists than integer lists. --- src/backend/access/transam/xact.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/backend/access/transam/xact.c b/src/backend/access/transam/xact.c index 737d1f3252..eabcb117cc 100644 --- a/src/backend/access/transam/xact.c +++ b/src/backend/access/transam/xact.c @@ -10,7 +10,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/access/transam/xact.c,v 1.213 2005/08/17 22:14:33 tgl Exp $ + * $PostgreSQL: pgsql/src/backend/access/transam/xact.c,v 1.214 2005/08/20 23:45:08 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -129,12 +129,12 @@ typedef struct TransactionStateData typedef TransactionStateData *TransactionState; /* - * childXids is currently implemented as an integer List, relying on the - * assumption that TransactionIds are no wider than int. We use these + * childXids is currently implemented as an Oid List, relying on the + * assumption that TransactionIds are no wider than Oid. We use these * macros to provide some isolation in case that changes in the future. */ -#define lfirst_xid(lc) ((TransactionId) lfirst_int(lc)) -#define lappend_xid(list, datum) lappend_int(list, (int) (datum)) +#define lfirst_xid(lc) ((TransactionId) lfirst_oid(lc)) +#define lappend_xid(list, datum) lappend_oid(list, (Oid) (datum)) /* * CurrentTransactionState always points to the current transaction state