maint: fix comma style issues: remaining drivers

Most of our code base uses space after comma but not before;
fix the remaining uses before adding a syntax check.

* src/lxc/lxc_container.c: Consistently use commas.
* src/openvz/openvz_driver.c: Likewise.
* src/openvz/openvz_util.c: Likewise.
* src/remote/remote_driver.c: Likewise.
* src/test/test_driver.c: Likewise.

Signed-off-by: Eric Blake <eblake@redhat.com>

diff --git a/src/lxc/lxc_container.c b/src/lxc/lxc_container.c
index 2bdf957d1ddce5cfb363d9306702b2791031441f..2059b83518c19e503cf0bbe44660fa702412a950 100644 (file)
--- a/src/lxc/lxc_container.c
+++ b/src/lxc/lxc_container.c
@@ -483,7 +483,7 @@ error_out:
 
 
 /*_syscall2(int, pivot_root, char *, newroot, const char *, oldroot)*/
-extern int pivot_root(const char * new_root,const char * put_old);
+extern int pivot_root(const char * new_root, const char * put_old);
 
 static int lxcContainerChildMountSort(const void *a, const void *b)
 {

diff --git a/src/openvz/openvz_driver.c b/src/openvz/openvz_driver.c
index df55568c19eedb082977b74fe991d3297e0de773..aee73904e08346b70536828e391d34a64a8c7bf2 100644 (file)
--- a/src/openvz/openvz_driver.c
+++ b/src/openvz/openvz_driver.c
@@ -1280,7 +1280,7 @@ openvzDomainGetAutostart(virDomainPtr dom, int *autostart)
     }
 
     *autostart = 0;
-    if (STREQ(value,"yes"))
+    if (STREQ(value, "yes"))
         *autostart = 1;
     ret = 0;
 
@@ -1523,7 +1523,7 @@ static int openvzConnectListDomains(virConnectPtr conn ATTRIBUTE_UNUSED,
     int ret;
     char buf[32];
     char *endptr;
-    virCommandPtr cmd = virCommandNewArgList(VZLIST, "-ovpsid", "-H" , NULL);
+    virCommandPtr cmd = virCommandNewArgList(VZLIST, "-ovpsid", "-H", NULL);
 
     virCommandSetOutputFD(cmd, &outfd);
     if (virCommandRunAsync(cmd, NULL) < 0)

diff --git a/src/openvz/openvz_util.c b/src/openvz/openvz_util.c
index dc69df2cb4f8da398d91b42b0d9744213428db39..8920d14f9b907f9dcda5a58f8afeba5e25b5f1ba 100644 (file)
--- a/src/openvz/openvz_util.c
+++ b/src/openvz/openvz_util.c
@@ -1,6 +1,7 @@
 /*
  * openvz_driver.c: core driver methods for managing OpenVZ VEs
  *
+ * Copyright (C) 2013 Red Hat, Inc.
  * Copyright (C) 2012 Guido Günther
  *
  * This library is free software; you can redistribute it and/or
@@ -64,7 +65,7 @@ openvzVEGetStringParam(virDomainPtr domain, const char* param)
                                              "-o",
                                              param,
                                              domain->name,
-                                             "-H" , NULL);
+                                             "-H", NULL);
 
     virCommandSetOutputBuffer(cmd, &output);
     if (virCommandRun(cmd, NULL) < 0) {

diff --git a/src/remote/remote_driver.c b/src/remote/remote_driver.c
index 71819490698a4379f074820e914ce278f3f2f089..df7558b9c0bf8c6b68c2d1cd9e3c0ac3f000e786 100644 (file)
--- a/src/remote/remote_driver.c
+++ b/src/remote/remote_driver.c
@@ -4484,7 +4484,7 @@ remoteDomainBuildEventIOErrorReason(virNetClientProgramPtr prog ATTRIBUTE_UNUSED
     virDomainPtr dom;
     virDomainEventPtr event = NULL;
 
-    dom = get_nonnull_domain(conn,msg->dom);
+    dom = get_nonnull_domain(conn, msg->dom);
     if (!dom)
         return;
 

diff --git a/src/test/test_driver.c b/src/test/test_driver.c
index 26782472287ed26b0a05c1cd6b4c89ec0f96fca9..469223ef569b21f0d3ada124583d56eb554a27ba 100644 (file)
--- a/src/test/test_driver.c
+++ b/src/test/test_driver.c
@@ -1,7 +1,7 @@
 /*
  * test.c: A "mock" hypervisor for use by application unit tests
  *
- * Copyright (C) 2006-2012 Red Hat, Inc.
+ * Copyright (C) 2006-2013 Red Hat, Inc.
  * Copyright (C) 2006 Daniel P. Berrange
  *
  * This library is free software; you can redistribute it and/or
@@ -306,7 +306,7 @@ testBuildCapabilities(virConnectPtr conn) {
 
     if (virCapabilitiesAddHostFeature(caps, "pae") < 0)
         goto error;
-    if (virCapabilitiesAddHostFeature(caps ,"nonpae") < 0)
+    if (virCapabilitiesAddHostFeature(caps, "nonpae") < 0)
         goto error;
 
     for (i = 0; i < privconn->numCells; i++) {
@@ -344,7 +344,7 @@ testBuildCapabilities(virConnectPtr conn) {
 
         if (virCapabilitiesAddGuestFeature(guest, "pae", 1, 1) == NULL)
             goto error;
-        if (virCapabilitiesAddGuestFeature(guest ,"nonpae", 1, 1) == NULL)
+        if (virCapabilitiesAddGuestFeature(guest, "nonpae", 1, 1) == NULL)
             goto error;
     }
 
@@ -1620,7 +1620,7 @@ testDomainCreateXML(virConnectPtr conn, const char *xml,
     virCheckFlags(0, NULL);
 
     testDriverLock(privconn);
-    if ((def = virDomainDefParseString(xml,privconn->caps, privconn->xmlopt,
+    if ((def = virDomainDefParseString(xml, privconn->caps, privconn->xmlopt,
                                        1 << VIR_DOMAIN_VIRT_TEST,
                                        VIR_DOMAIN_XML_INACTIVE)) == NULL)
         goto cleanup;
@@ -2598,7 +2598,7 @@ static int testDomainGetVcpus(virDomainPtr domain,
 
     if (!virDomainObjIsActive(privdom)) {
         virReportError(VIR_ERR_OPERATION_INVALID,
-                       "%s",_("cannot list vcpus for an inactive domain"));
+                       "%s", _("cannot list vcpus for an inactive domain"));
         goto cleanup;
     }
 
@@ -2685,7 +2685,7 @@ static int testDomainPinVcpu(virDomainPtr domain,
 
     if (!virDomainObjIsActive(privdom)) {
         virReportError(VIR_ERR_OPERATION_INVALID,
-                       "%s",_("cannot pin vcpus on an inactive domain"));
+                       "%s", _("cannot pin vcpus on an inactive domain"));
         goto cleanup;
     }