Re: additional GCC warning flags - Mailing list pgsql-patches

From Tom Lane
Subject Re: additional GCC warning flags
Date
Msg-id 26826.1098119970@sss.pgh.pa.us
Whole thread Raw
In response to additional GCC warning flags  (Neil Conway <neilc@samurai.com>)
Responses Re: additional GCC warning flags
Re: additional GCC warning flags
List pgsql-patches
Neil Conway <neilc@samurai.com> writes:
> This patch updates configure.in and configure (I re-ran autoconf 2.53).
> It doesn't introduce any additional warning messages locally (Linux, GCC
> 3.4), but might do so on some platforms (the code in src/port/ is likely
> to trigger some warnings, I think). Because of that, I think it is
> probably best to save this for 8.1

As long as it can only introduce warnings and not errors, I think it is
fine to apply now.  It might help us flush out some portability issues
--- we are just coming up on the stage of officially calling for port
reports for 8.0, so it seems like now is a good time to put it in.

One thing I'd suggest is that the "if test "$GCC" = yes; then"
conditional go outside the PGAC macros, ie

    if test "$GCC" = yes; then
        PGAC_PROG_CC_WARN_DECL_AFTER_STMT
        PGAC_PROG_CC_WARN_OLD_STYLE_DEFN
        PGAC_PROG_CC_WARN_ENDIF_LABELS
    fi

as otherwise people with non-gcc compilers will think configure is
pretty stupid even to be making the tests.

Also, how about removing the conditional CFLAGS hacking from
Makefile.global.in entirely, and change the above to

    if test "$GCC" = yes; then
        CFLAGS="$CFLAGS  -Wall -Wmissing-prototypes"
        PGAC_PROG_CC_WARN_DECL_AFTER_STMT
        PGAC_PROG_CC_WARN_OLD_STYLE_DEFN
        PGAC_PROG_CC_WARN_ENDIF_LABELS
    fi

which localizes the logic a tad more, and keeps the CFLAGS entries in a
rational-looking order.

            regards, tom lane

pgsql-patches by date:

Previous
From: Tom Lane
Date:
Subject: Re: pg_regress --temp-keep
Next
From: Bruce Momjian
Date:
Subject: Re: additional GCC warning flags