Since the GCC -Wconversion warnings are fixed (at least on a x86-64 environment), enable it in the Makefile.

This commit is contained in:
Edoardo Prezioso 2012-07-18 00:50:30 +02:00
parent aedaacba95
commit 826d5bb00b
2 changed files with 2 additions and 3 deletions

View File

@ -37,7 +37,7 @@ else # !COMSPEC
endif # COMSPEC
ifndef CXXFLAGS
CXXFLAGS=-pedantic -Wall -Wextra -Wabi -Wcast-qual -Wfloat-equal -Winline -Wmissing-declarations -Wmissing-format-attribute -Wno-long-long -Woverloaded-virtual -Wpacked -Wredundant-decls -Wshadow -Wsign-promo $(CPPCHK_GLIBCXX_DEBUG) -g
CXXFLAGS=-pedantic -Wall -Wextra -Wabi -Wcast-qual -Wconversion -Wfloat-equal -Winline -Wmissing-declarations -Wmissing-format-attribute -Wno-long-long -Woverloaded-virtual -Wpacked -Wredundant-decls -Wshadow -Wsign-promo $(CPPCHK_GLIBCXX_DEBUG) -g
endif
ifeq ($(HAVE_RULES),yes)

View File

@ -263,7 +263,6 @@ int main(int argc, char **argv)
} else {
// TODO: add more compiler warnings.
// -Wlogical-op : doesn't work on older GCC
// -Wconversion : too many warnings
// -Wsign-conversion : too many warnings
// -Wunreachable-code : some GCC versions report lots of warnings
makeConditionalVariable(fout, "CXXFLAGS",
@ -272,7 +271,7 @@ int main(int argc, char **argv)
"-Wextra "
"-Wabi "
"-Wcast-qual "
// "-Wconversion "
"-Wconversion "
"-Wfloat-equal "
"-Winline "
// "-Wlogical-op "