Re: [PATCH] Add the "-info-os" command to MI

[Stan Shebs <stanshebs at earthlink dot net>]

On 10/21/11 1:39 PM, Tom Tromey wrote:
> > > > > > ">" == Kwok Cheung Yeung<kcy@codesourcery.com>  writes:
> > > +  { "info-os", { "info os", 1 }, NULL },
> I'd prefer a new function specific to MI to do the command-line parsing.
>
> I think this approach should be deprecated and that no new instances go
> into the tree.  They unnecessarily link MI to the CLI, making it harder
> to change the CLI.

I can see the appeal of the decoupling, but we share argument parsing 
like this because there's not a lot of practical purpose to having two 
parsers - once the command is out there and being used, we're loath to 
change it much anyway.  To me, the MI's main value is in avoiding cases 
where CLI ambiguity and elision makes for a better user experience, but 
in this case the arguments are data-driven by platform-specific code, 
not much ambiguity involved.

Stan