Add more recommended switches

Commented out switches still give errors.
This commit is contained in:
Vasily
2019-02-21 18:48:53 +01:00
committed by username223
parent ac614c36e6
commit fddae4278b
3 changed files with 16 additions and 4 deletions
+7 -2
View File
@@ -8,7 +8,7 @@ include(common-msvc.pri)
#clang-msvc does not enable the c++17 flag with qmake's c++1z flag.
QMAKE_CXXFLAGS += /std:c++17
QMAKE_CXXFLAGS += -Wextra
QMAKE_CXXFLAGS += -Wextra -Wmost
QMAKE_CXXFLAGS += -Wshadow
@@ -16,4 +16,9 @@ QMAKE_CXXFLAGS += -Wold-style-cast
QMAKE_CXXFLAGS += -Wnon-virtual-dtor
QMAKE_CXXFLAGS += -Wpedantic
#QMAKE_CXXFLAGS += -Wpedantic
QMAKE_CXXFLAGS += -Wcast-align -Wunused -Woverloaded-virtual -Wformat=2
#QMAKE_CXXFLAGS += -Wconversion
#QMAKE_CXXFLAGS += -Wdouble-promotion
#QMAKE_CXXFLAGS += -Wsign-conversion
+3 -1
View File
@@ -1,4 +1,4 @@
QMAKE_CXXFLAGS += -Wextra
QMAKE_CXXFLAGS += -Wextra -Wmost
QMAKE_CXXFLAGS += -Werror
#All includes from the third-party directory will not warn.
@@ -11,3 +11,5 @@ QMAKE_CXXFLAGS += -Wold-style-cast
QMAKE_CXXFLAGS += -Wnon-virtual-dtor
QMAKE_CXXFLAGS += -Wpedantic
QMAKE_CXXFLAGS += -Wcast-align -Wunused -Woverloaded-virtual -Wformat=2
+6 -1
View File
@@ -1,7 +1,7 @@
#g++-7 is needed for C++17 features. travis does not supply this by default.
QMAKE_CXX = g++-7
QMAKE_CXXFLAGS += -Wextra
QMAKE_CXXFLAGS += -Wextra -Wmost
QMAKE_CXXFLAGS += -Werror
@@ -12,3 +12,8 @@ QMAKE_CXXFLAGS += -Wold-style-cast
QMAKE_CXXFLAGS += -Wnon-virtual-dtor
QMAKE_CXXFLAGS += -Wpedantic
QMAKE_CXXFLAGS += -Wcast-align -Wunused -Woverloaded-virtual -Wformat=2
# GCC only switches
QMAKE_CXXFLAGS += -Wduplicated-branches -Wduplicated-cond -Wlogical-op -Wrestrict -Wnull-dereference