diff options
author | Eric Blake <eblake@redhat.com> | 2017-09-25 09:55:07 -0500 |
---|---|---|
committer | Kevin Wolf <kwolf@redhat.com> | 2017-10-06 16:28:58 +0200 |
commit | a8b42a1c09e751b9f921a1a73756411fc118020b (patch) | |
tree | 451521f91c36fce585c73c690605d17aa1100fb7 /tests/test-hbitmap.c | |
parent | 765d9df9626f45a821f221f7a46ef524354b3600 (diff) | |
download | qemu-a8b42a1c09e751b9f921a1a73756411fc118020b.zip qemu-a8b42a1c09e751b9f921a1a73756411fc118020b.tar.gz qemu-a8b42a1c09e751b9f921a1a73756411fc118020b.tar.bz2 |
block: Make bdrv_img_create() size selection easier to read
All callers of bdrv_img_create() pass in a size, or -1 to read the
size from the backing file. We then set that size as the QemuOpt
default, which means we will reuse that default rather than the
final parameter to qemu_opt_get_size() several lines later. But
it is rather confusing to read subsequent checks of 'size == -1'
when it looks (without seeing the full context) like size defaults
to 0; it also doesn't help that a size of 0 is valid (for some
formats).
Rework the logic to make things more legible.
Signed-off-by: Eric Blake <eblake@redhat.com>
Reviewed-by: John Snow <jsnow@redhat.com>
Reviewed-by: Kevin Wolf <kwolf@redhat.com>
Reviewed-by: Fam Zheng <famz@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Diffstat (limited to 'tests/test-hbitmap.c')
0 files changed, 0 insertions, 0 deletions