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/hash.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/hash.c') diff --git a/src/hash.c b/src/hash.c index 9cd3c8cf..3355cadd 100644 --- a/src/hash.c +++ b/src/hash.c @@ -20,7 +20,7 @@ static void hash_expr_print(const struct expr *expr, struct output_ctx *octx) switch (expr->hash.type) { case NFT_HASH_SYM: nft_print(octx, "symhash"); - break; + break; case NFT_HASH_JENKINS: default: nft_print(octx, "jhash "); -- cgit v1.2.3