[prev in list] [next in list] [prev in thread] [next in thread] 

List:       wine-devel
Subject:    Re: configure.ac: remove an extraneous semicolon (Clang)
From:       Austin English <austinenglish () gmail ! com>
Date:       2012-01-31 7:32:26
Message-ID: CACC5Q1cc8DwY94sSTGfQ_ymRtOpVC8drcCBuX5ouAOOtncYbVw () mail ! gmail ! com
[Download RAW message or body]

On Mon, Jan 30, 2012 at 23:21, Ken Thomases <ken@codeweavers.com> wrote:
> 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.

You're right, misread the output from config.log, my mistake.

> 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.

Actually, it's mostly clean using Clang..

-- 
-Austin


[prev in list] [next in list] [prev in thread] [next in thread]