Subject: Re: [PATCH v2 16/21] refs: check submodules ref storage config

From: Junio C Hamano <[email-masked]>
Date: Wed, 13 Jan 2016 09:02:20 -0800

David Turner <[email-masked]> writes:

>> > +static int submodule_backend(const char *key, const char *value,
>> > void *data)
>> > +{
>> > +	char **path = data;
>> > +	if (!strcmp(key, "extensions.refstorage"))
>> > +		*path = xstrdup(value);
>> > +}
>> 
>> This should return "int" (namely 0).
>
> Fixed, thanks.

By the way, I initially thought "this function should return void",
until I realized that this is a callback function that is required
to have a fixed function signature.  It would save future readers a
similar confusion if it is named with "config" somewhere and/or
"_cb" at the end.

Thanks.