Re: [PATCH 14/39] virLogParseFilters: Refactor string list handling

Rewrite the code to remove the need to calculate the string list count. Signed-off-by: Peter Krempa <pkrempa(a)redhat.com&gt; --- src/util/virlog.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/util/virlog.c b/src/util/virlog.c index 56059f62f1..57eb0de538 100644 --- a/src/util/virlog.c +++ b/src/util/virlog.c @@ -1668,22 +1668,22 @@ int virLogParseFilters(const char *src, virLogFilterPtr **filters) { size_t nfilters = 0; - size_t i, count; g_auto(GStrv) strings = NULL; + GStrv next; virLogFilterPtr filter = NULL; virLogFilterPtr *list = NULL; VIR_DEBUG("filters=%s", src); - if (!(strings = virStringSplitCount(src, " ", 0, &count))) + if (!(strings = g_strsplit(src, " ", 0))) return -1; - for (i = 0; i < count; i++) { + for (next = strings; *next; next++) { /* virStringSplit may return empty strings */

- if (STREQ(strings[i], "")) + if (STREQ(*next, "")) continue; - if (!(filter = virLogParseFilter(strings[i]))) + if (!(filter = virLogParseFilter(*next))) return -1;