diff options
| author | Markus Armbruster <armbru@redhat.com> | 2018-08-06 08:53:41 +0200 |
|---|---|---|
| committer | Markus Armbruster <armbru@redhat.com> | 2018-08-16 08:42:06 +0200 |
| commit | c44a56d8ba1c13fea5c9d045ac178638807079cc (patch) | |
| tree | 5a6d7b5a751bbd6b75b49040713f54e3b1146bcc /tests/test-util-sockets.c | |
| parent | b5bbd3f315d686bd511f5be72e4aab3b2bd52de8 (diff) | |
| download | qemu-c44a56d8ba1c13fea5c9d045ac178638807079cc.zip qemu-c44a56d8ba1c13fea5c9d045ac178638807079cc.tar.gz qemu-c44a56d8ba1c13fea5c9d045ac178638807079cc.tar.bz2 | |
migration-test: Clean up string interpolation into QMP, part 3
Leaving interpolation into JSON to qmp() is more robust than building
QMP input manually, as explained in the recent commit "tests: Clean up
string interpolation into QMP input (simple cases)".
migration-test.c interpolates strings into JSON in a few places:
* migrate_set_parameter() interpolates string parameter @value as a
JSON number. Change it to long long. This requires changing
migrate_check_parameter() similarly.
* migrate_set_capability() interpolates string parameter @value as a
JSON boolean. Change it to bool.
* deprecated_set_speed() interpolates string parameter @value as a
JSON number. Change it to long long.
Bonus: gets rid of non-literal format strings. A step towards
compile-time format string checking without triggering
-Wformat-nonliteral.
Cc: Juan Quintela <quintela@redhat.com>
Cc: Dr. David Alan Gilbert <dgilbert@redhat.com>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Juan Quintela <quintela@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20180806065344.7103-21-armbru@redhat.com>
Diffstat (limited to 'tests/test-util-sockets.c')
0 files changed, 0 insertions, 0 deletions