diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2018-03-25 14:54:16 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2018-03-25 14:54:16 -0400 |
| commit | 3a2cb59887421a04b5ee158580198d731d115c61 (patch) | |
| tree | 0c105b3df85de0e935581334c69c2fea56a2bb25 /src/backend/executor/execMain.c | |
| parent | ee4a2c4a0345f2589ce32b64493b1b14e87f0465 (diff) | |
| download | postgresql-3a2cb59887421a04b5ee158580198d731d115c61.tar.gz postgresql-3a2cb59887421a04b5ee158580198d731d115c61.zip | |
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.
Diffstat (limited to 'src/backend/executor/execMain.c')
| -rw-r--r-- | src/backend/executor/execMain.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/backend/executor/execMain.c b/src/backend/executor/execMain.c index 890067757c0..f47c691d12b 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 |