Browse Source

[3213] Revert "[mavericks] suppress a spurious clang warning in libdns++"

This reverts commit f2f02dba84b5c12314f6018294423ebc45c4127e.
other commit reportedly addressed the issue that this hack tries to solve.
JINMEI Tatuya 11 years ago
parent
commit
89e9030039
2 changed files with 1 additions and 13 deletions
  1. 0 6
      configure.ac
  2. 1 7
      src/lib/dns/Makefile.am

+ 0 - 6
configure.ac

@@ -138,12 +138,6 @@ BIND10_CXX_TRY_FLAG([-Wno-missing-field-initializers],
 	[WARNING_NO_MISSING_FIELD_INITIALIZERS_CFLAG="-Wno-missing-field-initializers"])
 AC_SUBST(WARNING_NO_MISSING_FIELD_INITIALIZERS_CFLAG)
 
-# clang's -Wtautological-constant-out-of-range-compare triggers spurious
-# warnings in some of the code, so we have to selectively disable it.
-BIND10_CXX_TRY_FLAG([-Wno-tautological-constant-out-of-range-compare],
-	[WARNING_NO_TAUTOLOGICAL_CONSTANT_OOR_COMP_CFLAG="-Wno-tautological-constant-out-of-range-compare"])
-AC_SUBST(WARNING_NO_TAUTOLOGICAL_CONSTANT_OOR_COMP_CFLAG)
-
 # gcc specific settings:
 if test "X$GXX" = "Xyes"; then
 CXX_VERSION=`$CXX --version 2> /dev/null | head -1`

+ 1 - 7
src/lib/dns/Makefile.am

@@ -4,13 +4,7 @@ AM_CPPFLAGS = -I$(top_srcdir)/src/lib -I$(top_builddir)/src/lib
 AM_CPPFLAGS += -I$(top_srcdir)/src/lib/dns -I$(top_builddir)/src/lib/dns
 AM_CPPFLAGS += -I$(top_srcdir)/src/lib/util -I$(top_builddir)/src/lib/util
 AM_CPPFLAGS += $(BOOST_INCLUDES)
-#AM_CXXFLAGS = $(B10_CXXFLAGS)
-AM_CXXFLAGS = $(B10_CXXFLAGS) -Wno-tautological-constant-out-of-range-compare
-
-# This flag triggers a spurious warning in message.cc.  Disabling the flag
-# library-wide may be a bit too strong, but hopefully actual defects that
-# this flag would correctly detect would be very rare.
-AM_CXXFLAGS += $(WARNING_NO_TAUTOLOGICAL_CONSTANT_OOR_COMP_CFLAG)
+AM_CXXFLAGS = $(B10_CXXFLAGS)
 
 CLEANFILES = *.gcno *.gcda
 CLEANFILES += rrclass.h rrtype.h rrparamregistry.cc rdataclass.h rdataclass.cc s-rdatacode