Add a new parameter 'styleFormat' to be used when printing the
warning message so that it's "clearer" what style of compression
call caused the error. Add that style to both messages as a paremter.

Also a VIR_WARN error message doesn't need to be translated
 (e.g. inside _()), so remove the need for the translation.

Signed-off-by: John Ferlan <jfer...@redhat.com>
---
 src/qemu/qemu_driver.c | 17 ++++++++++-------
 1 file changed, 10 insertions(+), 7 deletions(-)

diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
index d5acc30..7dfba50 100644
--- a/src/qemu/qemu_driver.c
+++ b/src/qemu/qemu_driver.c
@@ -3270,6 +3270,7 @@ qemuCompressProgramAvailable(virQEMUSaveFormat compress)
 /* qemuGetCompressionProgram:
  * @imageFormat: String representation from qemu.conf for the compression
  *               image format being used (dump, save, or snapshot).
+ * @styleFormat: String representing the style of format (dump, save, snapshot)
  *
  * Returns:
  *    virQEMUSaveFormat    - Integer representation of the compression
@@ -3280,7 +3281,8 @@ qemuCompressProgramAvailable(virQEMUSaveFormat compress)
  *                           indicating none.
  */
 static virQEMUSaveFormat
-qemuGetCompressionProgram(const char *imageFormat)
+qemuGetCompressionProgram(const char *imageFormat,
+                          const char *styleFormat)
 {
     virQEMUSaveFormat ret;
 
@@ -3297,12 +3299,13 @@ qemuGetCompressionProgram(const char *imageFormat)
 
  error:
     if (ret < 0)
-        VIR_WARN("%s", _("Invalid dump image format specified in "
-                         "configuration file, using raw"));
+        VIR_WARN("Invalid %s image format specified in "
+                 "configuration file, using raw",
+                 styleFormat);
     else
-        VIR_WARN("%s", _("Compression program for dump image format "
-                         "in configuration file isn't available, "
-                         "using raw"));
+        VIR_WARN("Compression program for %s image format in "
+                 "configuration file isn't available, using raw",
+                 styleFormat);
 
     /* Use "raw" as the format if the specified format is not valid,
      * or the compress program is not available. */
@@ -3590,7 +3593,7 @@ doCoreDump(virQEMUDriverPtr driver,
 
     /* We reuse "save" flag for "dump" here. Then, we can support the same
      * format in "save" and "dump". */
-    compress = qemuGetCompressionProgram(cfg->dumpImageFormat);
+    compress = qemuGetCompressionProgram(cfg->dumpImageFormat, "dump");
 
     /* Create an empty file with appropriate ownership.  */
     if (dump_flags & VIR_DUMP_BYPASS_CACHE) {
-- 
2.7.4

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to