From a6bbfedcf7a899c91182f373f93f15f5af247647 Mon Sep 17 00:00:00 2001
From: Tom Lane <tgl@sss.pgh.pa.us>
Date: Wed, 6 Apr 2005 20:13:49 +0000
Subject: [PATCH] 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.

---
 src/backend/executor/execProcnode.c | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

diff --git a/src/backend/executor/execProcnode.c b/src/backend/executor/execProcnode.c
index 71b274530cf..ff8caf16f01 100644
--- 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 */
 
-- 
GitLab