On Fri, May 05, 2023 at 02:13:28PM +0000, gkokola...@pm.me wrote: > Good point. I thought about it before submitting the patch. I > concluded that given the complexity and operations involved in > LZ4Stream_read_internal() and the rest of t he pg_dump/pg_restore > code, the memset() call will be negligible. However from the > readability point of view, the function is a bit cleaner with the > memset(). > > I will not object to any suggestion though, as this is a very > trivial point. Please find attached a v2 of the patch following the > suggested approach.
Please note that an open item has been added for this stuff. -- Michael
signature.asc
Description: PGP signature