[PATCH] builtin/config.c: Fix a sparse warning
- From
Ramsay Jones <ramsay@ramsay1.demon.co.uk>
- Date
- Oct 28, 2012, 21:05 UTC
- Message-ID
- <508D9E15.1040105@ramsay1.demon.co.uk>
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 <ramsay@ramsay1.demon.co.uk> ---
Hi Jeff,
I try to catch these types of warnings while the patches are still in the pu branch. I don't know how I missed this one, but I don't remember your 'jk/config-ignore-duplicates' branch being in pu.
Sorry about that.
ATB, Ramsay Jones
builtin/config.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/builtin/config.c b/builtin/config.c index f881053..e796af4 100644 --- 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) {
-- 1.8.0