Remove CXT_printf/CXT1_printf macros. If anyone had found them to be of

any use in the past many years, we'd have made some effort to include
them in all executor node types; but in fact they were only in
nodeAppend.c and nodeIndexscan.c, up until I copied nodeIndexscan.c's
occurrence into the new bitmap node types.  Remove some other unused
macros in execdebug.h, too.  Some day the whole header probably ought to
go away in favor of better-designed facilities.

contrib/xml2/xpath.c

@@ -65,21 +65,21 @@ char	   *pgxml_errorMsg = NULL;		/* overall error message */
 static void *
 pgxml_palloc(size_t size)
 {
-/*	elog(DEBUG1,"Alloc %d in CMC %x",size,CurrentMemoryContext); */
+/*	elog(DEBUG1,"Alloc %d in CMC %p",size,CurrentMemoryContext); */
 	return palloc(size);
 }
 
 static void *
 pgxml_repalloc(void *ptr, size_t size)
 {
-/*	elog(DEBUG1,"ReAlloc in CMC %x",CurrentMemoryContext);*/
+/*	elog(DEBUG1,"ReAlloc in CMC %p",CurrentMemoryContext);*/
 	return repalloc(ptr, size);
 }
 
 static void
 pgxml_pfree(void *ptr)
 {
-/*	elog(DEBUG1,"Free in CMC %x",CurrentMemoryContext); */
+/*	elog(DEBUG1,"Free in CMC %p",CurrentMemoryContext); */
 	pfree(ptr);
 }
 

src/backend/executor/nodeAppend.c

@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *	  $PostgreSQL: pgsql/src/backend/executor/nodeAppend.c,v 1.68 2006/03/05 15:58:26 momjian Exp $
+ *	  $PostgreSQL: pgsql/src/backend/executor/nodeAppend.c,v 1.69 2006/05/23 15:21:51 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -151,8 +151,6 @@ ExecInitAppend(Append *node, EState *estate, int eflags)
 	/* check for unsupported flags */
 	Assert(!(eflags & EXEC_FLAG_MARK));
 
-	CXT1_printf("ExecInitAppend: context is %d\n", CurrentMemoryContext);
-
 	/*
 	 * Set up empty vector of subplan states
 	 */

src/backend/executor/nodeBitmapAnd.c

@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *	  $PostgreSQL: pgsql/src/backend/executor/nodeBitmapAnd.c,v 1.6 2006/03/05 15:58:26 momjian Exp $
+ *	  $PostgreSQL: pgsql/src/backend/executor/nodeBitmapAnd.c,v 1.7 2006/05/23 15:21:52 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -52,8 +52,6 @@ ExecInitBitmapAnd(BitmapAnd *node, EState *estate, int eflags)
 	/* check for unsupported flags */
 	Assert(!(eflags & (EXEC_FLAG_BACKWARD | EXEC_FLAG_MARK)));
 
-	CXT1_printf("ExecInitBitmapAnd: context is %d\n", CurrentMemoryContext);
-
 	/*
 	 * Set up empty vector of subplan states
 	 */

src/backend/executor/nodeBitmapHeapscan.c

@@ -21,7 +21,7 @@
  *
  *
  * IDENTIFICATION
- *	  $PostgreSQL: pgsql/src/backend/executor/nodeBitmapHeapscan.c,v 1.10 2006/03/05 15:58:26 momjian Exp $
+ *	  $PostgreSQL: pgsql/src/backend/executor/nodeBitmapHeapscan.c,v 1.11 2006/05/23 15:21:52 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -511,8 +511,6 @@ ExecInitBitmapHeapScan(BitmapHeapScan *node, EState *estate, int eflags)
 	ExecInitResultTupleSlot(estate, &scanstate->ss.ps);
 	ExecInitScanTupleSlot(estate, &scanstate->ss);
 
-	CXT1_printf("ExecInitBitmapHeapScan: context is %d\n", CurrentMemoryContext);
-
 	/*
 	 * open the base relation and acquire appropriate lock on it.
 	 */

src/backend/executor/nodeBitmapIndexscan.c

@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *	  $PostgreSQL: pgsql/src/backend/executor/nodeBitmapIndexscan.c,v 1.17 2006/03/05 15:58:26 momjian Exp $
+ *	  $PostgreSQL: pgsql/src/backend/executor/nodeBitmapIndexscan.c,v 1.18 2006/05/23 15:21:52 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -266,8 +266,6 @@ ExecInitBitmapIndexScan(BitmapIndexScan *node, EState *estate, int eflags)
 	 */
 	indexstate->biss_RuntimeKeysReady = false;
 
-	CXT1_printf("ExecInitBitmapIndexScan: context is %d\n", CurrentMemoryContext);
-
 	/*
 	 * build the index scan keys from the index qualification
 	 */

