[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH V5 4/7] monitor: avoid direct use of global *info_cm
From: |
Wenchao Xia |
Subject: |
[Qemu-devel] [PATCH V5 4/7] monitor: avoid direct use of global *info_cmds in help functions |
Date: |
Sat, 29 Jun 2013 11:52:58 +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>
---
monitor.c | 63 +++++++++++++++++++++++++++++++++++++++++++++++++++---------
1 files changed, 53 insertions(+), 10 deletions(-)
diff --git a/monitor.c b/monitor.c
index 03a017d..bc62fc7 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 = nb_args < MAX_ARGS ? 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 V5 0/7] monitor: support sub command group in auto completion and help, Wenchao Xia, 2013/06/28
- [Qemu-devel] [PATCH V5 1/7] monitor: avoid direct use of global *cur_mon in completion functions, Wenchao Xia, 2013/06/28
- [Qemu-devel] [PATCH V5 2/7] monitor: avoid direct use of global variable *mon_cmds, Wenchao Xia, 2013/06/28
- [Qemu-devel] [PATCH V5 3/7] monitor: code move for parse_cmdline(), Wenchao Xia, 2013/06/28
- [Qemu-devel] [PATCH V5 5/7] monitor: support sub commands in auto completion, Wenchao Xia, 2013/06/28
- [Qemu-devel] [PATCH V5 4/7] monitor: avoid direct use of global *info_cmds in help functions,
Wenchao Xia <=
- [Qemu-devel] [PATCH V5 6/7] monitor: improve "help" in auto completion for sub command, Wenchao Xia, 2013/06/28
- [Qemu-devel] [PATCH V5 7/7] monitor: improve "help" to allow show details of single command in sub group, Wenchao Xia, 2013/06/28