[libvirt] [libvirt-php][PATCH 13/14] libvirt-php.c: Fix coding style issues with extra/less spaces

Katerina Koukiou k.koukiou at gmail.com
Fri Sep 23 18:49:48 UTC 2016


Fix some remaining issues about missing spaces around comparison operators
and remove extra spaces

Signed-off-by: Katerina Koukiou <k.koukiou at gmail.com>
---
 src/libvirt-php.c | 48 ++++++++++++++++++++++++------------------------
 1 file changed, 24 insertions(+), 24 deletions(-)

diff --git a/src/libvirt-php.c b/src/libvirt-php.c
index 24e7f5d..2fb8d60 100644
--- a/src/libvirt-php.c
+++ b/src/libvirt-php.c
@@ -2239,7 +2239,7 @@ PHP_FUNCTION(libvirt_connect)
     if (virGetVersion(&libVer, NULL, NULL)!= 0)
         RETURN_FALSE;
 
-    if (libVer<6002) {
+    if (libVer < 6002) {
         set_error("Only libvirt 0.6.2 and higher supported. Please upgrade your libvirt" TSRMLS_CC);
         RETURN_FALSE;
     }
@@ -4267,7 +4267,7 @@ PHP_FUNCTION(libvirt_domain_lookup_by_name)
     php_libvirt_domain *res_domain;
 
     GET_CONNECTION_FROM_ARGS("rs", &zconn, &name, &name_len);
-    if ((name == NULL) || (name_len<1))
+    if ((name == NULL) || (name_len < 1))
         RETURN_FALSE;
     domain = virDomainLookupByName(conn->conn, name);
     if (domain == NULL)
@@ -4305,7 +4305,7 @@ PHP_FUNCTION(libvirt_domain_lookup_by_uuid)
 
     GET_CONNECTION_FROM_ARGS("rs", &zconn, &uuid, &uuid_len);
 
-    if ((uuid == NULL) || (uuid_len<1))
+    if ((uuid == NULL) || (uuid_len < 1))
         RETURN_FALSE;
     domain = virDomainLookupByUUID(conn->conn, uuid);
     if (domain == NULL)
@@ -4373,7 +4373,7 @@ PHP_FUNCTION(libvirt_domain_lookup_by_uuid_string)
 
     GET_CONNECTION_FROM_ARGS("rs", &zconn, &uuid, &uuid_len);
 
-    if ((uuid == NULL) || (uuid_len<1))
+    if ((uuid == NULL) || (uuid_len < 1))
         RETURN_FALSE;
     domain = virDomainLookupByUUIDString(conn->conn, uuid);
     if (domain == NULL)
@@ -7623,7 +7623,7 @@ PHP_FUNCTION(libvirt_domain_snapshot_lookup_by_name)
 
     GET_DOMAIN_FROM_ARGS("rs|l", &zdomain, &name, &name_len, &flags);
 
-    if ((name == NULL) || (name_len<1))
+    if ((name == NULL) || (name_len < 1))
         RETURN_FALSE;
     snapshot=virDomainSnapshotLookupByName(domain->domain, name, flags);
     if (snapshot == NULL)
@@ -7779,7 +7779,7 @@ PHP_FUNCTION(libvirt_list_domain_snapshots)
         count = virDomainSnapshotListNames(domain->domain, names, expectedcount, 0);
     }
 
