Hello,

There are warnings with -Wmissing-field-initializers for info/variables.c
VARIABLE_ALIST info_variables where the where_set field is never
explicitely set.  This code is from Gavin, and is by design, as there
is this comment:
/* Note that the 'where_set' field of each element in the array is
   not given and defaults to 0. */

I would have put 0 there myself if I had written that code, but omitting
fields seems to me to be also ok if it is documented as it is here.
However, the warning is also interesting to set as in other cases the
missing fields may be an overlook, and having false positives is a pain.

Should I add the 0 and remove the comments or leave it as is?

-- 
Pat

Reply via email to