src/backend/executor/nodeBitmapOr.c

@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *	  $PostgreSQL: pgsql/src/backend/executor/nodeBitmapOr.c,v 1.5 2006/03/05 15:58:26 momjian Exp $
+ *	  $PostgreSQL: pgsql/src/backend/executor/nodeBitmapOr.c,v 1.6 2006/05/23 15:21:52 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -53,8 +53,6 @@ ExecInitBitmapOr(BitmapOr *node, EState *estate, int eflags)
 	/* check for unsupported flags */
 	Assert(!(eflags & (EXEC_FLAG_BACKWARD | EXEC_FLAG_MARK)));
 
-	CXT1_printf("ExecInitBitmapOr: context is %d\n", CurrentMemoryContext);
-
 	/*
 	 * Set up empty vector of subplan states
 	 */

src/backend/executor/nodeHash.c

@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *	  $PostgreSQL: pgsql/src/backend/executor/nodeHash.c,v 1.101 2006/03/05 15:58:26 momjian Exp $
+ *	  $PostgreSQL: pgsql/src/backend/executor/nodeHash.c,v 1.102 2006/05/23 15:21:52 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -123,8 +123,6 @@ ExecInitHash(Hash *node, EState *estate, int eflags)
 	/* check for unsupported flags */
 	Assert(!(eflags & (EXEC_FLAG_BACKWARD | EXEC_FLAG_MARK)));
 
-	SO_printf("ExecInitHash: initializing hash node\n");
-
 	/*
 	 * create state structure
 	 */

src/backend/executor/nodeIndexscan.c

@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *	  $PostgreSQL: pgsql/src/backend/executor/nodeIndexscan.c,v 1.112 2006/03/05 15:58:26 momjian Exp $
+ *	  $PostgreSQL: pgsql/src/backend/executor/nodeIndexscan.c,v 1.113 2006/05/23 15:21:52 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -528,8 +528,6 @@ ExecInitIndexScan(IndexScan *node, EState *estate, int eflags)
 	 */
 	indexstate->iss_RuntimeKeysReady = false;
 
-	CXT1_printf("ExecInitIndexScan: context is %d\n", CurrentMemoryContext);
-
 	/*
 	 * build the index scan keys from the index qualification
 	 */

src/include/executor/execdebug.h

@@ -3,11 +3,14 @@
  * execdebug.h
  *	  #defines governing debugging behaviour in the executor
  *
+ * XXX this is all pretty old and crufty.  Newer code tends to use elog()
+ * for debug printouts, because that's more flexible than printf().
+ *
  *
  * Portions Copyright (c) 1996-2006, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $PostgreSQL: pgsql/src/include/executor/execdebug.h,v 1.29 2006/03/05 15:58:56 momjian Exp $
+ * $PostgreSQL: pgsql/src/include/executor/execdebug.h,v 1.30 2006/05/23 15:21:52 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -29,7 +32,7 @@
 
 /* ----------------
  *		EXEC_TUPLECOUNT is a #define which causes the
- *		executor keep track of tuple counts.  This might be
+ *		executor to keep track of tuple counts.  This might be
  *		causing some problems with the decstation stuff so
  *		you might want to undefine this if you are doing work
  *		on the decs  - cim 10/20/89
@@ -37,59 +40,30 @@
 #undef EXEC_TUPLECOUNT
  */
 
-/* ----------------
- *		EXEC_CONTEXTDEBUG turns on the printing of debugging information
- *		by CXT_printf() calls regarding which memory context is the
- *		CurrentMemoryContext for palloc() calls.
- * ----------------
-#undef EXEC_CONTEXTDEBUG
- */
-
-/* ----------------
- *		EXEC_UTILSDEBUG is a flag which turns on debugging of the
- *		executor utilities by EU_printf() in eutils.c
- * ----------------
-#undef EXEC_UTILSDEBUG
- */
-
 /* ----------------
  *		EXEC_NESTLOOPDEBUG is a flag which turns on debugging of the
- *		nest loop node by NL_printf() and ENL_printf() in nestloop.c
+ *		nest loop node by NL_printf() and ENL_printf() in nodeNestloop.c
  * ----------------
 #undef EXEC_NESTLOOPDEBUG
  */
 
-/* ----------------
- *		EXEC_PROCDEBUG is a flag which turns on debugging of
- *		ExecProcNode() by PN_printf() in procnode.c
- * ----------------
-#undef EXEC_PROCDEBUG
- */
-
 /* ----------------
  *		EXEC_EVALDEBUG is a flag which turns on debugging of
- *		ExecEval and ExecTargetList() stuff by EV_printf() in qual.c
+ *		ExecEval and ExecTargetList() stuff by EV_printf() in execQual.c
  * ----------------
 #undef EXEC_EVALDEBUG
  */
 
