On 12/03/2014 06:37 AM, Max Reitz wrote:
> qcow2_update_cluster_refcount() does not have any quick access to the
> new refcount value, it has to call qcow2_get_refcount(). Some callers do
> not need that new value at all, others call qcow2_get_refcount()
> themselves anyway (albeit in a different code path, which can however be
> easily changed), therefore there is no advantage in making
> qcow2_update_cluster_refcount() return the new value. Drop it.
> 
> Signed-off-by: Max Reitz <mre...@redhat.com>
> ---
>  block/qcow2-refcount.c | 25 +++++++++++++++----------
>  1 file changed, 15 insertions(+), 10 deletions(-)
> 

Reviewed-by: Eric Blake <ebl...@redhat.com>

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to