diff options
author | Benjamin Kaduk <bkaduk@akamai.com> | 2017-04-18 10:48:11 -0500 |
---|---|---|
committer | Rich Salz <rsalz@openssl.org> | 2017-05-01 14:23:28 -0400 |
commit | 560ad13c74fe6967991a2429d90eeeba815d1f9e (patch) | |
tree | 357f0715038f8dcabadf2f440b15fd27aac4bb81 | |
parent | 1ff86c5efa946427e20b3504c460e83edbe53905 (diff) | |
download | openssl-560ad13c74fe6967991a2429d90eeeba815d1f9e.tar.gz |
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)
-rwxr-xr-x | Configure | 3 |
1 files changed, 3 insertions, 0 deletions
@@ -119,6 +119,9 @@ my $gcc_devteam_warn = "-DDEBUG_UNUSED" . " -Wswitch" . " -DPEDANTIC -pedantic -Wno-long-long" . " -Wall" + . " -Wextra" + . " -Wno-unused-parameter" + . " -Wno-missing-field-initializers" . " -Wsign-compare" . " -Wmissing-prototypes" . " -Wshadow" |