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.

1 files changed, 9 insertions, 98 deletions

diff --git a/src/include/executor/execdebug.h b/src/include/executor/execdebug.h
index 2f5f55df28c..b909eb68442 100644
--- a/src/include/executor/execdebug.h
+++ b/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
@@ -38,58 +41,29 @@
  */
 
 /* ----------------
- *		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
  */
@@ -140,39 +114,6 @@ extern int	NIndexTupleInserted;
 #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,29 +121,15 @@ 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,29 +137,13 @@ 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
  * ----------------
  */