pushed: [RFA 1/2] New GDB/MI command "-info-gdb-mi-command"

Joel Brobecker
2013-12-03T04:06:21Z

> > gdb/ChangeLog:
> >
> > * mi/mi-cmds.h (mi_cmd_info_gdb_mi_command): Declare.
> > * mi/mi-cmd-info.c (mi_cmd_info_gdb_mi_command): New function.
> > * mi/mi-cmds.c (mi_cmds): Add -info-gdb-mi-command command.
> > * mi/mi-main.c (mi_cmd_list_features): Add "info-gdb-mi-command"
> > field to output of "-list-features".
> >
> > * NEWS: Add entry for new -info-gdb-mi-command.
> >
> > gdb/doc/ChangeLog:
> >
> > * gdb.texinfo (GDB/MI Miscellaneous Commands): Document
> > the new -info-gdb-mi-command GDB/MI command. Document
> > the meaning of "-info-gdb-mi-command" in the output of
> > -list-features.
> >
> > gdb/testsuite/ChangeLog:
> >
> > * gdb.mi/mi-i-cmd.exp: New file.
> >
> > Re-tested on x86_64-linux. OK to commit?
> >
>
> Looks fine to me.
>
> (I don't have a strong opinion about the dash issue.)
>
> > +# First, verify that the debugger correctly advertises support
> > +# for the -info-gdb-mi-command command.
> > +mi_gdb_test "-list-features" \
> > + "\\^done,features=\\\[.*\"info-gdb-mi-command\".*\\\]" \
> > + "-list-features should include \"info-gdb-mi-command\""
>
> Nit, I'd suggest:
>
> "-list-features includes \"info-gdb-mi-command\""
>

Thanks, Pedro. The suggestion does seem better to me too, so I applied
it. Patch now in.

--
Joel