diff options
| author | Phil Sutter <phil@nwl.cc> | 2018-02-28 16:06:16 +0100 |
|---|---|---|
| committer | Florian Westphal <fw@strlen.de> | 2018-02-28 16:59:19 +0100 |
| commit | 7f31d3191622b650521014e311ace96aa7c5522c (patch) | |
| tree | e27bde4056fd7b21f3d09aaf689aa2573238afc9 /src/netlink_delinearize.c | |
| parent | 48b99f03b5e43eac2bf93251f8fa1c9ceb516abb (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 <phil@nwl.cc>
Signed-off-by: Florian Westphal <fw@strlen.de>
Diffstat (limited to 'src/netlink_delinearize.c')
| -rw-r--r-- | src/netlink_delinearize.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/netlink_delinearize.c b/src/netlink_delinearize.c index a1f0e923..9dbd9141 100644 --- a/src/netlink_delinearize.c +++ b/src/netlink_delinearize.c @@ -1485,6 +1485,7 @@ static void ct_meta_common_postprocess(struct rule_pp_ctx *ctx, case OP_NEQ: if (right->ops->type != EXPR_SET && right->ops->type != EXPR_SET_REF) break; + /* fall through */ case OP_LOOKUP: expr_set_type(right, left->dtype, left->byteorder); break; |