From a00d9d06b5021c27333e52e88b99493fa56d15a1 Mon Sep 17 00:00:00 2001 From: Phil Sutter Date: Tue, 9 Mar 2021 13:41:44 +0100 Subject: Get rid of single option switch statements Replace each by a conditional testing the only valid case. There is one odd example, namely src/set.c: When printing a set with type NFTNL_OUTPUT_XML, the relevant function would return 0 instead of -1 like all others. Just drop it assuming nothing depends on that (faulty) behaviour. Cc: Arturo Borrero Signed-off-by: Phil Sutter --- src/table.c | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) (limited to 'src/table.c') diff --git a/src/table.c b/src/table.c index 5835788..59e7053 100644 --- a/src/table.c +++ b/src/table.c @@ -373,17 +373,11 @@ static int nftnl_table_cmd_snprintf(char *buf, size_t remain, { int ret, offset = 0; - switch (type) { - case NFTNL_OUTPUT_DEFAULT: - ret = nftnl_table_snprintf_default(buf + offset, remain, t); - SNPRINTF_BUFFER_SIZE(ret, remain, offset); - break; - case NFTNL_OUTPUT_XML: - case NFTNL_OUTPUT_JSON: - default: + if (type != NFTNL_OUTPUT_DEFAULT) return -1; - } + ret = nftnl_table_snprintf_default(buf + offset, remain, t); + SNPRINTF_BUFFER_SIZE(ret, remain, offset); return offset; } -- cgit v1.2.3