[PATCH] build: avoid 'devname' for BSD

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Commit 21b5daa1 was the last time we cleaned this up.

* tools/virt-host-validate-common.c (virHostValidateDevice):
Rename local variable.
---

Pushing under the build-breaker rule.
I considered adding a syntax-check rule to catch this, but we use
'devname' in enough other places where the name pollution was not
happening as to make writing a rule difficult.

 tools/virt-host-validate-common.c |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/tools/virt-host-validate-common.c b/tools/virt-host-validate-common.c
index bd118be..066343c 100644
--- a/tools/virt-host-validate-common.c
+++ b/tools/virt-host-validate-common.c
@@ -113,13 +113,13 @@ void virHostMsgFail(virHostValidateLevel level,


 int virHostValidateDevice(const char *hvname,
-                          const char *devname,
+                          const char *dev_name,
                           virHostValidateLevel level,
                           const char *hint)
 {
-    virHostMsgCheck(hvname, "for device %s", devname);
+    virHostMsgCheck(hvname, "for device %s", dev_name);

-    if (access(devname, R_OK|W_OK) < 0) {
+    if (access(dev_name, R_OK|W_OK) < 0) {
         virHostMsgFail(level, hint);
         return -1;
     }
-- 
1.7.7.6

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


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]