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/evaluate.c | 1 + 1 file changed, 1 insertion(+) (limited to 'src/evaluate.c') diff --git a/src/evaluate.c b/src/evaluate.c index 6be3bf03..54fd6b61 100644 --- a/src/evaluate.c +++ b/src/evaluate.c @@ -2712,6 +2712,7 @@ static int stmt_evaluate_objref_map(struct eval_ctx *ctx, struct stmt *stmt) map->mappings->set->flags |= map->mappings->set->init->set_flags; + /* fall through */ case EXPR_SYMBOL: if (expr_evaluate(ctx, &map->mappings) < 0) return -1; -- cgit v1.2.3