Re: [RFCv2 01/16] stringlist: add from_space_separated_string

From: Duy Nguyen <pclouds@gmail.com>
Date: Jun 2, 2015, 09:42 UTC
Message-ID: <CACsJy8BEEvgP_YzVnbPaLL9QBs48sd2fn6CeicXD_9U92=9UnA@mail.gmail.com>
In-Reply-To: <1433203338-27493-2-git-send-email-sbeller@google.com>

On Tue, Jun 2, 2015 at 7:02 AM, Stefan Beller <sbeller@google.com> wrote:

Show 9 quoted lines

> diff --git a/string-list.h b/string-list.h
> index d3809a1..88c18e9 100644
> --- a/string-list.h
> +++ b/string-list.h
> @@ -19,6 +19,7 @@ struct string_list {
>  #define STRING_LIST_INIT_DUP   { NULL, 0, 0, 1, NULL }
>
>  void string_list_init(struct string_list *list, int strdup_strings);
> +void from_space_separated_string(struct string_list *list, char *line);

The name feels out of place. All functions in here have "string_list" somewhere in their names. The implementation looks very close to string_list_split() but that name's already taken.. Maybe string_list_split_by_space()?

-- 
Duy