Re: [RFC/PATCH] parse-options: introduce parse_subcommands

Ramkumar Ramachandra [off-list ref] 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".

Hi Junio,

Junio C Hamano wrote:

Ramkumar Ramachandra [off-list ref] writes:

quoted

 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