Re: [PATCH] warnings: Add gl_WARN_COMPLEMENT and gl_WARN_SUPPORTED.

[Eric Blake]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

According to Bruno Haible on 12/8/2008 4:07 PM:
> Before doing that, I'd like to see the results that Simon, Jim, Eric, and
> others get on reasonably large code bases. My list of warnings to avoid
> is certainly biased. What we document in the manual should not be biased.

The master branch of m4.git tries these flags:

    CFLAGS="$CFLAGS -Wall"
    WMORE="-Wshadow -Wpointer-arith -Wcast-align -Wnested-externs"
    WMORE="$WMORE -Wmissing-prototypes -Wmissing-declarations -Winline"

but some of them fail while compiling libtool, so I haven't really tested
many of these optional warning flags.  -Wshadow is a policy flag (it can
catch bugs, but more often catches unwanted warnings about system headers
interfering with the choice of local variable name), -Wpointer-arith is
real (not all compilers handle math on void*), -Wmissing-prototypes and
- -Wmissing-declarations appear to be policy flags (I can't think of any
bugs ever caught in my use of them), and I'm not too sure of the value of
the others.  I'm probably going to change m4's ltdl/m4/debug.m4 file to
use the new manywarnings module.

- --
Don't work too hard, make some time for fun as well!

Eric Blake             address@hidden
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (Cygwin)
Comment: Public key at home.comcast.net/~ericblake/eblake.gpg
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iEYEARECAAYFAkk+brgACgkQ84KuGfSFAYDNTACeP3w59WxoSSCd1jUCzkPLGeNO
eIoAoLYwzb91gSR3zCuyhUGTL76jf8bt
=a9J2
-----END PGP SIGNATURE-----