From 3a2cb59887421a04b5ee158580198d731d115c61 Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Sun, 25 Mar 2018 14:54:16 -0400 Subject: [PATCH] Remove useless if-test. Coverity complained that this check is pointless, and it's right. There is no case where we'd call ExecutorStart with a null plannedstmt, and if we did, it'd have crashed before here. Thinko in commit cc415a56d. --- src/backend/executor/execMain.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/src/backend/executor/execMain.c b/src/backend/executor/execMain.c index 890067757c..f47c691d12 100644 --- a/src/backend/executor/execMain.c +++ b/src/backend/executor/execMain.c @@ -249,9 +249,7 @@ standard_ExecutorStart(QueryDesc *queryDesc, int eflags) estate->es_crosscheck_snapshot = RegisterSnapshot(queryDesc->crosscheck_snapshot); estate->es_top_eflags = eflags; estate->es_instrument = queryDesc->instrument_options; - - if (queryDesc->plannedstmt) - estate->es_jit_flags = queryDesc->plannedstmt->jitFlags; + estate->es_jit_flags = queryDesc->plannedstmt->jitFlags; /* * Set up an AFTER-trigger statement context, unless told not to, or