>>>>> "Mike" == Mike Frysinger <[email protected]> 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
_______________________________________________
Toolchain-devel mailing list
[email protected]
https://blackfin.uclinux.org/mailman/listinfo/toolchain-devel

Reply via email to