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>

vl.c

diff --git a/vl.c b/vl.c
index 66b0ad8..e2c1950 100644
--- a/vl.c
+++ b/vl.c
@@ -2971,13 +2971,12 @@
                         exit(1);
                     }
                     if (hda_opts != NULL) {
-                        char num[16];
-                        snprintf(num, sizeof(num), "%d", cyls);
-                        qemu_opt_set(hda_opts, "cyls", num, &error_abort);
-                        snprintf(num, sizeof(num), "%d", heads);
-                        qemu_opt_set(hda_opts, "heads", num, &error_abort);
-                        snprintf(num, sizeof(num), "%d", secs);
-                        qemu_opt_set(hda_opts, "secs", num, &error_abort);
+                        qemu_opt_set_number(hda_opts, "cyls", cyls,
+                                            &error_abort);
+                        qemu_opt_set_number(hda_opts, "heads", heads,
+                                            &error_abort);
+                        qemu_opt_set_number(hda_opts, "secs", secs,
+                                            &error_abort);
                         if (translation == BIOS_ATA_TRANSLATION_LARGE) {
                             qemu_opt_set(hda_opts, "trans", "large",
                                          &error_abort);