diff options
author | Benjamin Kaduk <bkaduk@akamai.com> | 2017-04-18 10:48:11 -0500 |
---|---|---|
committer | Rich Salz <rsalz@openssl.org> | 2017-06-20 13:35:33 -0400 |
commit | 725f121dcfc31d016fa23bfffde1b6342659949e (patch) | |
tree | 642d84d476e094abbbeeafb37a6b12bee89a2ed6 /Configure | |
parent | 4b4bc00a00456e6d5cc8b2a26489ae905c049f41 (diff) |
Add -Wextra to gcc devteam warnings
clang already has it; let's flip the switch and deal with the fallout.
Exclude -Wunused-parameter, as we have many places where we keep unused
parameters to conform to a uniform vtable-like interface.
Also exclude -Wmissing-field-initializers; it's okay to rely on
the standard-mandated behavior of filling out with 0/NULL.
Reviewed-by: Andy Polyakov <appro@openssl.org>
Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/3239)
(cherry picked from commit 560ad13c74fe6967991a2429d90eeeba815d1f9e)
Diffstat (limited to 'Configure')
-rwxr-xr-x | Configure | 3 |
1 files changed, 3 insertions, 0 deletions
@@ -115,6 +115,9 @@ my $gcc_devteam_warn = "-DDEBUG_UNUSED" # but 'long long' type. . " -DPEDANTIC -pedantic -Wno-long-long" . " -Wall" + . " -Wextra" + . " -Wno-unused-parameter" + . " -Wno-missing-field-initializers" . " -Wsign-compare" . " -Wmissing-prototypes" . " -Wshadow" |