1 files changed, 43 insertions, 0 deletions
diff --git a/cmake/CompilerWarnings.cmake b/cmake/CompilerWarnings.cmake
new file mode 100644
index 0000000..895a7e8
--- /dev/null
+++ b/cmake/CompilerWarnings.cmake
@@ -0,0 +1,43 @@
+# compiler warnings based on: https://lefticus.gitbooks.io/cpp-best-practices/content/02-Use_the_Tools_Available.html
+
+# only enable warnings for debug builds
+if (NOT ${CMAKE_BUILD_TYPE} STREQUAL Debug)
+    return()
+endif ()
+
+set(BASE_WARNINGS
+    -Wall -Wextra # reasonable and standard
+    -Wpedantic # warn if non-standard c++ is used
+    -Werror=shadow # warn the user if a variable shadows one from a parent context
+    -Wunused # warn on anything being unused
+
+    -Werror=non-virtual-dtor # warn the user if a class with virtual functions has a non-virtual dtor
+    -Werror=overloaded-virtual # warn if you overload (not override) a virtual function
+
+    -Werror=old-style-cast # warn for c-style casts
+    -Wcast-align # warn for potential performance problem casts
+
+    -Wconversion # warn on type conversions that may lose data
+    -Wsign-conversion # warn on sign conversions
+    -Wdouble-promotion # warn if float is implicit promoted to double
+
+    -Wnull-dereference # warn if a null dereference is detected
+    -Wformat=2 # warn on security issues around functions that format output (ie printf)
+)
+
+set(GCC_WARNINGS
+    -Wmisleading-indentation # (only in GCC >= 6.0) warn if indentation implies blocks where blocks do not exist
+    -Wduplicated-cond # (only in GCC >= 6.0) warn if if / else chain has duplicated conditions
+    -Wduplicated-branches # (only in GCC >= 7.0) warn if if / else branches have duplicated code
+    -Wlogical-op # (only in GCC) warn about logical operations being used where bitwise were probably wanted
+    -Wnull-dereference # (only in GCC >= 6.0) warn if a null dereference is detected
+    -Wuseless-cast # (only in GCC >= 4.8) warn if you perform a cast to the same type
+)
+
+if (${CMAKE_CXX_COMPILER_ID} MATCHES Clang)
+    message(STATUS "Adding more ${CMAKE_CXX_COMPILER_ID} compiler warnings")
+    add_compile_options(${BASE_WARNINGS})
+elseif (${CMAKE_CXX_COMPILER_ID} MATCHES GNU)
+    message(STATUS "Adding more ${CMAKE_CXX_COMPILER_ID} compiler warnings")
+    add_compile_options(${BASE_WARNINGS} ${GCC_WARNINGS})
+endif ()