From: Jiri Denemark <jdenemar@redhat.com>
Date: Thu, 19 Nov 2020 17:24:01 +0000 (+0100)
Subject: conf: Rename virCPUDefUpdateFeatureInternal
X-Git-Tag: v6.10.0-rc1~24
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=f06bb045498df9a7ca4a9d3c1c7cdbe314622eee;p=thirdparty%2Flibvirt.git

conf: Rename virCPUDefUpdateFeatureInternal

The function is supposed to add a feature to a CPU definition, let's
name it virCPUDefAddFeatureInternal. The behavior in case the feature is
already present in the CPU def is configurable and we will soon add a
new option to not do anything in that case, which wouldn't really work
well with the current *Update* name.

Signed-off-by: Jiri Denemark <jdenemar@redhat.com>
Reviewed-by: Tim Wiederhake <twiederh@redhat.com>
---

diff --git a/src/conf/cpu_conf.c b/src/conf/cpu_conf.c
index 5cf7716b12..55a7925d8a 100644
--- a/src/conf/cpu_conf.c
+++ b/src/conf/cpu_conf.c
@@ -858,10 +858,10 @@ virCPUDefFormatBuf(virBufferPtr buf,
 }
 
 static int
-virCPUDefUpdateFeatureInternal(virCPUDefPtr def,
-                               const char *name,
-                               int policy,
-                               bool update)
+virCPUDefAddFeatureInternal(virCPUDefPtr def,
+                            const char *name,
+                            int policy,
+                            bool update)
 {
     virCPUFeatureDefPtr feat;
 
@@ -898,7 +898,7 @@ virCPUDefUpdateFeature(virCPUDefPtr def,
                        const char *name,
                        int policy)
 {
-    return virCPUDefUpdateFeatureInternal(def, name, policy, true);
+    return virCPUDefAddFeatureInternal(def, name, policy, true);
 }
 
 int
@@ -906,7 +906,7 @@ virCPUDefAddFeature(virCPUDefPtr def,
                     const char *name,
                     int policy)
 {
-    return virCPUDefUpdateFeatureInternal(def, name, policy, false);
+    return virCPUDefAddFeatureInternal(def, name, policy, false);
 }