diff mbox

[6/8] qemu-img: img_create(): simplify

Message ID 1350502556-4885-7-git-send-email-lcapitulino@redhat.com
State New
Headers show

Commit Message

Luiz Capitulino Oct. 17, 2012, 7:35 p.m. UTC 
Drop unneeded goto and ret variable.

Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com>
---
 qemu-img.c | 15 ++++-----------
 1 file changed, 4 insertions(+), 11 deletions(-)
diff mbox

Patch

diff --git a/qemu-img.c b/qemu-img.c
index dfde588..0634402 100644
--- a/qemu-img.c
+++ b/qemu-img.c
@@ -294,7 +294,7 @@  static int add_old_style_options(const char *fmt, QEMUOptionParameter *list,
 
 static int img_create(int argc, char **argv)
 {
-    int c, ret = 0;
+    int c;
     uint64_t img_size = -1;
     const char *fmt = "raw";
     const char *base_fmt = NULL;
@@ -352,15 +352,13 @@  static int img_create(int argc, char **argv)
             error_report("Invalid image size specified! You may use k, M, G or "
                   "T suffixes for ");
             error_report("kilobytes, megabytes, gigabytes and terabytes.");
-            ret = -1;
-            goto out;
+            return 1;
         }
         img_size = (uint64_t)sval;
     }
 
     if (options && is_help_option(options)) {
-        ret = print_block_option_help(filename, fmt);
-        goto out;
+        return print_block_option_help(filename, fmt);
     }
 
     bdrv_img_create(filename, fmt, base_filename, base_fmt,
@@ -368,8 +366,7 @@  static int img_create(int argc, char **argv)
     if (error_is_set(&local_err)) {
         fprintf(stderr, "qemu-img: %s\n", error_get_pretty(local_err));
         error_free(local_err);
-        ret = -1;
-        goto out;
+        return 1;
     }
 
     assert(params);
@@ -378,10 +375,6 @@  static int img_create(int argc, char **argv)
     free_option_parameters(params);
     puts("");
 
-out:
-    if (ret) {
-        return 1;
-    }
     return 0;
 }