Seems that the childXids list would be better based on Oid lists than

integer lists.

diff --git a/src/backend/access/transam/xact.c b/src/backend/access/transam/xact.c
index 737d1f3252a98c2d6b0003f77e6b06511cb40a0f..eabcb117cc58cb8bea27e7fe7a04d371ea125316 100644 (file)
--- 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