Remove some dead code and obsolete, misleading comments.

diff --git a/src/include/executor/execdebug.h b/src/include/executor/execdebug.h
index 67f6ca7b4cc6afea54d5d39df3602955a3d4c6ac..2ce7771b6122353399036adfbc51dcd2e972f313 100644 (file)
--- a/src/include/executor/execdebug.h
+++ b/src/include/executor/execdebug.h
@@ -7,7 +7,7 @@
  * Portions Copyright (c) 1996-2001, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $Id: execdebug.h,v 1.15 2001/01/24 19:43:23 momjian Exp $
+ * $Id: execdebug.h,v 1.16 2001/09/20 21:07:57 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -27,14 +27,6 @@
  * ----------------------------------------------------------------
  */
 
-/* ----------------
- *             EXEC_DEBUGSTORETUP is for tuple table debugging - this
- *             will print a message every time we call ExecStoreTuple.
- *             -cim 3/20/91
- * ----------------
-#undef EXEC_DEBUGSTORETUP
- */
-
 /* ----------------
  *             EXEC_TUPLECOUNT is a #define which causes the
  *             executor keep track of tuple counts.  This might be
@@ -45,13 +37,6 @@
 #undef EXEC_TUPLECOUNT
  */
 
-/* ----------------
- *             EXEC_SHOWBUFSTATS controls whether or not buffer statistics
- *             are shown for each query.  -cim 2/9/89
- * ----------------
-#undef EXEC_SHOWBUFSTATS
- */
-
 /* ----------------
  *             EXEC_CONTEXTDEBUG turns on the printing of debugging information
  *             by CXT_printf() calls regarding which memory context is the
@@ -60,25 +45,6 @@
 #undef EXEC_CONTEXTDEBUG
  */
 
-/* ----------------
- *             EXEC_RETURNSIZE is a compile flag governing the
- *             behaviour of lispFmgr..  See ExecMakeFunctionResult().
- *             Undefining this avoids a problem in the system cache.
- *
- *             Note: undefining this means that there is incorrect
- *                       information in the const nodes corresponding
- *                       to function (or operator) results.  The thing is,
- *                       99% of the time this is fine because when you do
- *                       something like x = emp.sal + 1, you already know
- *                       the type and size of x so the fact that + didn't
- *                       return the correct size doesn't matter.
- *                       With variable length stuff the size is stored in
- *                       the first few bytes of the data so again, it's
- *                       not likely to matter.
- * ----------------
-#undef EXEC_RETURNSIZE
- */
-
 /* ----------------
  *             EXEC_UTILSDEBUG is a flag which turns on debugging of the
  *             executor utilities by EU_printf() in eutils.c
@@ -128,30 +94,6 @@
 #undef EXEC_MERGEJOINDEBUG
  */
 
-/* ----------------
- *             EXEC_DEBUGINTERACTIVE is a flag which enables the
- *             user to issue "DEBUG" commands from an interactive
- *             backend.
- * ----------------
-#undef EXEC_DEBUGINTERACTIVE
- */
-
-/* ----------------
- *             EXEC_DEBUGVARIABLEFILE is string, which if defined will
- *             be loaded when the executor is initialized.  If this
- *             string is not defined then nothing will be loaded..
- *
- *             Example:
- *
- * #define EXEC_DEBUGVARIABLEFILE "/a/postgres/cimarron/.pg_debugvars"
- #
- *             Note: since these variables are read at execution time,
- *             they can't affect the first query.. this hack should be
- *             replaced by something better sometime. -cim 11/2/89
- * ----------------
-#undef EXEC_DEBUGVARIABLEFILE
- */
-
 /* ----------------------------------------------------------------
  *             #defines controlled by above definitions
  *