path: root/src/ct.c
diff options
authorPhil Sutter <>2018-02-28 16:06:16 +0100
committerFlorian Westphal <>2018-02-28 16:59:19 +0100
commit7f31d3191622b650521014e311ace96aa7c5522c (patch)
treee27bde4056fd7b21f3d09aaf689aa2573238afc9 /src/ct.c
parent48b99f03b5e43eac2bf93251f8fa1c9ceb516abb (diff)
Review switch statements for unmarked fall through cases
While revisiting all of them, clear a few oddities as well: - There's no point in marking empty fall through cases: They are easy to spot and a common concept when using switch(). - Fix indenting of break statement in one occasion. - Drop needless braces around one case which doesn't declare variables. Signed-off-by: Phil Sutter <> Signed-off-by: Florian Westphal <>
Diffstat (limited to 'src/ct.c')
1 files changed, 1 insertions, 1 deletions
diff --git a/src/ct.c b/src/ct.c
index d5347974..bdf50d7a 100644
--- a/src/ct.c
+++ b/src/ct.c
@@ -289,7 +289,7 @@ static void ct_print(enum nft_ct_keys key, int8_t dir, uint8_t nfproto,
switch (key) {
- case NFT_CT_SRC: /* fallthrough */
+ case NFT_CT_SRC:
case NFT_CT_DST:
desc = proto_find_upper(&proto_inet, nfproto);
if (desc)