aboutsummaryrefslogtreecommitdiff
path: root/tests/test-hbitmap.c
diff options
context:
space:
mode:
authorEric Blake <eblake@redhat.com>2017-09-25 09:55:07 -0500
committerKevin Wolf <kwolf@redhat.com>2017-10-06 16:28:58 +0200
commita8b42a1c09e751b9f921a1a73756411fc118020b (patch)
tree451521f91c36fce585c73c690605d17aa1100fb7 /tests/test-hbitmap.c
parent765d9df9626f45a821f221f7a46ef524354b3600 (diff)
downloadqemu-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