[libvirt] [PATCH] virsh: Fix version numbers in comments

Osier Yang jyang at redhat.com
Thu Sep 13 08:59:49 UTC 2012


And redundant error resetting.

Pushed under trivial rule.
---
 tools/virsh-network.c |    8 +++-----
 tools/virsh-pool.c    |    8 +++-----
 tools/virsh-volume.c  |    8 +++-----
 3 files changed, 9 insertions(+), 15 deletions(-)

diff --git a/tools/virsh-network.c b/tools/virsh-network.c
index 5c3daa3..2c32a78 100644
--- a/tools/virsh-network.c
+++ b/tools/virsh-network.c
@@ -397,7 +397,7 @@ vshNetworkListCollect(vshControl *ctl,
     int nInactiveNets = 0;
     int nAllNets = 0;
 
-    /* try the list with flags support (0.10.0 and later) */
+    /* try the list with flags support (0.10.2 and later) */
     if ((ret = virConnectListAllNetworks(ctl->conn,
                                          &list->nets,
                                          flags)) >= 0) {
@@ -406,10 +406,8 @@ vshNetworkListCollect(vshControl *ctl,
     }
 
     /* check if the command is actually supported */
-    if (last_error && last_error->code == VIR_ERR_NO_SUPPORT) {
-        vshResetLibvirtError();
+    if (last_error && last_error->code == VIR_ERR_NO_SUPPORT)
         goto fallback;
-    }
 
     if (last_error && last_error->code ==  VIR_ERR_INVALID_ARG) {
         /* try the new API again but mask non-guaranteed flags */
@@ -430,7 +428,7 @@ vshNetworkListCollect(vshControl *ctl,
 
 
 fallback:
-    /* fall back to old method (0.9.13 and older) */
+    /* fall back to old method (0.10.1 and older) */
     vshResetLibvirtError();
 
     /* Get the number of active networks */
diff --git a/tools/virsh-pool.c b/tools/virsh-pool.c
index 15d1883..bc10f76 100644
--- a/tools/virsh-pool.c
+++ b/tools/virsh-pool.c
@@ -606,7 +606,7 @@ vshStoragePoolListCollect(vshControl *ctl,
     int nInactivePools = 0;
     int nAllPools = 0;
 
-    /* try the list with flags support (0.10.0 and later) */
+    /* try the list with flags support (0.10.2 and later) */
     if ((ret = virConnectListAllStoragePools(ctl->conn,
                                              &list->pools,
                                              flags)) >= 0) {
@@ -615,10 +615,8 @@ vshStoragePoolListCollect(vshControl *ctl,
     }
 
     /* check if the command is actually supported */
-    if (last_error && last_error->code == VIR_ERR_NO_SUPPORT) {
-        vshResetLibvirtError();
+    if (last_error && last_error->code == VIR_ERR_NO_SUPPORT)
         goto fallback;
-    }
 
     if (last_error && last_error->code ==  VIR_ERR_INVALID_ARG) {
         /* try the new API again but mask non-guaranteed flags */
@@ -638,7 +636,7 @@ vshStoragePoolListCollect(vshControl *ctl,
 
 
 fallback:
-    /* fall back to old method (0.9.13 and older) */
+    /* fall back to old method (0.10.1 and older) */
     vshResetLibvirtError();
 
     /* There is no way to get the pool type */
diff --git a/tools/virsh-volume.c b/tools/virsh-volume.c
index ec0b5b0..c5bd4af 100644
--- a/tools/virsh-volume.c
+++ b/tools/virsh-volume.c
@@ -1017,7 +1017,7 @@ vshStorageVolListCollect(vshControl *ctl,
     int nvols = 0;
     int ret = -1;
 
-    /* try the list with flags support (0.10.0 and later) */
+    /* try the list with flags support (0.10.2 and later) */
     if ((ret = virStoragePoolListAllVolumes(pool,
                                             &list->vols,
                                             flags)) >= 0) {
@@ -1026,17 +1026,15 @@ vshStorageVolListCollect(vshControl *ctl,
     }
 
     /* check if the command is actually supported */
-    if (last_error && last_error->code == VIR_ERR_NO_SUPPORT) {
-        vshResetLibvirtError();
+    if (last_error && last_error->code == VIR_ERR_NO_SUPPORT)
         goto fallback;
-    }
 
     /* there was an error during the call */
     vshError(ctl, "%s", _("Failed to list volumes"));
     goto cleanup;
 
 fallback:
-    /* fall back to old method (0.9.13 and older) */
+    /* fall back to old method (0.10.1 and older) */
     vshResetLibvirtError();
 
     /* Determine the number of volumes in the pool */
-- 
1.7.7.3




More information about the libvir-list mailing list