On Mon, Jan 14, 2013 at 10:08:47AM -0800, Junio C Hamano wrote:
> > +extern int match_config_key(const char *var,
> > + const char *section,
> > + const char **subsection, int *subsection_len,
> > + const char **key);
> > +
> I agree with Jonathan about the naming s/match/parse/.
I see this is marked for re-roll in WC. I'm happy to re-roll it with the
suggestions from Jonathan, but before I do, did you have any comment on
the "struct config_key" alternative I sent as a follow-up?
> After looking at the callers in your later patches, I think the
> counted interface to subsection is probably fine. The caller can
> check !subsection to see if it is a two- or three- level name, and
> if (parse_config_key(var, "submodule", &name, &namelen, &key) < 0 ||
> return 0;
> is very easy to follow (that is the result of your 5th step).
but I wasn't sure if that was "it is not worth the trouble of the other
one" or "I did not yet read the other one".
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html