diff mbox

[nft] expression: Fix inconsistent output in set

Message ID 1daccfe0798bd032f1770b0ae7888a88db46f3f6.1397204678.git.anarey@gmail.com
State Deferred
Headers show

Commit Message

Ana Rey April 11, 2014, 8:26 a.m. UTC
Add a whitespace between the last element in a set and the close braces.

Old output set:

	set set-test {
		type ipv4_address
		elements = { 192.168.1.44, 192.168.2.4}
	}

New output set:
	set set-test {
		type ipv4_address
		elements = { 192.168.1.44, 192.168.2.4 }
	}

Signed-off-by: Ana Rey <anarey@gmail.com>
---
 src/expression.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Patrick McHardy April 12, 2014, 9:49 a.m. UTC | #1
On Fri, Apr 11, 2014 at 10:26:46AM +0200, Ana Rey wrote:
> Add a whitespace between the last element in a set and the close braces.
> 
> Old output set:
> 
> 	set set-test {
> 		type ipv4_address
> 		elements = { 192.168.1.44, 192.168.2.4}
> 	}
> 
> New output set:
> 	set set-test {
> 		type ipv4_address
> 		elements = { 192.168.1.44, 192.168.2.4 }
> 	}
> 

That's actually deliberate. A space is always added before a set member, but never
after.
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/src/expression.c b/src/expression.c
index fa14d99..3b299ff 100644
--- a/src/expression.c
+++ b/src/expression.c
@@ -734,7 +734,7 @@  static void set_expr_print(const struct expr *expr)
 {
 	printf("{ ");
 	compound_expr_print(expr, ", ");
-	printf("}");
+	printf(" }");
 }
 
 static void set_expr_set_type(const struct expr *expr,