Re: [libvirt] [PATCH] virstring: Unify string list function names

We have couple of functions that operate over NULL terminated lits of strings. However, our naming sucks:   virStringJoin virStringFreeList virStringFreeListCount virStringArrayHasString virStringGetFirstWithPrefix   We can do better:   virStringListJoin virStringListFree virStringListFreeCount virStringListHasString virStringListGetFirstWithPrefix   Signed-off-by: Michal Privoznik <mprivozn(a)redhat.com&gt;

@@ -227,7 +227,7 @@ virStringArrayHasString(const char **strings,   }      char * -virStringGetFirstWithPrefix(char **strings, const char *prefix) +virStringListGetFirstWithPrefix(char **strings, const char *prefix)

@@ -37,15 +37,15 @@ char **virStringSplit(const char *string,                         size_t max_tokens)       ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(2);    -char *virStringJoin(const char **strings, -                    const char *delim) +char *virStringListJoin(const char **strings, +                        const char *delim)       ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(2);    -void virStringFreeList(char **strings); -void virStringFreeListCount(char **strings, size_t count); +void virStringListFree(char **strings); +void virStringListFreeCount(char **strings, size_t count);    -bool virStringArrayHasString(const char **strings, const char *needle); -char *virStringGetFirstWithPrefix(char **strings, const char *prefix) +bool virStringListHasString(const char **strings, const char *needle); +char *virStringListGetFirstWithPrefix(char **strings, const char *prefix)       ATTRIBUTE_NONNULL(2);