Martin Roth merged this change.

View Change

Approvals: build bot (Jenkins): Verified Martin Roth: Looks good to me, approved Paul Menzel: Looks good to me, but someone else must approve
util/cbmem: Enable -Wextra

This enables extra useful warnings.

Change-Id: I4afbbb0fefb32a7d954aafd87df17075b0abe6f7
Signed-off-by: Jacob Garber <jgarber1@ualberta.ca>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/33851
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Martin Roth <martinroth@google.com>
---
M util/cbmem/Makefile
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/util/cbmem/Makefile b/util/cbmem/Makefile
index eed08a4..e7318a1 100644
--- a/util/cbmem/Makefile
+++ b/util/cbmem/Makefile
@@ -19,7 +19,7 @@
INSTALL ?= /usr/bin/env install
PREFIX ?= /usr/local
CFLAGS ?= -O2
-CFLAGS += -Wall -Werror
+CFLAGS += -Wall -Wextra -Werror
CPPFLAGS += -I . -I $(ROOT)/commonlib/include
CPPFLAGS += -include ../../src/commonlib/include/commonlib/compiler.h


To view, visit change 33851. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I4afbbb0fefb32a7d954aafd87df17075b0abe6f7
Gerrit-Change-Number: 33851
Gerrit-PatchSet: 2
Gerrit-Owner: Jacob Garber <jgarber1@ualberta.ca>
Gerrit-Reviewer: Martin Roth <martinroth@google.com>
Gerrit-Reviewer: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-MessageType: merged