Patchwork [21/21] Monitor: Convert do_screen_dump() to QObject

login
register
mail settings
Submitter Luiz Capitulino
Date April 5, 2010, 8:34 p.m.
Message ID <1270499642-31543-22-git-send-email-lcapitulino@redhat.com>
Download mbox | patch
Permalink /patch/49459/
State New
Headers show

Comments

Luiz Capitulino - April 5, 2010, 8:34 p.m.
Trivial, as it never fails, doesn't have output nor return
any data.

Note that it's also available under QMP, as kvm-autotest
needs this.

Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com>
---
 monitor.c       |    3 ++-
 qemu-monitor.hx |    3 ++-
 2 files changed, 4 insertions(+), 2 deletions(-)

Patch

diff --git a/monitor.c b/monitor.c
index c5a4dbf..709b326 100644
--- a/monitor.c
+++ b/monitor.c
@@ -1164,9 +1164,10 @@  static int do_change(Monitor *mon, const QDict *qdict, QObject **ret_data)
     return ret;
 }
 
-static void do_screen_dump(Monitor *mon, const QDict *qdict)
+static int do_screen_dump(Monitor *mon, const QDict *qdict, QObject **ret_data)
 {
     vga_hw_screen_dump(qdict_get_str(qdict, "filename"));
+    return 0;
 }
 
 static void do_logfile(Monitor *mon, const QDict *qdict)
diff --git a/qemu-monitor.hx b/qemu-monitor.hx
index 62fa346..5ea5748 100644
--- a/qemu-monitor.hx
+++ b/qemu-monitor.hx
@@ -199,7 +199,8 @@  ETEXI
         .args_type  = "filename:F",
         .params     = "filename",
         .help       = "save screen into PPM image 'filename'",
-        .mhandler.cmd = do_screen_dump,
+        .user_print = monitor_user_noop,
+        .mhandler.cmd_new = do_screen_dump,
     },
 
 STEXI