diff options
| author | 2016-08-25 00:12:03 -0400 | |
|---|---|---|
| committer | 2016-08-25 00:22:14 -0400 | |
| commit | 717e1eb9d2f3d4399f1793773b3c48397c728b43 (patch) | |
| tree | 5d1a42c6eb8b3a3a355e657649d91b1268dc4b63 /scripts | |
| parent | 0d08c04009a954a64b7bf4d7be978a744940d721 (diff) | |
Change warning flags for Clang build (nw)
-Wno-missing-braces is unfortunately now necessary to avoid errors on almost every single use of the std::array initializer in devfind.h. (Apparently C++14 does make double braces optional for std::array initializers, but clang and GCC's -Wmissing-braces never took that into proper account, which is why GCC may still have it disabled by default.)
Several other Clang-specific warnings have been undisabled; some might perhaps need to be locally reenabled if legacy 3rdparty code or sundry OSD includes happen to disagree with them. The only warning to require any code changes was -Wabsolute-value, which seems to have caught a minor render bug (and nothing more).
Diffstat (limited to 'scripts')
| -rw-r--r-- | scripts/genie.lua | 10 |
1 files changed, 1 insertions, 9 deletions
diff --git a/scripts/genie.lua b/scripts/genie.lua index 3f6455a5cf8..2488671cc2e 100644 --- a/scripts/genie.lua +++ b/scripts/genie.lua @@ -958,25 +958,17 @@ end buildoptions { "-Wno-cast-align", "-Wno-tautological-compare", - "-Wno-dynamic-class-memaccess", "-Wno-unused-value", - "-Wno-inline-new-delete", "-Wno-constant-logical-operand", - "-Wno-deprecated-register", + "-Wno-missing-braces", -- clang is not as permissive as GCC about std::array initialization "-fdiagnostics-show-note-include-stack", } if (version >= 30500) then buildoptions { - "-Wno-absolute-value", "-Wno-unknown-warning-option", "-Wno-extern-c-compat", } end - if (version >= 70000) then - buildoptions { - "-Wno-tautological-undefined-compare", - } - end else if (version < 40900) then print("GCC version 4.9 or later needed") |
