On 01/26/2018 08:59 AM, Alberto Garcia wrote: > Adding support for L2 slices to l2_allocate() needs (among other > things) an extra loop that iterates over all slices of a new L2 table. > > Putting all changes in one patch would make it hard to read because > all semantic changes would be mixed with pure indentation changes. > > To make things easier this patch simply creates a new block and > changes the indentation of all lines of code inside it. Thus, all > modifications in this patch are cosmetic. There are no semantic > changes and no variables are renamed yet. The next patch will take > care of that.
Thanks for the split - it does make reviewing easier.
>
> Signed-off-by: Alberto Garcia <be...@igalia.com>
> ---
> block/qcow2-cluster.c | 55
> ++++++++++++++++++++++++++++-----------------------
> 1 file changed, 30 insertions(+), 25 deletions(-)
More lines than before, because of the added {}. But diff ignoring
whitespace makes this one easy to validate.
Reviewed-by: Eric Blake <ebl...@redhat.com>
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
signature.asc
Description: OpenPGP digital signature
