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

Peter Krempa pkrempa at redhat.com
Thu Apr 1 13:52:13 UTC 2021


Rewrite the code to remove the need to calculate the string list count.

Signed-off-by: Peter Krempa <pkrempa at redhat.com>
---
 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;

         if (VIR_APPEND_ELEMENT(list, nfilters, filter)) {
-- 
2.29.2




More information about the libvir-list mailing list