From 9952201ca72b42819d64a9174fa7b5b898215668 Mon Sep 17 00:00:00 2001
From: nicm <nicm>
Date: Tue, 27 Oct 2015 09:28:31 +0000
Subject: Count brackets in #{?...} so that nested conditional formats work,
 from Daniel De Graaf.

---
 format.c | 32 +++++++++++++++++++-------------
 1 file changed, 19 insertions(+), 13 deletions(-)

(limited to 'format.c')

diff --git a/format.c b/format.c
index 0d7d7c20..285bfadb 100644
--- a/format.c
+++ b/format.c
@@ -670,7 +670,7 @@ format_replace(struct format_tree *ft, const char *key, size_t keylen,
 	char		*copy, *copy0, *endptr, *ptr, *saved, *trimmed, *value;
 	size_t		 valuelen;
 	u_long		 limit = 0;
-	int		 modifiers = 0;
+	int		 modifiers = 0, brackets;
 
 	/* Make a copy of the key. */
 	copy0 = copy = xmalloc(keylen + 1);
@@ -718,20 +718,26 @@ format_replace(struct format_tree *ft, const char *key, size_t keylen,
 			goto fail;
 		*ptr = '\0';
 
-		value = saved = format_find(ft, copy + 1, modifiers);
-		if (value != NULL && *value != '\0' &&
-		    (value[0] != '0' || value[1] != '\0')) {
-			value = ptr + 1;
-			ptr = strchr(value, ',');
-			if (ptr == NULL)
-				goto fail;
+		value = ptr + 1;
+		saved = format_find(ft, copy + 1, modifiers);
+
+		brackets = 0;
+		for (ptr = ptr + 1; *ptr != '\0'; ptr++) {
+			if (*ptr == '{')
+				brackets++;
+			if (*ptr == '}')
+				brackets--;
+			if (*ptr == ',' && brackets == 0)
+				break;
+		}
+		if (*ptr == '\0')
+			goto fail;
+
+		if (saved != NULL && *saved != '\0' &&
+		    (saved[0] != '0' || saved[1] != '\0')) {
 			*ptr = '\0';
-		} else {
-			ptr = strchr(ptr + 1, ',');
-			if (ptr == NULL)
-				goto fail;
+		} else
 			value = ptr + 1;
-		}
 		value = format_expand(ft, value);
 		free(saved);
 		saved = value;
-- 
cgit