Add -Wextra to gcc devteam warnings
authorBenjamin Kaduk <bkaduk@akamai.com>
Tue, 18 Apr 2017 15:48:11 +0000 (10:48 -0500)
committerRich Salz <rsalz@openssl.org>
Tue, 20 Jun 2017 17:35:33 +0000 (13:35 -0400)
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)

Configure

index f8faa678dabb3ff992a091c91a73c76573a351c9..e55db22d44e8cd7adf7d20adfbc0f10c6e403604 100755 (executable)
--- a/Configure
+++ b/Configure
@@ -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"