[13/13] Makefile: Actually compile with -Wextra

Message ID	20210127085752.120571-14-aik@ozlabs.ru
State	Superseded
Headers	show Return-Path: <slof-bounces+incoming=patchwork.ozlabs.org@lists.ozlabs.org> From: Alexey Kardashevskiy <aik@ozlabs.ru> To: slof@lists.ozlabs.org Date: Wed, 27 Jan 2021 19:57:52 +1100 Message-Id: <20210127085752.120571-14-aik@ozlabs.ru> In-Reply-To: <20210127085752.120571-1-aik@ozlabs.ru> References: <20210127085752.120571-1-aik@ozlabs.ru> Subject: [SLOF] [PATCH slof 13/13] Makefile: Actually compile with -Wextra Precedence: list MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: slof-bounces+incoming=patchwork.ozlabs.org@lists.ozlabs.org Sender: "SLOF" <slof-bounces+incoming=patchwork.ozlabs.org@lists.ozlabs.org>
Series	Compile with -Wextra \| expand [00/13] Compile with -Wextra [01/13] helpers: Define MIN() [02/13] libc: Compile with -Wextra [03/13] elf: Compile with -Wextra [04/13] usb: Compile with -Wextra [05/13] veth: Compile with -Wextra [06/13] virtio: Compile with -Wextra [07/13] e1000: Compile with -Wextra [08/13] libnet: Compile with -Wextra [09/13] libhv: Compile with -Wextra [10/13] libnvram: Compile with -Wextra [11/13] libtpm: Compile with -Wextra [12/13] slof/prim: Compile with -Wextra [13/13] Makefile: Actually compile with -Wextra

Message ID

20210127085752.120571-14-aik@ozlabs.ru

State

Superseded

Headers

From: Alexey Kardashevskiy <aik@ozlabs.ru>
To: slof@lists.ozlabs.org
Date: Wed, 27 Jan 2021 19:57:52 +1100
Message-Id: <20210127085752.120571-14-aik@ozlabs.ru>
In-Reply-To: <20210127085752.120571-1-aik@ozlabs.ru>
References: <20210127085752.120571-1-aik@ozlabs.ru>
Subject: [SLOF] [PATCH slof 13/13] Makefile: Actually compile with -Wextra
Precedence: list
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Errors-To: slof-bounces+incoming=patchwork.ozlabs.org@lists.ozlabs.org
Sender: "SLOF" <slof-bounces+incoming=patchwork.ozlabs.org@lists.ozlabs.org>

Series

Compile with -Wextra | expand

Commit Message

Alexey Kardashevskiy Jan. 27, 2021, 8:57 a.m. UTC

-Wextra enables a bunch of rather useful checks which this fixes.

The only exception is -Wno-unused-parameter, one thing at the time.

Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru>
---
 make.rules | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/make.rules b/make.rules
index 3dfbb5b136c2..885eea3f48b0 100644
--- a/make.rules
+++ b/make.rules
@@ -76,7 +76,7 @@  AR		?= $(CROSS)ar
 RANLIB		?= $(CROSS)ranlib
 CPP		?= $(CROSS)cpp
 
-WARNFLAGS = -Wall -Wmissing-prototypes -Wstrict-prototypes -Wformat-security
+WARNFLAGS = -Wall -Wmissing-prototypes -Wstrict-prototypes -Wformat-security -Wextra -Wno-unused-parameter
 CFLAGS ?= -g -O2 -fno-builtin -ffreestanding -nostdinc -msoft-float \
 	  -fno-strict-aliasing -mno-altivec -mabi=no-altivec \
 	  -fno-stack-protector -fno-asynchronous-unwind-tables $(WARNFLAGS)