From: Benjamin Kaduk <bkaduk@akamai.com>
Date: Tue, 18 Apr 2017 15:48:11 +0000 (-0500)
Subject: Add -Wextra to gcc devteam warnings
X-Git-Tag: OpenSSL_1_1_1-pre1~1623
X-Git-Url: https://git.librecmc.org/?a=commitdiff_plain;ds=sidebyside;h=560ad13c74fe6967991a2429d90eeeba815d1f9e;p=oweals%2Fopenssl.git

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)
---

diff --git a/Configure b/Configure
index c699ae051b..d23b29cb68 100755
--- a/Configure
+++ b/Configure
@@ -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"