On a Thursday in 2021, Peter Krempa wrote:
Rewrite the code to remove the need to calculate the string list count.
Signed-off-by: Peter Krempa <pkrempa@xxxxxxxxxx>
---
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 */
This comment is now outdated: the sequel
- if (STREQ(strings[i], ""))
+ if (STREQ(*next, ""))
continue;
- if (!(filter = virLogParseFilter(strings[i])))
+ if (!(filter = virLogParseFilter(*next)))
return -1;
Reviewed-by: Ján Tomko <jtomko@xxxxxxxxxx> Jano
Attachment:
signature.asc
Description: PGP signature
