Hi Junio, Junio C Hamano wrote: > Ramkumar Ramachandra <artagnon@xxxxxxxxx> writes: > >> I found this patch in one of my branches from a long time ago. I >> haven't written documentation and tests yet because I'm not fully >> convinced that I like this. >> >> Thoughts? > > It looks like a static version of string_list_lookup() and does not > have much reason to tie it to "subcommand". How so? I can use a string_list to keep subcommand->name and stuff the callback into util. Where do I put subcommand->help then? I agree that there's no reason to tie it to subcommand though. Ram -- 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