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 /include/qemu/hbitmap.h | |
| parent | 765d9df9626f45a821f221f7a46ef524354b3600 (diff) | |
| download | focaccia-qemu-a8b42a1c09e751b9f921a1a73756411fc118020b.tar.gz focaccia-qemu-a8b42a1c09e751b9f921a1a73756411fc118020b.zip | |
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 'include/qemu/hbitmap.h')
0 files changed, 0 insertions, 0 deletions