diff options
| author | Jack Lloyd <[email protected]> | 2016-01-17 19:02:13 -0500 |
|---|---|---|
| committer | Jack Lloyd <[email protected]> | 2016-01-17 19:10:33 -0500 |
| commit | 6fcbd57915d7870c2782bd6e098903353e8756b0 (patch) | |
| tree | c40505fbf8021819dc15d8a632a2d6fb5557f2aa /src/build-data/cc/gcc.txt | |
| parent | de22a9eda8bfb087b690feb962fae8313ee526d4 (diff) | |
Add missing overrides and fix -Wpedantic 'extra ;' warnings
Remove -Wsuggest-attribute=noreturn from maintainer mode flags as it
seems like outside of the assertion failure macro any other suggestion
would always be a false positive (an unimplemented function or the like).
Or at least, if such a function needing noreturn to assist with static
analysis is added in the future it will be obvious, by virtue of the
static analyzer warnings which occur due to the missing noreturn
preventing the analyzer from understanding code flow.
Diffstat (limited to 'src/build-data/cc/gcc.txt')
| -rw-r--r-- | src/build-data/cc/gcc.txt | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/build-data/cc/gcc.txt b/src/build-data/cc/gcc.txt index 634b9e751..d5eff4734 100644 --- a/src/build-data/cc/gcc.txt +++ b/src/build-data/cc/gcc.txt @@ -8,7 +8,7 @@ add_lib_dir_option -L add_lib_option -l lang_flags "-std=c++11 -D_REENTRANT" -maintainer_warning_flags "-Wold-style-cast -Werror -Wno-error=old-style-cast -Wno-error=zero-as-null-pointer-constant -Wno-error=unused-parameter -Wno-error=unused-variable -Wno-error=strict-overflow -Wsuggest-override -Wsuggest-attribute=noreturn" +maintainer_warning_flags "-Wold-style-cast -Werror -Wno-error=old-style-cast -Wno-error=zero-as-null-pointer-constant -Wno-error=unused-parameter -Wno-error=unused-variable -Wno-error=strict-overflow -Wsuggest-override" warning_flags "-Wall -Wextra -Wpedantic -Wstrict-aliasing -Wstrict-overflow=5 -Wcast-align -Wmissing-declarations -Wpointer-arith -Wcast-qual -Wzero-as-null-pointer-constant -Wnon-virtual-dtor" compile_flags "-c" |