diff mbox series

[ethtool,7/7] build: add -Wextra to default CFLAGS

Message ID 12f1db189afc7798ff4d53326221ee6758628bc3.1597007533.git.mkubecek@suse.cz
State Changes Requested
Delegated to: Michal Kubecek
Headers show
Series compiler warnings cleanup, part 2 | expand

Commit Message

Michal Kubecek Aug. 9, 2020, 9:24 p.m. UTC
As a result of previous commits, ethtool source now builds with gcc
versions 7-11 without any compiler warning with "-Wall -Wextra". Add
"-Wextra" to default cflags to make sure that any new warnings are
caught as early as possible.

Suggested-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: Michal Kubecek <mkubecek@suse.cz>
---
 Makefile.am | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Andrew Lunn Aug. 10, 2020, 2:22 p.m. UTC | #1
On Sun, Aug 09, 2020 at 11:24:38PM +0200, Michal Kubecek wrote:
> As a result of previous commits, ethtool source now builds with gcc
> versions 7-11 without any compiler warning with "-Wall -Wextra". Add
> "-Wextra" to default cflags to make sure that any new warnings are
> caught as early as possible.
> 
> Suggested-by: Andrew Lunn <andrew@lunn.ch>
> Signed-off-by: Michal Kubecek <mkubecek@suse.cz>

Reviewed-by: Andrew Lunn <andrew@lunn.ch>

    Andrew
diff mbox series

Patch

diff --git a/Makefile.am b/Makefile.am
index 2abb2742c335..099182e8d6ad 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -1,4 +1,4 @@ 
-AM_CFLAGS = -Wall
+AM_CFLAGS = -Wall -Wextra
 AM_CPPFLAGS = -I$(top_srcdir)/uapi
 LDADD = -lm