Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Commit 3a2cb59

Browse files
committed
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 cc415a5.
1 parent ee4a2c4 commit 3a2cb59

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

src/backend/executor/execMain.c

+1-3
Original file line numberDiff line numberDiff line change
@@ -249,9 +249,7 @@ standard_ExecutorStart(QueryDesc *queryDesc, int eflags)
249249
estate->es_crosscheck_snapshot = RegisterSnapshot(queryDesc->crosscheck_snapshot);
250250
estate->es_top_eflags = eflags;
251251
estate->es_instrument = queryDesc->instrument_options;
252-
253-
if (queryDesc->plannedstmt)
254-
estate->es_jit_flags = queryDesc->plannedstmt->jitFlags;
252+
estate->es_jit_flags = queryDesc->plannedstmt->jitFlags;
255253

256254
/*
257255
* Set up an AFTER-trigger statement context, unless told not to, or

0 commit comments

Comments
 (0)