[libvirt] [libvirt-python PATCH 04/23] drop unnecessary goto

Pavel Hrdina phrdina at redhat.com
Thu Sep 24 14:01:41 UTC 2015


Signed-off-by: Pavel Hrdina <phrdina at redhat.com>
---
 libvirt-override.c | 19 +++++++------------
 1 file changed, 7 insertions(+), 12 deletions(-)

diff --git a/libvirt-override.c b/libvirt-override.c
index 114104b..b18196f 100644
--- a/libvirt-override.c
+++ b/libvirt-override.c
@@ -476,27 +476,22 @@ cleanup:
  */
 static int
 getPyNodeCPUCount(virConnectPtr conn) {
-    int i_retval = -1;
+    int i_retval;
     virNodeInfo nodeinfo;
 
 #if LIBVIR_CHECK_VERSION(1, 0, 0)
     LIBVIRT_BEGIN_ALLOW_THREADS;
     i_retval = virNodeGetCPUMap(conn, NULL, NULL, 0);
     LIBVIRT_END_ALLOW_THREADS;
-#endif /* LIBVIR_CHECK_VERSION(1, 0, 0) */
-
-    if (i_retval < 0) {
-        /* fallback: use nodeinfo */
-        LIBVIRT_BEGIN_ALLOW_THREADS;
-        i_retval = virNodeGetInfo(conn, &nodeinfo);
-        LIBVIRT_END_ALLOW_THREADS;
-        if (i_retval < 0)
-            goto cleanup;
+#else /* fallback: use nodeinfo */
+    LIBVIRT_BEGIN_ALLOW_THREADS;
+    i_retval = virNodeGetInfo(conn, &nodeinfo);
+    LIBVIRT_END_ALLOW_THREADS;
 
+    if (i_retval >= 0)
         i_retval = VIR_NODEINFO_MAXCPUS(nodeinfo);
-    }
+#endif /* LIBVIR_CHECK_VERSION(1, 0, 0) */
 
-cleanup:
     return i_retval;
 }
 
-- 
2.5.3




More information about the libvir-list mailing list