From 725f121dcfc31d016fa23bfffde1b6342659949e Mon Sep 17 00:00:00 2001 From: Benjamin Kaduk Date: Tue, 18 Apr 2017 10:48:11 -0500 Subject: [PATCH] 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 Reviewed-by: Rich Salz (Merged from https://github.com/openssl/openssl/pull/3239) (cherry picked from commit 560ad13c74fe6967991a2429d90eeeba815d1f9e) --- Configure | 3 +++ 1 file changed, 3 insertions(+) diff --git a/Configure b/Configure index f8faa678da..e55db22d44 100755 --- 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" -- 2.25.1