diff options
author | onqtam <vik.kirilov@gmail.com> | 2018-05-08 19:33:38 +0300 |
---|---|---|
committer | onqtam <vik.kirilov@gmail.com> | 2018-05-10 16:18:43 +0300 |
commit | 6b2c99fb0f7a6de2800a674cd7603360ade671b3 (patch) | |
tree | fb74217e3adce5b1eff53969b27e2325372928f5 | |
parent | 3eaf68740aac2430c8f3f0197a66133e8fccd35e (diff) |
added... the only new warning option for GCC 8 that is not in Wall/Wextra
-rw-r--r-- | scripts/cmake/common.cmake | 5 | ||||
-rw-r--r-- | scripts/random_dev_notes.md | 2 |
2 files changed, 4 insertions, 3 deletions
diff --git a/scripts/cmake/common.cmake b/scripts/cmake/common.cmake index cdba5f0..d4dc7d8 100644 --- a/scripts/cmake/common.cmake +++ b/scripts/cmake/common.cmake @@ -137,7 +137,10 @@ if(CMAKE_CXX_COMPILER_ID MATCHES "GNU") add_compiler_flags(-Walloc-zero) add_compiler_flags(-Walloca) add_compiler_flags(-Wduplicated-branches) - add_compiler_flags(-Wrestrict) + endif() + + if(NOT CMAKE_CXX_COMPILER_VERSION VERSION_LESS 8.0) + add_compiler_flags(-Wcast-align=strict) endif() endif() diff --git a/scripts/random_dev_notes.md b/scripts/random_dev_notes.md index 4601977..bdaf446 100644 --- a/scripts/random_dev_notes.md +++ b/scripts/random_dev_notes.md @@ -3,8 +3,6 @@ check console colors after refactor... (num) vs :num: for line numbers (__GNUG__) (also test with a post-build command to run the tests - if the errors are in the IDE) -add gcc8 warnings - https://github.com/catchorg/Catch2/commit/de36b2ada6e4593a9a32c4c86cd47d4bc002b148 add [[noreturn]] to MessageBuilder::react() - and actually make a separate function (react2) for the FAIL() case |