From: "D'Arcy J.M. Cain" <darcy@druid.net>

Subject: [HACKERS] src.original/./backend/lib/fstack.c

Another change I suggested.  I bracket an unused function and add a
return to quiet the compiler.  In addition I added an internal
consistency check.

diff --git a/src/backend/optimizer/geqo/geqo_eval.c b/src/backend/optimizer/geqo/geqo_eval.c
index d1265f74344a6a72a2159d8c92ca36bbe9efe6be..4ff1b4748d52c4cec3ed5bf48c00119371f33782 100644 (file)
--- a/src/backend/optimizer/geqo/geqo_eval.c
+++ b/src/backend/optimizer/geqo/geqo_eval.c
@@ -5,7 +5,7 @@
  *
  * Copyright (c) 1994, Regents of the University of California
  *
- * $Id: geqo_eval.c,v 1.8 1997/04/09 08:31:29 scrappy Exp $
+ * $Id: geqo_eval.c,v 1.9 1997/06/06 00:38:17 scrappy Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -415,6 +415,7 @@ new_joininfo_list(List *joininfo_list, List *join_relids)
     return(current_joininfo_list);
 }
 
+#ifdef NOTUSED
 /*
  * add-new-joininfos--
  *    For each new join relation, create new joininfos that
@@ -636,6 +637,7 @@ nonoverlap_sets(List *s1, List *s2)
     }
     return(true);
 }
+#endif /* NOTUSED */
 
 /*
  * geqo_joinrel_size--
@@ -682,4 +684,6 @@ geqo_nth(int stop, List *rels)
                if (i == stop) return lfirst(r);
                i++;
                }
+       elog(WARN,"geqo_nth: Internal error - ran off end of list");
+       return NULL;    /* to keep compiler happy */
 }