summaryrefslogtreecommitdiffstats
path: root/src/rule.c
diff options
context:
space:
mode:
authorJeremy Sowden <jeremy@azazel.net>2021-10-07 21:12:22 +0100
committerPablo Neira Ayuso <pablo@netfilter.org>2021-10-12 01:06:51 +0200
commit88c44060855db67a13170886720658e11ef09763 (patch)
tree1eea538a03cad680a3481ae4af7df34fb63a25b1 /src/rule.c
parentf1d26b51fde8f4ff12f5c98ef8b65229a17276e3 (diff)
rule: replace three conditionals with one
When outputting set definitions, merge three consecutive `if (!list_empty(&set->stmt_list))` conditionals. Signed-off-by: Jeremy Sowden <jeremy@azazel.net> Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to 'src/rule.c')
-rw-r--r--src/rule.c8
1 files changed, 3 insertions, 5 deletions
diff --git a/src/rule.c b/src/rule.c
index b566adf0..7c048fcc 100644
--- a/src/rule.c
+++ b/src/rule.c
@@ -366,12 +366,11 @@ static void set_print_declaration(const struct set *set,
nft_print(octx, "%s", opts->stmt_separator);
}
- if (!list_empty(&set->stmt_list))
- nft_print(octx, "%s%s", opts->tab, opts->tab);
-
if (!list_empty(&set->stmt_list)) {
unsigned int flags = octx->flags;
+ nft_print(octx, "%s%s", opts->tab, opts->tab);
+
octx->flags |= NFT_CTX_OUTPUT_STATELESS;
list_for_each_entry(stmt, &set->stmt_list, list) {
stmt_print(stmt, octx);
@@ -379,10 +378,9 @@ static void set_print_declaration(const struct set *set,
nft_print(octx, " ");
}
octx->flags = flags;
- }
- if (!list_empty(&set->stmt_list))
nft_print(octx, "%s", opts->stmt_separator);
+ }
if (set->automerge)
nft_print(octx, "%s%sauto-merge%s", opts->tab, opts->tab,