summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAllan McRae <allan@archlinux.org>2019-02-12 11:17:30 +1000
committerAllan McRae <allan@archlinux.org>2019-02-12 11:29:13 +1000
commit86004227d14bef059f71290b3c0454ad88ea0f86 (patch)
tree6876053a8632521131d72b9654994b68973aa491
parentdb102c67efe9cfd4367a9110c8f97ceff5afcbe3 (diff)
downloadpacman-86004227d14bef059f71290b3c0454ad88ea0f86.tar.xz
Add implicit fall through warning
Requires modification to our comment about fall through to match compilers expectations. Works for GCC and Clang. Signed-off-by: Allan McRae <allan@archlinux.org>
-rw-r--r--configure.ac1
-rw-r--r--meson.build1
-rw-r--r--src/pacman/pacman.c2
3 files changed, 3 insertions, 1 deletions
diff --git a/configure.ac b/configure.ac
index 415ed3cb..2f345b5d 100644
--- a/configure.ac
+++ b/configure.ac
@@ -462,6 +462,7 @@ if test "x$warningflags" = "xyes" ; then
CFLAGS_ADD([-Wformat-nonliteral], [WARNING_CFLAGS])
CFLAGS_ADD([-Wformat-security], [WARNING_CFLAGS])
CFLAGS_ADD([-Wignored-qualifiers], [WARNING_CFLAGS])
+ CFLAGS_ADD([-Wimplicit-fallthrough], [WARNING_CFLAGS])
CFLAGS_ADD([-Winit-self], [WARNING_CFLAGS])
CFLAGS_ADD([-Wlogical-op], [WARNING_CFLAGS])
CFLAGS_ADD([-Wmissing-declarations], [WARNING_CFLAGS])
diff --git a/meson.build b/meson.build
index 0a710653..02a3a3d3 100644
--- a/meson.build
+++ b/meson.build
@@ -220,6 +220,7 @@ if get_option('buildtype').startswith('debug')
'-Wformat-nonliteral',
'-Wformat-security',
'-Wignored-qualifiers',
+ '-Wimplicit-fallthrough',
'-Winit-self',
'-Wlogical-op',
'-Wmissing-declarations',
diff --git a/src/pacman/pacman.c b/src/pacman/pacman.c
index a2a420b6..3bb622e6 100644
--- a/src/pacman/pacman.c
+++ b/src/pacman/pacman.c
@@ -196,7 +196,7 @@ static void usage(int op, const char * const myname)
addlist(_(" --ignore <pkg> ignore a package upgrade (can be used more than once)\n"));
addlist(_(" --ignoregroup <grp>\n"
" ignore a group upgrade (can be used more than once)\n"));
- /* pass through */
+ /* fall through */
case PM_OP_REMOVE:
addlist(_(" -d, --nodeps skip dependency version checks (-dd to skip all checks)\n"));
addlist(_(" --assume-installed <package=version>\n"