This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] gdb: sim: add command line completion


>>>>> "Mike" == Mike Frysinger <vapier@gentoo.org> writes:

Mike> For now, only the sub-command name is completed.  No support yet for
Mike> completing options to that command.  But even this is a huge step as
Mike> currently, nothing is completed, and the basic "help sim" is fairly
Mike> obtuse as to what exactly the "sim" command accepts.

Thanks for doing this.

Mike> +char **sim_command_completer (struct cmd_list_element *ignore,
Mike> +			      char *text, char *word)

I think this could be static.
And, newline after the initial "char **".

I don't really know anything about sim, but I didn't see anything
obviously bad in there.  So, ok with the above change.

Tom


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]