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 1b98f26791a2036e1df69d92e38d89e4f628d437..d4a20eb3f67782726cfbba15a22057b40c375ef5 100644 (file)
--- a/src/backend/executor/execProcnode.c
+++ b/src/backend/executor/execProcnode.c
@@ -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 */