Change in flashrom[master]: Makefile,meson.build: Enable/assume -Wextra

[Nico Huber (Code Review)]

Nico Huber has submitted this change. ( https://review.coreboot.org/c/flashrom/+/30411 )

Change subject: Makefile,meson.build: Enable/assume -Wextra
......................................................................

Makefile,meson.build: Enable/assume -Wextra

Enable all -Wextra warnings but -Wunused-parameter. Nobody seems to
miss warnings about unused parameters and we have a lot unavoidable
occurrences in flashrom because of common interfaces.

Change-Id: Id2ece264c2d483e34019985dd3a7631c4889abe6
Signed-off-by: Nico Huber 
Reviewed-on: https://review.coreboot.org/c/flashrom/+/30411
Tested-by: build bot (Jenkins) 
Reviewed-by: Jacob Garber 
---
M Makefile
M meson.build
2 files changed, 1 insertion(+), 3 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Jacob Garber: Looks good to me, approved

diff --git a/Makefile b/Makefile
index b0c242d..f0a1b61 100644
--- a/Makefile
+++ b/Makefile
@@ -30,7 +30,7 @@
 DIFF    = diff
 PREFIX  ?= /usr/local
 MANDIR  ?= $(PREFIX)/share/man
-CFLAGS  ?= -Os -Wall -Wshadow -Wmissing-prototypes -Wwrite-strings
+CFLAGS  ?= -Os -Wall -Wextra -Wno-unused-parameter -Wshadow -Wmissing-prototypes -Wwrite-strings
 EXPORTDIR ?= .
 RANLIB  ?= ranlib
 PKG_CONFIG ?= pkg-config
diff --git a/meson.build b/meson.build
index fda1ab8..ca52ccf 100644
--- a/meson.build
+++ b/meson.build
@@ -15,10 +15,8 @@
 warning_flags = [
   '-Wwrite-strings',
   '-Wno-unused-parameter',
-  '-Wno-sign-compare',
   '-Wno-address-of-packed-member',
   '-Wno-enum-conversion',
-  '-Wno-missing-field-initializers',
   '-Wno-missing-braces',
 ]