-    if ((count != expectedcount) || (count<0)) {
+    if ((count != expectedcount) || (count < 0)) {
         RETURN_FALSE;
     } else {
         array_init(return_value);
@@ -7812,7 +7812,7 @@ PHP_FUNCTION(libvirt_storagepool_lookup_by_name)
 
     GET_CONNECTION_FROM_ARGS("rs", &zconn, &name, &name_len);
 
-    if ((name == NULL) || (name_len<1))
+    if ((name == NULL) || (name_len < 1))
         RETURN_FALSE;
     pool = virStoragePoolLookupByName(conn->conn, name);
     DPRINTF("%s: virStoragePoolLookupByName(%p, %s) returned %p\n", PHPFUNC, conn->conn, name, pool);
@@ -7896,7 +7896,7 @@ PHP_FUNCTION(libvirt_storagepool_list_volumes)
     DPRINTF("%s: virStoragePoolListVolumes(%p,%p,%d) returned %d\n", PHPFUNC, pool->pool, names, expectedcount, count);
     array_init(return_value);
 
-    if ((count != expectedcount) || (count<0))
+    if ((count != expectedcount) || (count < 0))
 		RETURN_FALSE;
     for (i = 0; i < count; i++) {
         VIRT_ADD_NEXT_INDEX_STRING(return_value,  names[i]);
@@ -7955,7 +7955,7 @@ PHP_FUNCTION(libvirt_storagevolume_lookup_by_name)
     virStorageVolPtr volume = NULL;
 
     GET_STORAGEPOOL_FROM_ARGS("rs", &zpool, &name, &name_len);
-    if ((name == NULL) || (name_len<1))
+    if ((name == NULL) || (name_len < 1))
         RETURN_FALSE;
 
     volume = virStorageVolLookupByName(pool->pool, name);
@@ -7994,7 +7994,7 @@ PHP_FUNCTION(libvirt_storagevolume_lookup_by_path)
     virStorageVolPtr volume = NULL;
 
     GET_CONNECTION_FROM_ARGS("rs", &zconn, &name, &name_len);
-    if ((name == NULL) || (name_len<1))
+    if ((name == NULL) || (name_len < 1))
         RETURN_FALSE;
 
     volume = virStorageVolLookupByPath(conn->conn, name);
@@ -8576,7 +8576,7 @@ PHP_FUNCTION(libvirt_storagepool_create)
 
     GET_STORAGEPOOL_FROM_ARGS("r", &zpool);
 
-    retval = virStoragePoolCreate (pool->pool, 0);
+    retval = virStoragePoolCreate(pool->pool, 0);
     DPRINTF("%s: virStoragePoolCreate(%p, 0) returned %d\n", PHPFUNC, pool->pool, retval);
     if (retval != 0)
         RETURN_FALSE;
@@ -8782,7 +8782,7 @@ PHP_FUNCTION(libvirt_list_storagepools)
     names = (char **)emalloc(expectedcount*sizeof(char *));
     count = virConnectListStoragePools(conn->conn, names, expectedcount);
 
-    if ((count != expectedcount) || (count<0)) {
+    if ((count != expectedcount) || (count < 0)) {
         efree(names);
         RETURN_FALSE;
     }
@@ -8835,7 +8835,7 @@ PHP_FUNCTION(libvirt_list_active_storagepools)
     names = (char **)emalloc(expectedcount*sizeof(char *));
     count = virConnectListStoragePools(conn->conn, names, expectedcount);
 
-    if ((count != expectedcount) || (count<0)) {
+    if ((count != expectedcount) || (count < 0)) {
         efree(names);
         RETURN_FALSE;
     }
@@ -8992,7 +8992,7 @@ PHP_FUNCTION(libvirt_list_domain_resources)
 
     ids = (int *)emalloc(sizeof(int) * expectedcount);
     count = virConnectListDomains(conn->conn, ids, expectedcount);
-    if ((count != expectedcount) || (count<0)) {
+    if ((count != expectedcount) || (count < 0)) {
         efree(ids);
         RETURN_FALSE;
     }
@@ -9023,7 +9023,7 @@ PHP_FUNCTION(libvirt_list_domain_resources)
 
     names = (char **)emalloc(expectedcount*sizeof(char *));
     count = virConnectListDefinedDomains(conn->conn, names, expectedcount);
-    if ((count != expectedcount) || (count<0)) {
+    if ((count != expectedcount) || (count < 0)) {
         efree(names);
         RETURN_FALSE;
     }
@@ -9068,12 +9068,12 @@ PHP_FUNCTION(libvirt_list_active_domain_ids)
 
     GET_CONNECTION_FROM_ARGS("r", &zconn);
 
-    if ((expectedcount = virConnectNumOfDomains (conn->conn)) < 0)
+    if ((expectedcount = virConnectNumOfDomains(conn->conn)) < 0)
         RETURN_FALSE;
 
     ids = (int *)emalloc(sizeof(int) * expectedcount);
     count = virConnectListDomains(conn->conn, ids, expectedcount);
-    if ((count != expectedcount) || (count<0)) {
+    if ((count != expectedcount) || (count < 0)) {
         efree(ids);
         RETURN_FALSE;
     }
@@ -9103,12 +9103,12 @@ PHP_FUNCTION(libvirt_list_active_domains)
 
     GET_CONNECTION_FROM_ARGS("r", &zconn);
 
-    if ((expectedcount = virConnectNumOfDomains (conn->conn)) < 0)
+    if ((expectedcount = virConnectNumOfDomains(conn->conn)) < 0)
         RETURN_FALSE;
 
     ids = (int *)emalloc(sizeof(int) * expectedcount);
     count = virConnectListDomains(conn->conn, ids, expectedcount);
-    if ((count != expectedcount) || (count<0)) {
+    if ((count != expectedcount) || (count < 0)) {
         efree(ids);
         RETURN_FALSE;
     }
@@ -9154,7 +9154,7 @@ PHP_FUNCTION(libvirt_list_inactive_domains)
 
     names = (char **)emalloc(expectedcount*sizeof(char *));
     count = virConnectListDefinedDomains(conn->conn, names, expectedcount);
-    if ((count != expectedcount) || (count<0)) {
+    if ((count != expectedcount) || (count < 0)) {
         efree(names);
         RETURN_FALSE;
     }
@@ -9194,7 +9194,7 @@ PHP_FUNCTION(libvirt_list_networks)
 
         names = (char **)emalloc(expectedcount*sizeof(char *));
         count = virConnectListNetworks(conn->conn, names, expectedcount);
-        if ((count != expectedcount) || (count<0)) {
+        if ((count != expectedcount) || (count < 0)) {
             efree(names);
             RETURN_FALSE;
         }
@@ -9213,7 +9213,7 @@ PHP_FUNCTION(libvirt_list_networks)
             RETURN_FALSE;
         names = (char **)emalloc(expectedcount*sizeof(char *));
         count = virConnectListDefinedNetworks(conn->conn, names, expectedcount);
-        if ((count != expectedcount) || (count<0)) {
+        if ((count != expectedcount) || (count < 0)) {
             efree(names);
             RETURN_FALSE;
         }
@@ -9256,7 +9256,7 @@ PHP_FUNCTION(libvirt_list_nodedevs)
         RETURN_FALSE;
     names = (char **)emalloc(expectedcount*sizeof(char *));
     count = virNodeListDevices(conn->conn, cap, names, expectedcount, 0);
-    if ((count != expectedcount) || (count<0)) {
+    if ((count != expectedcount) || (count < 0)) {
         efree(names);
         RETURN_FALSE;
     }
@@ -9330,7 +9330,7 @@ PHP_FUNCTION(libvirt_nodedev_capabilities)
         RETURN_FALSE;
     names = (char **)emalloc(expectedcount*sizeof(char *));
     count = virNodeDeviceListCaps(nodedev->device, names, expectedcount);
-    if ((count != expectedcount) || (count<0))
+    if ((count != expectedcount) || (count < 0))
         RETURN_FALSE;
 
     array_init(return_value);
-- 
2.7.3




More information about the libvir-list mailing list