summaryrefslogtreecommitdiffstatshomepage
path: root/scripts
diff options
context:
space:
mode:
author AJR <ajrhacker@users.noreply.github.com>2016-08-25 00:12:03 -0400
committer AJR <ajrhacker@users.noreply.github.com>2016-08-25 00:22:14 -0400
commit717e1eb9d2f3d4399f1793773b3c48397c728b43 (patch)
tree5d1a42c6eb8b3a3a355e657649d91b1268dc4b63 /scripts
parent0d08c04009a954a64b7bf4d7be978a744940d721 (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.lua10
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")