This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [RFA] Implement -thread-info.
- From: Nick Roberts <nickrob at snap dot net dot nz>
- To: Daniel Jacobowitz <drow at false dot org>
- Cc: Vladimir Prus <vladimir at codesourcery dot com>, gdb-patches at sources dot redhat dot com, Eli Zaretskii <eliz at gnu dot org>
- Date: Sun, 23 Mar 2008 16:55:55 +1200
- Subject: Re: [RFA] Implement -thread-info.
- References: <200802171833.26673.vladimir@codesourcery.com> <200802181038.04497.vladimir@codesourcery.com> <18361.64577.771676.184618@kahikatea.snap.net.nz> <200802201427.36288.vladimir@codesourcery.com> <18371.25340.162317.39030@kahikatea.snap.net.nz> <20080226005822.GA3346@caradoc.them.org>
Daniel Jacobowitz writes:
> On Tue, Feb 26, 2008 at 01:53:16PM +1300, Nick Roberts wrote:
> > + { "thread-info", { "info threads", 1 }, NULL, NULL },
>
> I would suggest against using args_p == 1 for any new commands. The
> quoting rules between CLI and MI are different; we ignore that for
> some commands, but it's a bug and it bites frontend developers.
>
> If the arguments are just going to be integers then it doesn't make
> much difference but it will make things simpler if we want to add
> either quoted arguments or dashed options later.
With non-stop mode there will presumably be many new MI commands. Providing
the same functionality in CLI duplicates a lot of effort. Perhaps the parsing
of MI and CLI arguments should be unified.
--
Nick http://www.inet.net.nz/~nickrob