diff options
| author | Jason Gerecke <jason.gerecke@wacom.com> | 2022-02-19 12:16:55 -0800 |
|---|---|---|
| committer | Peter Hutterer <peter.hutterer@who-t.net> | 2022-02-22 10:41:51 +1000 |
| commit | 5390ebb2f55f40a3d41f4352447fb603ab4e2fd2 (patch) | |
| tree | a48cfbd0c510346bb9bed76111516de134123121 | |
| parent | e217fe67690ff8d00af6ccc2bbe65720af21a979 (diff) | |
| download | xf86-input-wacom-5390ebb2f55f40a3d41f4352447fb603ab4e2fd2.tar.gz | |
Enable additional compiler warnings by default
Removes the two `-Wno-xxxx` directives from out default set of build flags
enables several new warnings as well.
Signed-off-by: Jason Gerecke <jason.gerecke@wacom.com>
| -rw-r--r-- | .github/workflows/build.yml | 2 | ||||
| -rw-r--r-- | configure.ac | 8 | ||||
| -rw-r--r-- | meson.build | 8 |
3 files changed, 13 insertions, 5 deletions
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index b82cbbb..a88d8b7 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -1,7 +1,7 @@ on: [ push, pull_request ] env: - CFLAGS: -Werror -Wno-error=sign-compare -Wno-error=missing-field-initializers + CFLAGS: -Werror UBUNTU_PACKAGES: | xutils-dev xserver-xorg-dev libx11-dev libxi-dev libxrandr-dev libxinerama-dev libudev-dev libgirepository1.0-dev libevdev-dev diff --git a/configure.ac b/configure.ac index 86f30b4..e8dcc5b 100644 --- a/configure.ac +++ b/configure.ac @@ -51,7 +51,6 @@ m4_define(COMPILER_FLAGS, [ -Wall -Wextra -Wno-unused-parameter - -Wno-sign-compare dnl lots of work to get rid of this -Wmissing-prototypes -Wstrict-prototypes -Wlogical-op @@ -63,8 +62,13 @@ m4_define(COMPILER_FLAGS, [ -Wredundant-decls -Wincompatible-pointer-types -Wformat=2 - -Wno-missing-field-initializers + -Wformat-overflow=2 + -Wformat-signedness + -Wformat-truncation=2 -Wmissing-declarations + -Wshift-overflow=2 + -Wstrict-overflow=2 + -Wswitch-enum -fvisibility=hidden ]) diff --git a/meson.build b/meson.build index eeb4d31..d61fdb3 100644 --- a/meson.build +++ b/meson.build @@ -25,7 +25,6 @@ cflags = [ '-Wall', '-Wextra', '-Wno-unused-parameter', - '-Wno-sign-compare', # lots of work to get rid of this '-Wmissing-prototypes', '-Wstrict-prototypes', '-Wlogical-op', @@ -37,8 +36,13 @@ cflags = [ '-Wredundant-decls', '-Wincompatible-pointer-types', '-Wformat=2', - '-Wno-missing-field-initializers', + '-Wformat-overflow=2', + '-Wformat-signedness', + '-Wformat-truncation=2', '-Wmissing-declarations', + '-Wshift-overflow=2', + '-Wstrict-overflow=2', + '-Wswitch-enum', '-fvisibility=hidden', ] |