diff options
author | Markus Armbruster <armbru@redhat.com> | 2018-08-06 08:53:23 +0200 |
---|---|---|
committer | Markus Armbruster <armbru@redhat.com> | 2018-08-16 08:42:06 +0200 |
commit | f94648fe3d298354d9b9fa03d8eaf622d92d781f (patch) | |
tree | 700be7ebd7f0c9eeae1369235acb3dd02cf6d761 /tests/libqtest.c | |
parent | 4277f1ebd9b70ee38caddfe3c514d7a91e676bc1 (diff) | |
download | qemu-f94648fe3d298354d9b9fa03d8eaf622d92d781f.zip qemu-f94648fe3d298354d9b9fa03d8eaf622d92d781f.tar.gz qemu-f94648fe3d298354d9b9fa03d8eaf622d92d781f.tar.bz2 |
libqtest: Clean up how we read device_del messages
qtest_qmp_device_del() still uses the qmp("") hack to receive a
message, even though we have qmp_receive() since commit 66e0c7b187e.
Put it to use.
Bonus: gets rid of empty format strings. A step towards compile-time
format string checking without triggering -Wformat-zero-length.
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20180806065344.7103-3-armbru@redhat.com>
Diffstat (limited to 'tests/libqtest.c')
-rw-r--r-- | tests/libqtest.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/libqtest.c b/tests/libqtest.c index 9cb4096..071d7eb 100644 --- a/tests/libqtest.c +++ b/tests/libqtest.c @@ -1090,7 +1090,7 @@ void qtest_qmp_device_del(const char *id) g_assert(response1); g_assert(!qdict_haskey(response1, "error")); - response2 = qmp(""); + response2 = qmp_receive(); g_assert(response2); g_assert(!qdict_haskey(response2, "error")); |