On Jan 30, 2012, at 10:09 PM, Austin English wrote:
> @@ -1723,7 +1723,7 @@ then
> saved_CFLAGS="$CFLAGS"
> CFLAGS="$CFLAGS -Wlogical-op -Werror"
> AC_CACHE_CHECK([for broken string.h that generates warnings with -Wlogical-op], ac_cv_c_logicalop_noisy,
> - AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[#include <string.h>]], [[char*f(const char *h,char n) {return strchr(h,n);}]])],[ac_cv_c_logicalop_noisy=no],[ac_cv_c_logicalop_noisy=yes]))
> + AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[#include <string.h>]], [[char*f(const char *h,char n) {return strchr(h,n)}]])],[ac_cv_c_logicalop_noisy=no],[ac_cv_c_logicalop_noisy=yes]))
> CFLAGS="$saved_CFLAGS"
> if test "$ac_cv_c_logicalop_noisy" = "no"
> then
That doesn't look extraneous to me.
Also, static analysis of the configure test programs seems like it would produce tons of noise. Those are often not sane programs by normal standards.
Regards,
Ken