From acb3d22264fd3736f95f71aa38e1d6fa01f5e9e1 Mon Sep 17 00:00:00 2001 From: Tanay Abhra Date: Tue, 3 Jun 2014 02:13:18 -0700 Subject: string-list: spell all values out that are given to a string_list initializer STRING_LIST_INIT_{NODUP,DUP} initializers list values only for earlier structure members, relying on the usual convention in C that the omitted members are initailized to 0, i.e. the former is expanded to the latter: struct string_list l = STRING_LIST_INIT_DUP; struct string_list l = { NULL, 0, 0, 1 }; and the last member that is not mentioned (i.e. 'cmp') is initialized to NULL. While there is nothing wrong in this construct, spelling out all the values where the macros are defined will serve also as a documentation, so let's do so. Signed-off-by: Tanay Abhra Signed-off-by: Junio C Hamano --- string-list.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'string-list.h') diff --git a/string-list.h b/string-list.h index de6769c92d..dd5e294465 100644 --- a/string-list.h +++ b/string-list.h @@ -15,8 +15,8 @@ struct string_list { compare_strings_fn cmp; /* NULL uses strcmp() */ }; -#define STRING_LIST_INIT_NODUP { NULL, 0, 0, 0 } -#define STRING_LIST_INIT_DUP { NULL, 0, 0, 1 } +#define STRING_LIST_INIT_NODUP { NULL, 0, 0, 0, NULL } +#define STRING_LIST_INIT_DUP { NULL, 0, 0, 1, NULL } void print_string_list(const struct string_list *p, const char *text); void string_list_clear(struct string_list *list, int free_util); -- cgit v1.3