From 7f31d3191622b650521014e311ace96aa7c5522c Mon Sep 17 00:00:00 2001 From: Phil Sutter Date: Wed, 28 Feb 2018 16:06:16 +0100 Subject: 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 --- src/ct.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/ct.c') 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) -- cgit v1.2.3