diff options
| author | Andres Freund <andres@anarazel.de> | 2016-03-07 14:24:52 -0800 |
|---|---|---|
| committer | Andres Freund <andres@anarazel.de> | 2016-03-07 14:24:52 -0800 |
| commit | 250e5bd7129d0b54594dc225f7380545189b70c1 (patch) | |
| tree | 30eb398370d1067cfdc5e770d56c66a3aa846030 /src/backend/storage/file/reinit.c | |
| parent | 45b87cc57fe02f571de33ce20b4d7bec881a785a (diff) | |
Further improvements to c8f621c43.
Coverity and inspection for the issue addressed in fd45d16f found some
questionable code.
Specifically coverity noticed that the wrong length was added in
ReorderBufferSerializeChange() - without immediate negative consequences
as the variable isn't used afterwards. During code-review and testing I
noticed that a bit of space was wasted when allocating tuple bufs in
several places. Thirdly, the debug memset()s in
ReorderBufferGetTupleBuf() reduce the error checking valgrind can do.
Backpatch: 9.4, like c8f621c43.
Diffstat (limited to 'src/backend/storage/file/reinit.c')
0 files changed, 0 insertions, 0 deletions