diff options
| author | Andres Freund <andres@anarazel.de> | 2023-01-20 21:16:47 -0800 |
|---|---|---|
| committer | Andres Freund <andres@anarazel.de> | 2023-01-20 21:16:47 -0800 |
| commit | 25b2aba0c3a501e2af9579f1b629f155544935db (patch) | |
| tree | 563063c0c002285d51f100d027683afaf50169fe /src/bin/psql/common.c | |
| parent | 5d29d525ffe028fdf6b2d3ff7502243e56c6c79a (diff) | |
| download | postgresql-25b2aba0c3a501e2af9579f1b629f155544935db.tar.gz postgresql-25b2aba0c3a501e2af9579f1b629f155544935db.zip | |
Zero initialize uses of instr_time about to trigger compiler warnings
These are all not necessary from a correctness POV. However, in the near
future instr_time will be simplified to an int64, at which point gcc would
otherwise start to warn about the changed places.
Reviewed-by: Tom Lane <tgl@sss.pgh.pa.us>
Discussion: https://postgr.es/m/20230116023639.rn36vf6ajqmfciua@awork3.anarazel.de
Diffstat (limited to 'src/bin/psql/common.c')
| -rw-r--r-- | src/bin/psql/common.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/src/bin/psql/common.c b/src/bin/psql/common.c index 00627830c47..f907f5d4e8d 100644 --- a/src/bin/psql/common.c +++ b/src/bin/psql/common.c @@ -1276,6 +1276,8 @@ DescribeQuery(const char *query, double *elapsed_msec) if (timing) INSTR_TIME_SET_CURRENT(before); + else + INSTR_TIME_SET_ZERO(before); /* * To parse the query but not execute it, we prepare it, using the unnamed @@ -1406,6 +1408,8 @@ ExecQueryAndProcessResults(const char *query, if (timing) INSTR_TIME_SET_CURRENT(before); + else + INSTR_TIME_SET_ZERO(before); if (pset.bind_flag) success = PQsendQueryParams(pset.db, query, pset.bind_nparams, NULL, (const char * const *) pset.bind_params, NULL, NULL, 0); @@ -1702,6 +1706,8 @@ ExecQueryUsingCursor(const char *query, double *elapsed_msec) if (timing) INSTR_TIME_SET_CURRENT(before); + else + INSTR_TIME_SET_ZERO(before); /* if we're not in a transaction, start one */ if (PQtransactionStatus(pset.db) == PQTRANS_IDLE) |