commit a8b18f8fd2d9f0df33c3739e76fedfd9355dfa8b
author Markus Armbruster <armbru@redhat.com> Fri Feb 13 15:50:43 2015 +0100
committer Markus Armbruster <armbru@redhat.com> Thu Feb 26 14:51:46 2015 +0100
tree 5897e1319aa3f3d4433c0e13065a6c8a8ab6024f
parent dc523cd348c47372faa7271c9aab2030f94c290d

block: Simplify setting numeric options

Don't convert numbers to strings for use with qemu_opt_set(), simply
use qemu_opt_set_number() instead.

Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>

blockdev.c

diff --git a/blockdev.c b/blockdev.c
index 5aa23b5..b9c1c0c 100644
--- a/blockdev.c
+++ b/blockdev.c
@@ -180,7 +180,6 @@
                     const char *optstr)
 {
     QemuOpts *opts;
-    char buf[32];
 
     opts = drive_def(optstr);
     if (!opts) {
@@ -190,8 +189,7 @@
         qemu_opt_set(opts, "if", if_name[type], &error_abort);
     }
     if (index >= 0) {
-        snprintf(buf, sizeof(buf), "%d", index);
-        qemu_opt_set(opts, "index", buf, &error_abort);
+        qemu_opt_set_number(opts, "index", index, &error_abort);
     }
     if (file)
         qemu_opt_set(opts, "file", file, &error_abort);