-/* ----------------
- *		EXEC_SCANDEBUG is a flag which turns on debugging of
- *		the ExecSeqScan() stuff by S_printf() in seqscan.c
- * ----------------
-#undef EXEC_SCANDEBUG
- */
-
 /* ----------------
  *		EXEC_SORTDEBUG is a flag which turns on debugging of
- *		the ExecSort() stuff by SO_printf() in sort.c
+ *		the ExecSort() stuff by SO_printf() in nodeSort.c
  * ----------------
 #undef EXEC_SORTDEBUG
  */
 
 /* ----------------
  *		EXEC_MERGEJOINDEBUG is a flag which turns on debugging of
- *		the ExecMergeJoin() stuff by MJ_printf() in mergejoin.c
+ *		the ExecMergeJoin() stuff by MJ_printf() in nodeMergejoin.c
  * ----------------
 #undef EXEC_MERGEJOINDEBUG
  */
@@ -139,39 +113,6 @@ extern int	NIndexTupleInserted;
 #define IncrIndexInserted()		(void)(0)
 #endif   /* EXEC_TUPLECOUNT */
 
-/* ----------------
- *		memory context debugging defines
- * ----------------
- */
-#ifdef EXEC_CONTEXTDEBUG
-#define CXT_printf(s)					printf(s)
-#define CXT1_printf(s, a)				printf(s, a)
-#else
-#define CXT_printf(s)
-#define CXT1_printf(s, a)
-#endif   /* EXEC_CONTEXTDEBUG */
-
-/* ----------------
- *		eutils debugging defines
- * ----------------
- */
-#ifdef EXEC_UTILSDEBUG
-#define EU_nodeDisplay(l)				nodeDisplay(l)
-#define EU_printf(s)					printf(s)
-#define EU1_printf(s, a)				printf(s, a)
-#define EU2_printf(s, a, b)				printf(s, a, b)
-#define EU3_printf(s, a, b, c)			printf(s, a, b, c)
-#define EU4_printf(s, a, b, c, d)		printf(s, a, b, c, d)
-#else
-#define EU_nodeDisplay(l)
-#define EU_printf(s)
-#define EU1_printf(s, a)
-#define EU2_printf(s, a, b)
-#define EU3_printf(s, a, b, c)
-#define EU4_printf(s, a, b, c, d)
-#endif   /* EXEC_UTILSDEBUG */
-
-
 /* ----------------
  *		nest loop debugging defines
  * ----------------
@@ -180,28 +121,14 @@ extern int	NIndexTupleInserted;
 #define NL_nodeDisplay(l)				nodeDisplay(l)
 #define NL_printf(s)					printf(s)
 #define NL1_printf(s, a)				printf(s, a)
-#define NL4_printf(s, a, b, c, d)		printf(s, a, b, c, d)
 #define ENL1_printf(message)			printf("ExecNestLoop: %s\n", message)
 #else
 #define NL_nodeDisplay(l)
 #define NL_printf(s)
 #define NL1_printf(s, a)
-#define NL4_printf(s, a, b, c, d)
 #define ENL1_printf(message)
 #endif   /* EXEC_NESTLOOPDEBUG */
 
-/* ----------------
- *		proc node debugging defines
- * ----------------
- */
-#ifdef EXEC_PROCDEBUG
-#define PN_printf(s)					printf(s)
-#define PN1_printf(s, p)				printf(s, p)
-#else
-#define PN_printf(s)
-#define PN1_printf(s, p)
-#endif   /* EXEC_PROCDEBUG */
-
 /* ----------------
  *		exec eval / target list debugging defines
  * ----------------
@@ -210,28 +137,12 @@ extern int	NIndexTupleInserted;
 #define EV_nodeDisplay(l)				nodeDisplay(l)
 #define EV_printf(s)					printf(s)
 #define EV1_printf(s, a)				printf(s, a)
-#define EV5_printf(s, a, b, c, d, e)	printf(s, a, b, c, d, e)
 #else
 #define EV_nodeDisplay(l)
 #define EV_printf(s)
 #define EV1_printf(s, a)
-#define EV5_printf(s, a, b, c, d, e)
 #endif   /* EXEC_EVALDEBUG */
 
-/* ----------------
- *		scan debugging defines
- * ----------------
- */
-#ifdef EXEC_SCANDEBUG
-#define S_nodeDisplay(l)				nodeDisplay(l)
-#define S_printf(s)						printf(s)
-#define S1_printf(s, p)					printf(s, p)
-#else
-#define S_nodeDisplay(l)
-#define S_printf(s)
-#define S1_printf(s, p)
-#endif   /* EXEC_SCANDEBUG */
-
 /* ----------------
  *		sort node debugging defines
  * ----------------