NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commita6bbfed

committed

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

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

1 parentad161bc commita6bbfedCopy full SHA for a6bbfed

File tree

1 file changed

-7

lines changed

src/backend/executor
- execProcnode.c

1 file changed

-7

lines changed

`‎src/backend/executor/execProcnode.c`

Lines changed: 1 addition & 7 deletions

Original file line number	Diff line number	Diff line change
`@@ -12,7 +12,7 @@`
`12`	`12`	`*`
`13`	`13`	`*`
`14`	`14`	`* IDENTIFICATION`
`15`		`- * $PostgreSQL: pgsql/src/backend/executor/execProcnode.c,v 1.47 2005/03/25 21:57:58 tgl Exp $`
	`15`	`+ * $PostgreSQL: pgsql/src/backend/executor/execProcnode.c,v 1.48 2005/04/06 20:13:49 tgl Exp $`
`16`	`16`	`*`
`17`	`17`	`*-------------------------------------------------------------------------`
`18`	`18`	`*/`
`@@ -271,12 +271,6 @@ ExecProcNode(PlanState *node)`
`271`	`271`
`272`	`272`	`CHECK_FOR_INTERRUPTS();`
`273`	`273`
`274`		`-/*`
`275`		`- * deal with NULL nodes..`
`276`		`- */`
`277`		`-if (node==NULL)`
`278`		`-returnNULL;`
`279`		`-`
`280`	`274`	`if (node->chgParam!=NULL)/* something changed */`
`281`	`275`	`ExecReScan(node,NULL);/* let ReScan handle this */`
`282`	`276`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commita6bbfed

File tree

1 file changed

1 file changed

`‎src/backend/executor/execProcnode.c`

0 commit comments