Remove test for NULL node in ExecProcNode(). No place ever calls

ExecProcNode() with a NULL value, so the test couldn't do anything
for us except maybe mask bugs.  Removing it probably doesn't save
anything much either, but then again this is a hot-spot routine.

diff --git a/src/backend/executor/execProcnode.c b/src/backend/executor/execProcnode.c
index 71b2745..ff8caf1 100644 (file)
--- a/src/backend/executor/execProcnode.c
+++ b/src/backend/executor/execProcnode.c
@@ -12,7 +12,7 @@
  *
  *
  * IDENTIFICATION
- *       $PostgreSQL: pgsql/src/backend/executor/execProcnode.c,v 1.47 2005/03/25 21:57:58 tgl Exp $
+ *       $PostgreSQL: pgsql/src/backend/executor/execProcnode.c,v 1.48 2005/04/06 20:13:49 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -271,12 +271,6 @@ ExecProcNode(PlanState *node)
 
        CHECK_FOR_INTERRUPTS();
 
-       /*
-        * deal with NULL nodes..
-        */
-       if (node == NULL)
-               return NULL;
-
        if (node->chgParam != NULL) /* something changed */
                ExecReScan(node, NULL); /* let ReScan handle this */