hyperv: return void from hypervAddSimpleParam

Signed-off-by: Ján Tomko <jtomko@redhat.com>
Reviewed-by: Peter Krempa <pkrempa@redhat.com>

diff --git a/src/hyperv/hyperv_driver.c b/src/hyperv/hyperv_driver.c
index 68b2e7fa67d903d3eaa64311caafb8de524a3772..940fa58d0d9f11bf071fdb3b8b9fd590604271e0 100644 (file)
--- a/src/hyperv/hyperv_driver.c
+++ b/src/hyperv/hyperv_driver.c
@@ -3739,8 +3739,7 @@ hypervDomainSendKey(virDomainPtr domain, unsigned int codeset,
         if (!params)
             return -1;
 
-        if (hypervAddSimpleParam(params, "keyCode", keycodeStr) < 0)
-            return -1;
+        hypervAddSimpleParam(params, "keyCode", keycodeStr);
 
         if (hypervInvokeMethod(priv, &params, NULL) < 0)
             return -1;
@@ -3759,8 +3758,7 @@ hypervDomainSendKey(virDomainPtr domain, unsigned int codeset,
         if (!params)
             return -1;
 
-        if (hypervAddSimpleParam(params, "keyCode", keycodeStr) < 0)
-            return -1;
+        hypervAddSimpleParam(params, "keyCode", keycodeStr);
 
         if (hypervInvokeMethod(priv, &params, NULL) < 0)
             return -1;

diff --git a/src/hyperv/hyperv_wmi.c b/src/hyperv/hyperv_wmi.c
index 1e39e6d61fbb881e51b8b6892b0612448944e59c..ab3eb5ffc097dfce5287b00f2a776e2c30b51a5c 100644 (file)
--- a/src/hyperv/hyperv_wmi.c
+++ b/src/hyperv/hyperv_wmi.c
@@ -204,10 +204,8 @@ hypervCheckParams(hypervInvokeParamsList *params)
  *
  * Add a param of type HYPERV_SIMPLE_PARAM, which is essentially a serialized
  * key/value pair.
- *
- * Returns -1 on failure, 0 on success.
  */
-int
+void
 hypervAddSimpleParam(hypervInvokeParamsList *params, const char *name,
                      const char *value)
 {
@@ -222,8 +220,6 @@ hypervAddSimpleParam(hypervInvokeParamsList *params, const char *name,
     p->simple.value = value;
 
     params->nbParams++;
-
-    return 0;
 }
 
 
@@ -1672,8 +1668,7 @@ hypervImageManagementServiceGetVHDSD(hypervPrivate *priv,
     params = hypervCreateInvokeParamsList("GetVirtualHardDiskSettingData",
                                           MSVM_IMAGEMANAGEMENTSERVICE_SELECTOR,
                                           Msvm_ImageManagementService_WmiInfo);
-    if (hypervAddSimpleParam(params, "Path", vhdPath) < 0)
-        return -1;
+    hypervAddSimpleParam(params, "Path", vhdPath);
 
     if (hypervInvokeMethod(priv, &params, &response) < 0)
         return -1;

diff --git a/src/hyperv/hyperv_wmi.h b/src/hyperv/hyperv_wmi.h
index 36ca85a5921a801756d48c4377bc1babb9dfcb77..706fd85e8793ed4294c94374384306b66c9d6e41 100644 (file)
--- a/src/hyperv/hyperv_wmi.h
+++ b/src/hyperv/hyperv_wmi.h
@@ -141,8 +141,8 @@ hypervInvokeParamsList *hypervCreateInvokeParamsList(const char *method,
 void hypervFreeInvokeParams(hypervInvokeParamsList *params);
 G_DEFINE_AUTOPTR_CLEANUP_FUNC(hypervInvokeParamsList, hypervFreeInvokeParams);
 
-int hypervAddSimpleParam(hypervInvokeParamsList *params, const char *name,
-                         const char *value);
+void hypervAddSimpleParam(hypervInvokeParamsList *params, const char *name,
+                          const char *value);
 
 int hypervAddEprParam(hypervInvokeParamsList *params,
                       const char *name,