On 05/19/2017 02:31 AM, Philipp Stephani wrote:
> Apparently on some systems mbstate_t is a nested struct, and the compiler
> warns about missing braces. Note that memset to initialize a mbstate_t is
> explicitly recommended in the libc manual:
Yes, of course memset works (which is all that the glibc manual really says).
It's just that it's verbose and the verbosity isn't needed.
I guess the problem here is that clang's -Wmissing-braces option generates false
alarms. The Clang folks may fix that someday. In the meantime I installed the
first attached patch, to turn off that option for Clang.
This apparently hasn't worked, at least I still get the warning, and there's no -Wno-missing-braces in WARN_CFLAGS.
For Clang, it'd probably be better to use -Weverything and then disable individual warnings with -Wno... instead of checking for the existence of every warning flag. That would make configure runs faster and the command line shorter.