Stefan Beller <sbeller@xxxxxxxxxx> 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). -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html