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.
*** ../src.original/./backend/optimizer/geqo/geqo_eval.c Mon Apr 14 16:33:25 1997
- --- ./backend/optimizer/geqo/geqo_eval.c Fri Apr 18 09:11:20 1997
***************
*** 415,420 ****
- --- 409,415 ----
return(current_joininfo_list);
}
+ #ifdef NOTUSED
/*
* add-new-joininfos--
* For each new join relation, create new joininfos that
***************
*** 636,641 ****
- --- 631,637 ----
}
return(true);
}
+ #endif /* NOTUSED */
/*
* geqo_joinrel_size--
***************
*** 682,685 ****
- --- 678,683 ----
if (i == stop) return lfirst(r);
i++;
}
+ elog(WARN,"geqo_nth: Internal error - ran off end of list");
+ return NULL; /* to keep compiler happy */
}
- --
D'Arcy J.M. Cain | Democracy is three wolves
darcy@{druid.net|vex.net} | and a sheep voting on
+1 416 424 2871 (DoD#0082) (eNTP) | what's for dinner.
-- http://www.druid.net/darcy --
------------------------------
