[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 3/5] monitor: Port do_info() to QObject
From: |
Luiz Capitulino |
Subject: |
[Qemu-devel] [PATCH 3/5] monitor: Port do_info() to QObject |
Date: |
Thu, 3 Sep 2009 11:24:17 -0300 |
do_info() is special, its job is to call 'info handlers'. This
is similar to what monitor_handle_command() does, therefore
do_info() also has to distinguish among new and old style
info handlers.
This commit ports do_info() to call info handlers according to
the new QObject style.
It's important to note that:
1. the return value of do_info() will be the same of the called
handler
2. monitor_print_nothing() will be used by handlers to...
print nothing :)
Signed-off-by: Luiz Capitulino <address@hidden>
---
monitor.c | 45 +++++++++++++++++++++++++++++++++------------
qemu-monitor.hx | 2 +-
2 files changed, 34 insertions(+), 13 deletions(-)
diff --git a/monitor.c b/monitor.c
index 9886bb0..f9f3cbd 100644
--- a/monitor.c
+++ b/monitor.c
@@ -208,6 +208,11 @@ static int monitor_fprintf(FILE *stream, const char *fmt,
...)
return 0;
}
+static void monitor_print_nothing(Monitor *mon, const QObject *data)
+{
+ return;
+}
+
static int compare_cmd(const char *name, const char *list)
{
const char *p, *pstart;
@@ -277,24 +282,40 @@ static void do_commit(Monitor *mon, const QDict *qdict)
}
}
-static void do_info(Monitor *mon, const QDict *qdict)
+static int do_info(Monitor *mon, const QDict *qdict, QObject **ret_data)
{
+ int ret;
const mon_cmd_t *cmd;
const char *item = qdict_get_try_str(qdict, "item");
- void (*handler)(Monitor *);
- if (!item)
- goto help;
- for(cmd = info_cmds; cmd->name != NULL; cmd++) {
+ if (!item) {
+ help_cmd(mon, "info");
+ return 0;
+ }
+
+ for (cmd = info_cmds; cmd->name != NULL; cmd++) {
if (compare_cmd(item, cmd->name))
- goto found;
+ break;
}
- help:
- help_cmd(mon, "info");
- return;
- found:
- handler = cmd->handler;
- handler(mon);
+
+ if (cmd->name == NULL)
+ return -1;
+
+ ret = 0;
+ if (cmd->user_print) {
+ int (*handler_new)(Monitor *mon, QObject **ret_data);
+
+ handler_new = cmd->handler;
+ ret = handler_new(mon, ret_data);
+
+ cmd->user_print(mon, *ret_data);
+ } else {
+ void (*handler_old)(Monitor *);
+ handler_old = cmd->handler;
+ handler_old(mon);
+ }
+
+ return ret;
}
static void do_info_version(Monitor *mon)
diff --git a/qemu-monitor.hx b/qemu-monitor.hx
index a6d0f2e..44402c5 100644
--- a/qemu-monitor.hx
+++ b/qemu-monitor.hx
@@ -22,7 +22,7 @@ STEXI
Commit changes to the disk images (if -snapshot is used) or backing files.
ETEXI
- { "info", "item:s?", do_info, NULL,
+ { "info", "item:s?", do_info, monitor_print_nothing,
"[subcommand]", "show various information about the system state" },
STEXI
@item info @var{subcommand}
--
1.6.4.2.253.g0b1fac
- [Qemu-devel] [RFC 0/5] Monitor handlers convertion to QObject, Luiz Capitulino, 2009/09/03
- [Qemu-devel] [PATCH 1/5] monitor: Add user_print() to mon_cmd_t, Luiz Capitulino, 2009/09/03
- [Qemu-devel] [PATCH 2/5] monitor: Handle new and old style handlers, Luiz Capitulino, 2009/09/03
- [Qemu-devel] [PATCH 3/5] monitor: Port do_info() to QObject,
Luiz Capitulino <=
- [Qemu-devel] [PATCH 4/5] monitor: Port do_info_balloon() to QObject, Luiz Capitulino, 2009/09/03
- [Qemu-devel] [PATCH 5/5] monitor: Port do_balloon() to QObject, Luiz Capitulino, 2009/09/03
- Re: [Qemu-devel] [RFC 0/5] Monitor handlers convertion to QObject, Gerd Hoffmann, 2009/09/03