qemu-io: Make alloc output useful when nb_sectors=1
authorStefan Hajnoczi <stefanha@linux.vnet.ibm.com>
Thu, 2 Sep 2010 09:38:03 +0000 (10:38 +0100)
committerKevin Wolf <kwolf@redhat.com>
Wed, 8 Sep 2010 10:39:14 +0000 (12:39 +0200)
There is no indication whether or not the sector is allocated when
nb_sectors=1:

  sector allocated at offset 64 KiB

This message is produced whether or not the sector is allocated.

Simply use the same message as the plural case, I don't think the
English is so broken that we need special case output here:

  0/1 sectors allocated at offset 64 KiB

This change does not affect qemu-iotests since nb_sectors=1 is not used
there.

Signed-off-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
qemu-io.c

index 2dbe20f..bd3bd16 100644 (file)
--- a/qemu-io.c
+++ b/qemu-io.c
@@ -1427,11 +1427,8 @@ alloc_f(int argc, char **argv)
 
        cvtstr(offset, s1, sizeof(s1));
 
-       if (nb_sectors == 1)
-               printf("sector allocated at offset %s\n", s1);
-       else
-               printf("%d/%d sectors allocated at offset %s\n",
-                       sum_alloc, nb_sectors, s1);
+       printf("%d/%d sectors allocated at offset %s\n",
+              sum_alloc, nb_sectors, s1);
        return 0;
 }