maint: use VIR_ERROR0 rather than VIR_ERROR with a bare "%s"

Change VIR_ERROR("%s", "..."
to     VIR_ERROR0("..."

and

Change VIR_ERROR("%s", _("...")
to     VIR_ERROR0(_("...")

Use this command:
  git grep -E -l 'VIR_ERROR\("%s", (_\()?"'|xargs perl -pi -e \
  's/VIR_ERROR\("%s", (_\()?"/VIR_ERROR0($1"/'

diff --git a/daemon/libvirtd.c b/daemon/libvirtd.c
index e14b92a26c5fb104f393386067b65891d61b05df..f09e80390ee304e7a19228779513c63948398ca1 100644 (file)
--- a/daemon/libvirtd.c
+++ b/daemon/libvirtd.c
@@ -539,7 +539,7 @@ static int qemudListenUnix(struct qemud_server *server,
     char ebuf[1024];
 
     if (VIR_ALLOC(sock) < 0) {
-        VIR_ERROR("%s", _("Failed to allocate memory for struct qemud_socket"));
+        VIR_ERROR0(_("Failed to allocate memory for struct qemud_socket"));
         return -1;
     }
 
@@ -844,12 +844,12 @@ static struct qemud_server *qemudInitialize(void) {
     server->sigread = server->sigwrite = -1;
 
     if (virMutexInit(&server->lock) < 0) {
-        VIR_ERROR("%s", _("cannot initialize mutex"));
+        VIR_ERROR0(_("cannot initialize mutex"));
         VIR_FREE(server);
         return NULL;
     }
     if (virCondInit(&server->job) < 0) {
-        VIR_ERROR("%s", _("cannot initialize condition variable"));
+        VIR_ERROR0(_("cannot initialize condition variable"));
         virMutexDestroy(&server->lock);
         VIR_FREE(server);
         return NULL;
@@ -1359,7 +1359,7 @@ static int qemudDispatchServer(struct qemud_server *server, struct qemud_socket
     if (VIR_ALLOC(client) < 0)
         goto cleanup;
     if (virMutexInit(&client->lock) < 0) {
-        VIR_ERROR("%s", _("cannot initialize mutex"));
+        VIR_ERROR0(_("cannot initialize mutex"));
         VIR_FREE(client);
         goto cleanup;
     }
@@ -2771,7 +2771,7 @@ remoteReadConfigFile (struct qemud_server *server, const char *filename)
                 maxbuf = 1024;
 
             if (VIR_ALLOC_N(buf, maxbuf) < 0) {
-                VIR_ERROR("%s", _("Failed to allocate memory for buffer"));
+                VIR_ERROR0(_("Failed to allocate memory for buffer"));
                 goto free_and_fail;
             }
 
@@ -2780,7 +2780,7 @@ remoteReadConfigFile (struct qemud_server *server, const char *filename)
                                      &grp)) == ERANGE) {
                     maxbuf *= 2;
                     if (maxbuf > 65536 || VIR_REALLOC_N(buf, maxbuf) < 0) {
-                        VIR_ERROR("%s", _("Failed to reallocate enough memory for buffer"));
+                        VIR_ERROR0(_("Failed to reallocate enough memory for buffer"));
                         goto free_and_fail;
                     }
             }

diff --git a/src/phyp/phyp_driver.c b/src/phyp/phyp_driver.c
index 8f4f310d36ca232dd7db9f18dc987044c8c867ee..fbb094ffa023c96a4d264bf36fb95a75fabb756c 100644 (file)
--- a/src/phyp/phyp_driver.c
+++ b/src/phyp/phyp_driver.c
@@ -1233,30 +1233,30 @@ phypDomainDumpXML(virDomainPtr dom, int flags)
                                       dom->conn);
 
     if (lpar_name == NULL) {
-        VIR_ERROR("%s", "Unable to determine domain's name.");
+        VIR_ERROR0("Unable to determine domain's name.");
         goto err;
     }
 
     if (phypGetLparUUID(def.uuid, dom->id, dom->conn) == -1) {
-        VIR_ERROR("%s", "Unable to generate random uuid.");
+        VIR_ERROR0("Unable to generate random uuid.");
         goto err;
     }
 
     if ((def.maxmem =
          phypGetLparMem(dom->conn, managed_system, dom->id, 0)) == 0) {
-        VIR_ERROR("%s", "Unable to determine domain's max memory.");
+        VIR_ERROR0("Unable to determine domain's max memory.");
         goto err;
     }
 
     if ((def.memory =
          phypGetLparMem(dom->conn, managed_system, dom->id, 1)) == 0) {
-        VIR_ERROR("%s", "Unable to determine domain's memory.");
+        VIR_ERROR0("Unable to determine domain's memory.");
         goto err;
     }
 
     if ((def.vcpus =
          phypGetLparCPU(dom->conn, managed_system, dom->id)) == 0) {
-        VIR_ERROR("%s", "Unable to determine domain's CPU.");
+        VIR_ERROR0("Unable to determine domain's CPU.");
         goto err;
     }
 
@@ -1695,7 +1695,7 @@ phypBuildLpar(virConnectPtr conn, virDomainDefPtr def)
     }
 
     if (phypUUIDTable_AddLpar(conn, def->uuid, def->id) == -1) {
-        VIR_ERROR("%s", "Unable to add LPAR to the table");
+        VIR_ERROR0("Unable to add LPAR to the table");
         goto err;
     }
 
@@ -1835,13 +1835,13 @@ phypUUIDTable_WriteFile(virConnectPtr conn)
         if (safewrite(fd, &uuid_table->lpars[i]->id,
                       sizeof(uuid_table->lpars[i]->id)) !=
             sizeof(uuid_table->lpars[i]->id)) {
-            VIR_ERROR("%s", "Unable to write information to local file.");
+            VIR_ERROR0("Unable to write information to local file.");
             goto err;
         }
 
         if (safewrite(fd, uuid_table->lpars[i]->uuid, VIR_UUID_BUFLEN) !=
             VIR_UUID_BUFLEN) {
-            VIR_ERROR("%s", "Unable to write information to local file.");
+            VIR_ERROR0("Unable to write information to local file.");
             goto err;
         }
     }

diff --git a/src/qemu/qemu_conf.c b/src/qemu/qemu_conf.c
index 2e117c69bd06b0a89589664f560ffe75d7cfc02b..f37553c2382c069475cf2f7c0a6bc90cf58bfea7 100644 (file)
--- a/src/qemu/qemu_conf.c
+++ b/src/qemu/qemu_conf.c
@@ -254,7 +254,7 @@ int qemudLoadDriverConfig(struct qemud_driver *driver,
         for (i = 0, pp = p->list; pp; ++i, pp = pp->next) {
             int ctl;
             if (pp->type != VIR_CONF_STRING) {
-                VIR_ERROR("%s", _("cgroup_controllers must be a list of strings"));
+                VIR_ERROR0(_("cgroup_controllers must be a list of strings"));
                 virConfFree(conf);
                 return -1;
             }
@@ -292,7 +292,7 @@ int qemudLoadDriverConfig(struct qemud_driver *driver,
         }
         for (i = 0, pp = p->list; pp; ++i, pp = pp->next) {
             if (pp->type != VIR_CONF_STRING) {
-                VIR_ERROR("%s", _("cgroup_device_acl must be a list of strings"));
+                VIR_ERROR0(_("cgroup_device_acl must be a list of strings"));
                 virConfFree(conf);
                 return -1;
             }

diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
index 3823dbde4c076d3f42bed123a3045a227c68a1d6..bf959d0bfb2f0e15dbdfc1a3452e32b049912d1c 100644 (file)
--- a/src/qemu/qemu_driver.c
+++ b/src/qemu/qemu_driver.c
@@ -1453,7 +1453,7 @@ qemudStartup(int privileged) {
         return -1;
 
     if (virMutexInit(&qemu_driver->lock) < 0) {
-        VIR_ERROR("%s", _("cannot initialize mutex"));
+        VIR_ERROR0(_("cannot initialize mutex"));
         VIR_FREE(qemu_driver);
         return -1;
     }