Subject: Re: [PATCH 4/7] submodule-config: check if a submodule is in a group

From: Junio C Hamano <[email-masked]>
Date: Tue, 10 May 2016 18:19:17 -0700

Stefan Beller <[email-masked]> writes:

> +static int in_group(int argc, const char **argv, const char *prefix)
> + ...
> +	if (!group)
> +		list = git_config_get_value_multi("submodule.updateGroup");
> +	else {
> +		string_list_split(&actual_list, group, ',', -1);

Is this expected to be used only by test scripts, or will it have
real scripted Porcelain as its users?  I am wondering if
concatenation with ' ' would be more natural if it is the latter (if
only used for testing, I don't care that much).