diff options
author | Markus Armbruster <armbru@redhat.com> | 2018-08-06 08:53:32 +0200 |
---|---|---|
committer | Markus Armbruster <armbru@redhat.com> | 2018-08-16 08:42:06 +0200 |
commit | 62fff696d56b90e5820d2c3c3085b778b23f0d93 (patch) | |
tree | a09f5b7ba04860e5d118052a347c38cd5034138e /tests/ahci-test.c | |
parent | 2d36e843042d2ef47f3bfc47a1a83401fdb07b84 (diff) |
tests: Pass literal format strings directly to qmp_FOO()
The qmp_FOO() take a printf-like format string. In a few places, we
assign a string literal to a variable and pass that instead of simply
passing the literal. Clean that up.
Bonus: gets rid of non-literal format strings. A step towards
compile-time format string checking without triggering
-Wformat-nonliteral.
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20180806065344.7103-12-armbru@redhat.com>
Diffstat (limited to 'tests/ahci-test.c')
-rw-r--r-- | tests/ahci-test.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/tests/ahci-test.c b/tests/ahci-test.c index 555f02112e..60ba32f4f2 100644 --- a/tests/ahci-test.c +++ b/tests/ahci-test.c @@ -1355,7 +1355,6 @@ static void test_flush_migrate(void) AHCIQState *src, *dst; AHCICommand *cmd; uint8_t px; - const char *s; char *uri = g_strdup_printf("unix:%s", mig_socket); prepare_blkdebug_script(debug_path, "flush_to_disk"); @@ -1391,8 +1390,7 @@ static void test_flush_migrate(void) ahci_migrate(src, dst, uri); /* Complete the command */ - s = "{'execute':'cont' }"; - qmp_send(s); + qmp_send("{'execute':'cont' }"); qmp_eventwait("RESUME"); ahci_command_wait(dst, cmd); ahci_command_verify(dst, cmd); |