-----Original Message----- From: Tom Lane <t...@sss.pgh.pa.us> Sent: Thursday, 3 October 2019 1:46 AM
> Right. I think that in general it's bad practice for an initializer to not
> specify all fields/elements of the target.
> It is okay in the specific case that we're substituting for a memset(..., 0,
> ...).
> Perhaps we could make this explicit by using a coding style like
>
>/* in c.h or some such place: */
>#define INIT_ALL_ZEROES {0}
>
>/* in code: */
> Datum values[N] = INIT_ALL_ZEROES;
The patch has been updated per your suggestion. Now using macros for these
partial initialisers.
Please see attachment.
Kind Regards
---
Peter Smith
Fujitsu Australia
c99_init_nulls_3.patch
Description: c99_init_nulls_3.patch
