diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2018-02-14 16:06:49 -0500 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2018-02-14 16:06:49 -0500 |
| commit | 9a725f7b5cb7e8c8894ef121b49eff9c265245c8 (patch) | |
| tree | 37e438b1542f8c1e1f57d78fa5cee47cd35357ce /src/backend/utils | |
| parent | 0c62356cc8777961221a643fa77f62e1c7361085 (diff) | |
Silence assorted "variable may be used uninitialized" warnings.
All of these are false positives, but in each case a fair amount of
analysis is needed to see that, and it's not too surprising that not all
compilers are smart enough. (In particular, in the logtape.c case, a
compiler lacking the knowledge provided by the Assert would almost surely
complain, so that this warning will be seen in any non-assert build.)
Some of these are of long standing while others are pretty recent,
but it only seems worth fixing them in HEAD.
Jaime Casanova, tweaked a bit by me
Discussion: https://postgr.es/m/CAJGNTeMcYAMJdPAom52dppLMtF-UnEZi0dooj==75OEv1EoBZA@mail.gmail.com
Diffstat (limited to 'src/backend/utils')
| -rw-r--r-- | src/backend/utils/sort/logtape.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/backend/utils/sort/logtape.c b/src/backend/utils/sort/logtape.c index 66bfcced8d7..d6794bf3de1 100644 --- a/src/backend/utils/sort/logtape.c +++ b/src/backend/utils/sort/logtape.c @@ -411,7 +411,7 @@ ltsConcatWorkerTapes(LogicalTapeSet *lts, TapeShare *shared, SharedFileSet *fileset) { LogicalTape *lt = NULL; - long tapeblocks; + long tapeblocks = 0L; long nphysicalblocks = 0L; int i; |