icc adjustments

diff --git a/acinclude.m4 b/acinclude.m4
index 61f6ef96f31b2b583aa6295816f15d41ee566cf9..a1278428524a8e15f95eded0e4130bcecb841710 100644 (file)
--- a/acinclude.m4
+++ b/acinclude.m4
@@ -3231,6 +3231,7 @@ AC_DEFUN([CURL_CC_DEBUG_OPTS],
          dnl   which we know and do on purpose.
 
          WARN="-wd279,269,981,1418,1419"
+         WARN=""
 
          if test "$gccnum" -gt "600"; then
             dnl icc 6.0 and older doesn't have the -Wall flag

diff --git a/ares/acinclude.m4 b/ares/acinclude.m4
index 707ba7ee1547e4e7a889f0a3399882f040ea01ef..9cb54ae54b4d1e64942f3c975b19848c3b978aa8 100644 (file)
--- a/ares/acinclude.m4
+++ b/ares/acinclude.m4
@@ -1892,6 +1892,7 @@ AC_DEFUN([CURL_CC_DEBUG_OPTS],
          dnl   which we know and do on purpose.
 
          WARN="-wd279,269,981,1418,1419"
+         WARN=""
 
          if test "$gccnum" -gt "600"; then
             dnl icc 6.0 and older doesn't have the -Wall flag

diff --git a/ares/configure.ac b/ares/configure.ac
index fe8a7a7f07441de1ad6f10afcd07ae621e715131..9c5c426a0d24707dd38f52d87e108dfbd4cfa47c 100644 (file)
--- a/ares/configure.ac
+++ b/ares/configure.ac
@@ -167,13 +167,6 @@ esac
 AC_MSG_RESULT($need_no_undefined)
 AM_CONDITIONAL(NO_UNDEFINED, test x$need_no_undefined = xyes)
 
-dnl **********************************************************************
-dnl check if this is the Intel ICC compiler, and if so make it stricter
-dnl (convert warning 147 into an error) so that it properly can detect the
-dnl gethostbyname_r() version
-dnl **********************************************************************
-CURL_DETECT_ICC([CFLAGS="$CFLAGS -we 147"])
-
 dnl **********************************************************************
 dnl platform/compiler/architecture specific checks/flags
 dnl **********************************************************************
@@ -239,6 +232,12 @@ if test "$curl_cv_have_def___INTEL_COMPILER" = "yes"; then
   CFLAGS="$CFLAGS -std=c89"
   dnl Disable use of ANSI C aliasing rules in optimizations
   CFLAGS="$CFLAGS -no-ansi-alias"
+  dnl Change some warnings into errors
+  dnl #140: too many arguments in function call
+  dnl #147: declaration is incompatible with 'previous one'
+  dnl #165: too few arguments in function call
+  dnl #266: function declared implicitly
+  CFLAGS="$CFLAGS -we 140,147,165,266"
 else
   AC_MSG_RESULT([no])
 fi

diff --git a/configure.ac b/configure.ac
index 71464f43f06d5a0079c317fdfeb118d97d687378..cad4a09d578ad115b176a10584dd11d2c3f08438 100644 (file)
--- a/configure.ac
+++ b/configure.ac
@@ -306,6 +306,12 @@ if test "$curl_cv_have_def___INTEL_COMPILER" = "yes"; then
   CFLAGS="$CFLAGS -std=c89"
   dnl Disable use of ANSI C aliasing rules in optimizations
   CFLAGS="$CFLAGS -no-ansi-alias"
+  dnl Change some warnings into errors
+  dnl #140: too many arguments in function call
+  dnl #147: declaration is incompatible with 'previous one'
+  dnl #165: too few arguments in function call
+  dnl #266: function declared implicitly
+  CFLAGS="$CFLAGS -we 140,147,165,266"
 else
   AC_MSG_RESULT([no])
 fi
@@ -541,13 +547,6 @@ AC_HELP_STRING([--disable-manual],[Disable built-in manual]),
 dnl The actual use of the USE_MANUAL variable is done much later in this
 dnl script to allow other actions to disable it as well.
 
-dnl **********************************************************************
-dnl check if this is the Intel ICC compiler, and if so make it stricter
-dnl (convert warning 147 into an error) so that it properly can detect the
-dnl gethostbyname_r() version
-dnl **********************************************************************
-CURL_DETECT_ICC([CFLAGS="$CFLAGS -we 147"])
-
 dnl **********************************************************************
 dnl Checks for libraries.
 dnl **********************************************************************