Re: [libvirt] [PATCH 05/32] Convert 'int i' to 'size_t i' in tools/ files

From: "Daniel P. Berrange" <berrange(a)redhat.com&gt; Convert the type of loop iterators named 'i', 'j', k', 'ii', 'jj', 'kk', to be 'size_t' instead of 'int' or 'unsigned int', also santizing 'ii', 'jj', 'kk' to use the normal 'i', 'j', 'k' naming Signed-off-by: Daniel P. Berrange <berrange(a)redhat.com&gt; --- tools/virsh-domain-monitor.c | 17 +++++++++-------- tools/virsh-domain.c | 41 +++++++++++++++++++++++------------------ tools/virsh-host.c | 8 ++++---- tools/virsh-interface.c | 6 +++--- tools/virsh-network.c | 6 +++--- tools/virsh-nodedev.c | 8 ++++---- tools/virsh-nwfilter.c | 6 +++--- tools/virsh-pool.c | 7 ++++--- tools/virsh-secret.c | 6 +++--- tools/virsh-snapshot.c | 12 ++++++------ tools/virsh-volume.c | 6 +++--- tools/virsh.c | 15 ++++++++------- 12 files changed, 73 insertions(+), 65 deletions(-)

diff --git a/tools/virsh-secret.c b/tools/virsh-secret.c index fb10119..ac11e53 100644 --- a/tools/virsh-secret.c +++ b/tools/virsh-secret.c @@ -367,7 +367,7 @@ typedef struct vshSecretList *vshSecretListPtr; static void vshSecretListFree(vshSecretListPtr list) { - int i; + size_t i; if (list && list->nsecrets) { for (i = 0; i < list->nsecrets; i++) { @@ -384,7 +384,7 @@ vshSecretListCollect(vshControl *ctl, unsigned int flags) { vshSecretListPtr list = vshMalloc(ctl, sizeof(*list)); - int i; + size_t i; int ret; virSecretPtr secret; bool success = false;

@@ -1095,7 +1095,7 @@ vshSnapshotListCollect(vshControl *ctl, virDomainPtr dom, virDomainSnapshotPtr from, unsigned int orig_flags, bool tree) { - int i; + size_t i; char **names = NULL; int count = -1; bool descendants = false;

@@ -1203,7 +1203,7 @@ vshStorageVolListCollect(vshControl *ctl, unsigned int flags) { vshStorageVolListPtr list = vshMalloc(ctl, sizeof(*list)); - int i; + size_t i; char **names = NULL; virStorageVolPtr vol = NULL; bool success = false;