Re: [PATCH] maint: enable -Wmissing-field-initializers

[Pádraig Brady]

On 27/04/11 21:14, Jim Meyering wrote:
> Pádraig Brady wrote:
>> I'll apply the attached after the pending release.
>>
>> Subject: [PATCH] maint: enable -Wmissing-field-initializers
>>
>> * configure.ac: Enable -Wmissing-field-initializers for src/
>> but leave disabled for lib/.
>> * src/system.h: Enable the static variable hack only for
>> non gcc compilers and versions of gcc older than 4.7.
>> Starting with that version, the warning is no longer issued for
>> variables initialized with a single zero value.
>> Rename DECLARE_ZEROED_AGGREGATE to the more concise
>> and accurate DECLARE_ZEROED_AUTO.
>> * src/ls.c: Use the renamed macro.
>> * src/pathchk.c: Likewise.
>> * src/shred.c: Likewise.
>> * src/stty.c: Likewise.
>> * src/wc.c: Likewise.
> 
> Nice.  You beat me to it.
> Thanks.

I was thinking actually of changing this, and doing this solely within
manywarnings.m4 in gnulib. I.E. apply -Wno-missing-field-initializers
conditionally there (depending on result of compiling {0,}).

Then we could remove all this stuff from coreutils,
including the DECLARE_ZEROED_AGGREGATE macro.