I'm not opposed directly to this patch, but I'd like to keep the
        warnings in the WARNLIST that have been removed since they are a
        constant reminder of what is left to fix in the tree.

Regards
-steve

On Mon, 2009-03-30 at 20:28 +0200, Jim Meyering wrote:
> From: Jim Meyering <[email protected]>
> 
> ---
>  configure.ac |   65 ++++++++++++++++++++++++++++++++++++++++++++++++---------
>  1 files changed, 54 insertions(+), 11 deletions(-)
> 
> diff --git a/configure.ac b/configure.ac
> index cbba6ca..7cccba3 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -237,26 +237,69 @@ fi
>  EXTRA_WARNINGS=""
> 
>  WARNLIST="
> -     all
> -     missing-prototypes
> -     missing-declarations
> -     strict-prototypes
> -     declaration-after-statement
> -     pointer-arith
> -     write-strings
> -     cast-qual
>       cast-align
> -     bad-function-cast
> -     missing-format-attribute
>       format=2
> -     format-security
>       format-nonliteral
>       no-long-long
>       unsigned-char
>       gnu89-inline
>       no-strict-aliasing
> +        no-system-headers
>       "
> 
> +fix_soon="
> +  shadow
> +  unreachable-code
> +  old-style-definition
> +"
> +
> +too_distracting_for_now="
> +  undef
> +  cast-qual
> +  missing-declarations
> +  missing-prototypes
> +  redundant-decls
> +  strict-aliasing
> +  strict-prototypes
> +  missing-declarations
> +  inline
> +  switch-default
> +  switch-enum
> +"
> +
> +# don't use this: -Wdeclaration-after-statement
> +# do use this: -Wshadow
> +
> +all="
> +   all
> +   format-y2k
> +   format-nonliteral
> +   format-security
> +   init-self
> +   missing-include-dirs
> +   unused
> +   unknown-pragmas
> +   strict-overflow
> +   float-equal
> +   unsafe-loop-optimizations
> +   pointer-arith
> +   bad-function-cast
> +   write-strings
> +   logical-op
> +   aggregate-return
> +   missing-noreturn
> +   missing-format-attribute
> +   nested-externs
> +   invalid-pch
> +   long-long
> +   vla
> +   volatile-register-var
> +   disabled-optimization
> +   stack-protector
> +   overlength-strings
> +"
> +WARNLIST="$WARNLIST $all"
> +
>  for j in $WARNLIST; do
>       if cc_supports_flag -W$j; then
>               EXTRA_WARNINGS="$EXTRA_WARNINGS -W$j";

_______________________________________________
Openais mailing list
[email protected]
https://lists.linux-foundation.org/mailman/listinfo/openais

Reply via email to