On Wed, May 20, 2020 at 01:22:22AM +0100, Ramsay Jones wrote:
> >
> > diff --git a/parse.c b/parse.c
> > index a29c67c8cf41..48494afc6f2c 100644
> > --- a/parse.c
> > +++ b/parse.c
> > @@ -2750,6 +2750,13 @@ static struct token *initializer_list(struct expression_list **list, struct toke
> > {
> > struct expression *expr;
> >
> > + // '{ 0 }' is equivalent to '{ }' unless wanting all possible
> > + // warnings about using '0' to initialize a null-pointer.
> > + if (!Wuniversal_initializer) {
> > + if (match_token_zero(token) && match_op(token->next, '}'))
> > + token = token->next;
> > + }
> > +
>
> Ha! This made me LOL! (see my patch below).
>
> So simple. (I did think, at first, that deleting the '0' token was
> not a good idea - then I realized that it's more like skipping/ignoring
> the token than deleting it.)
>
> I wish I had thought of it.
Well, it ended that it wasn't that smart after all because it
caused several regressions when used with scalars.
So, I finally had to do a sort of hybrid between your version
(for the parsing) and mine (dropping the '0' element from the list,
but now, later, at evaluation time).
-- Luc
