builtin/config.c: Fix a sparse warning

Ramsay Jones · peff · commit 5ba1a8a735c7 · 2012-10-29T01:48:06.000-04:00
Sparse issues an "Using plain integer as NULL pointer" warning while
checking a 'struct strbuf_list' initializer expression. The initial
field of the struct has pointer type, but the initializer expression
is given as '{0}'. In order to suppress the warning, we simply replace
the initializer with '{NULL}'.

Signed-off-by: Ramsay Jones &lt;ramsay@ramsay1.demon.co.uk&gt;
Signed-off-by: Jeff King &lt;peff@peff.net&gt;
diff --git a/builtin/config.c b/builtin/config.c
@@ -165,7 +165,7 @@ static int collect_config(const char *key_, const char *value_, void *cb)
 static int get_value(const char *key_, const char *regex_)
 {
 	int ret = CONFIG_GENERIC_ERROR;
-	struct strbuf_list values = {0};
+	struct strbuf_list values = {NULL};
 	int i;
 
 	if (use_key_regexp) {

Original file line number	Diff line number	Diff line change
`@@ -165,7 +165,7 @@ static int collect_config(const char key_, const char value_, void *cb)`
`165`	`165`	`static int get_value(const char key_, const char regex_)`
`166`	`166`	`{`
`167`	`167`	`int ret = CONFIG_GENERIC_ERROR;`
`168`		`- struct strbuf_list values = {0};`
	`168`	`+ struct strbuf_list values = {NULL};`
`169`	`169`	`int i;`
`170`	`170`
`171`	`171`	`if (use_key_regexp) {`