[PATCH v5 03/13] virdnsmasq: Drop @force argument of dnsmasqCapsRefreshInternal()

Michal Privoznik mprivozn at redhat.com
Tue Jan 18 14:37:10 UTC 2022


This argument is not used really as the only caller passes true
and dnsmasqCapsRefreshInternal() only checks for false value.

Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
Reviewed-by: Andrea Bolognani <abologna at redhat.com>
---
 src/util/virdnsmasq.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/util/virdnsmasq.c b/src/util/virdnsmasq.c
index 9f3da1d5e6..7fcaadda3f 100644
--- a/src/util/virdnsmasq.c
+++ b/src/util/virdnsmasq.c
@@ -649,7 +649,7 @@ dnsmasqCapsSetFromBuffer(dnsmasqCaps *caps, const char *buf)
 }
 
 static int
-dnsmasqCapsRefreshInternal(dnsmasqCaps *caps, bool force)
+dnsmasqCapsRefreshInternal(dnsmasqCaps *caps)
 {
     struct stat sb;
     g_autoptr(virCommand) vercmd = NULL;
@@ -666,7 +666,7 @@ dnsmasqCapsRefreshInternal(dnsmasqCaps *caps, bool force)
                              caps->binaryPath);
         return -1;
     }
-    if (!force && caps->mtime == sb.st_mtime)
+    if (caps->mtime == sb.st_mtime)
         return 0;
     caps->mtime = sb.st_mtime;
 
@@ -734,7 +734,7 @@ dnsmasqCapsNewFromBinary(void)
     if (!caps)
         return NULL;
 
-    if (dnsmasqCapsRefreshInternal(caps, true) < 0)
+    if (dnsmasqCapsRefreshInternal(caps) < 0)
         return NULL;
 
     return g_steal_pointer(&caps);
-- 
2.34.1




More information about the libvir-list mailing list