From patchwork Sat Apr 17 09:49:07 2010 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: [2/2] qcow2: Avoid shadowing variable in alloc_clusters_noref() Date: Fri, 16 Apr 2010 23:49:07 -0000 From: Stefan Hajnoczi X-Patchwork-Id: 50378 Message-Id: <1271497747-4463-2-git-send-email-stefanha@linux.vnet.ibm.com> To: qemu-devel@nongnu.org Cc: Kevin Wolf , Stefan Hajnoczi The i loop iterator is shadowed by the next free cluster index. Both using the variable name 'i' makes the code harder to read. Signed-off-by: Stefan Hajnoczi --- block/qcow2-refcount.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/block/qcow2-refcount.c b/block/qcow2-refcount.c index 47c9978..088d96d 100644 --- a/block/qcow2-refcount.c +++ b/block/qcow2-refcount.c @@ -554,8 +554,8 @@ static int64_t alloc_clusters_noref(BlockDriverState *bs, int64_t size) nb_clusters = size_to_clusters(s, size); retry: for(i = 0; i < nb_clusters; i++) { - int64_t i = s->free_cluster_index++; - if (get_refcount(bs, i) != 0) + int64_t next_cluster_index = s->free_cluster_index++; + if (get_refcount(bs, next_cluster_index) != 0) goto retry; } #ifdef DEBUG_ALLOC2