Patches speak louder than words. This is my counter-proposal to Leonid's thread on how best to respresent the S_*iB macros in units.h, where my proposal is that we don't need them at all. (hence my subject line, even though it is completely unrelated to the series)
True, my diffstat is even bigger, but I think it is more maintainable in the long run (if calling QemuOpts maintainable is even appropriate). Eric Blake (6): qemu-option: Allow integer defaults block: Take advantage of QemuOpt default integers Revert "vdi: Use a literal number of bytes for DEFAULT_CLUSTER_SIZE" qemu: Prefer '(x * MiB)' over 'S_xiB' Revert "include: Add a comment to explain the origin of sizes' lookup table" Revert "include: Add a lookup table of sizes" block/qcow2.h | 10 +++--- include/qemu/option.h | 12 +++++++ include/qemu/units.h | 73 ------------------------------------------- block/parallels.c | 2 +- block/qcow2.c | 2 +- block/qed.c | 2 +- block/vdi.c | 6 ++-- block/vhdx.c | 3 +- util/qemu-option.c | 69 ++++++++++++++++++++++++++++++++++------ 9 files changed, 84 insertions(+), 95 deletions(-) -- 2.20.1