[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH V4 4/7] monitor: avoid direct use of global *info_cm
From: |
Wenchao Xia |
Subject: |
[Qemu-devel] [PATCH V4 4/7] monitor: avoid direct use of global *info_cmds in help functions |
Date: |
Fri, 28 Jun 2013 11:39:28 +0800 |
In help functions info_cmds is treated as sub command group now, not as
a special case any more. Still help can't show message for single command
under "info", since command parser reject additional parameter, which
can be improved by change "help" item parameter define later. "log" is
still treated as special help case. compare_cmd() is used instead of strcmp()
in command searching.
Signed-off-by: Wenchao Xia <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
---
monitor.c | 63 +++++++++++++++++++++++++++++++++++++++++++++++++++---------
1 files changed, 53 insertions(+), 10 deletions(-)
diff --git a/monitor.c b/monitor.c
index 3d16592..7cfda75 100644
--- a/monitor.c
+++ b/monitor.c
@@ -831,33 +831,76 @@ static void parse_cmdline(const char *cmdline,
*pnb_args = nb_args;
}
+static void help_cmd_dump_one(Monitor *mon,
+ const mon_cmd_t *cmd,
+ char **prefix_args,
+ int prefix_args_nb)
+{
+ int i;
+
+ for (i = 0; i < prefix_args_nb; i++) {
+ monitor_printf(mon, "%s ", prefix_args[i]);
+ }
+ monitor_printf(mon, "%s %s -- %s\n", cmd->name, cmd->params, cmd->help);
+}
+
static void help_cmd_dump(Monitor *mon, const mon_cmd_t *cmds,
- const char *prefix, const char *name)
+ char **args, int nb_args, int arg_index)
{
const mon_cmd_t *cmd;
- for(cmd = cmds; cmd->name != NULL; cmd++) {
- if (!name || !strcmp(name, cmd->name))
- monitor_printf(mon, "%s%s %s -- %s\n", prefix, cmd->name,
- cmd->params, cmd->help);
+ /* Dump all */
+ if (arg_index >= nb_args) {
+ for (cmd = cmds; cmd->name != NULL; cmd++) {
+ help_cmd_dump_one(mon, cmd, args, arg_index);
+ }
+ return;
+ }
+
+ /* Find one entry to dump */
+ for (cmd = cmds; cmd->name != NULL; cmd++) {
+ if (compare_cmd(args[arg_index], cmd->name)) {
+ if (cmd->sub_table) {
+ help_cmd_dump(mon, cmd->sub_table,
+ args, nb_args, arg_index + 1);
+ } else {
+ help_cmd_dump_one(mon, cmd, args, arg_index);
+ }
+ break;
+ }
}
}
static void help_cmd(Monitor *mon, const char *name)
{
- if (name && !strcmp(name, "info")) {
- help_cmd_dump(mon, info_cmds, "info ", NULL);
- } else {
- help_cmd_dump(mon, mon->cmd_table, "", name);
- if (name && !strcmp(name, "log")) {
+ char *args[MAX_ARGS];
+ int nb_args = 0, i;
+
+ if (name) {
+ /* special case for log */
+ if (!strcmp(name, "log")) {
const QEMULogItem *item;
monitor_printf(mon, "Log items (comma separated):\n");
monitor_printf(mon, "%-10s %s\n", "none", "remove all logs");
for (item = qemu_log_items; item->mask != 0; item++) {
monitor_printf(mon, "%-10s %s\n", item->name, item->help);
}
+ return;
+ }
+
+ parse_cmdline(name, &nb_args, args);
+ if (nb_args >= MAX_ARGS) {
+ goto cleanup;
}
}
+
+ help_cmd_dump(mon, mon->cmd_table, args, nb_args, 0);
+
+cleanup:
+ nb_args = min(nb_args, MAX_ARGS);
+ for (i = 0; i < nb_args; i++) {
+ g_free(args[i]);
+ }
}
static void do_help_cmd(Monitor *mon, const QDict *qdict)
--
1.7.1
- [Qemu-devel] [PATCH V4 0/7] monitor: support sub command group in auto completion and help, Wenchao Xia, 2013/06/27
- [Qemu-devel] [PATCH V4 1/7] monitor: avoid direct use of global *cur_mon in completion functions, Wenchao Xia, 2013/06/27
- [Qemu-devel] [PATCH V4 2/7] monitor: avoid direct use of global variable *mon_cmds, Wenchao Xia, 2013/06/27
- [Qemu-devel] [PATCH V4 4/7] monitor: avoid direct use of global *info_cmds in help functions,
Wenchao Xia <=
- [Qemu-devel] [PATCH V4 5/7] monitor: support sub commands in auto completion, Wenchao Xia, 2013/06/27
- [Qemu-devel] [PATCH V4 3/7] monitor: code move for parse_cmdline(), Wenchao Xia, 2013/06/27
- [Qemu-devel] [PATCH V4 6/7] monitor: improve "help" in auto completion for sub command, Wenchao Xia, 2013/06/27
- [Qemu-devel] [PATCH V4 7/7] monitor: improve "help" to allow show details of single command in sub group, Wenchao Xia, 2013/06/27
- Re: [Qemu-devel] [PATCH V4 0/7] monitor: support sub command group in auto completion and help, Wenchao Xia, 2013/06/28