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 /doc/src | |
parent | 45b87cc57fe02f571de33ce20b4d7bec881a785a (diff) | |
download | postgresql-250e5bd7129d0b54594dc225f7380545189b70c1.tar.gz postgresql-250e5bd7129d0b54594dc225f7380545189b70c1.zip |
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 'doc/src')
0 files changed, 0 insertions, 0 deletions