[libvirt] [PATCH 1/6] qemu_conf.c: Fix naming of *AddRemove* functions

Michal Privoznik mprivozn at redhat.com
Mon Sep 9 16:00:22 UTC 2019


Our naming rules prefer qemuObjectOperation() scheme rather than
qemuOperationObject() for function names. These were not honoured
in recent commits to qemu_conf.c.

Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
---
 src/qemu/qemu_conf.c | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/src/qemu/qemu_conf.c b/src/qemu/qemu_conf.c
index 86b2eef060..32d411e536 100644
--- a/src/qemu/qemu_conf.c
+++ b/src/qemu/qemu_conf.c
@@ -1679,7 +1679,7 @@ qemuSharedDeviceEntryRemove(virQEMUDriverPtr driver,
 
 
 static int
-qemuAddRemoveSharedDiskInternal(virQEMUDriverPtr driver,
+qemuSharedDiskAddRemoveInternal(virQEMUDriverPtr driver,
                                 virDomainDiskDefPtr disk,
                                 const char *name,
                                 bool addDisk)
@@ -1730,7 +1730,7 @@ qemuAddSharedDisk(virQEMUDriverPtr driver,
                   virDomainDiskDefPtr disk,
                   const char *name)
 {
-    return qemuAddRemoveSharedDiskInternal(driver, disk, name, true);
+    return qemuSharedDiskAddRemoveInternal(driver, disk, name, true);
 }
 
 
@@ -1768,7 +1768,7 @@ qemuGetHostdevPath(virDomainHostdevDefPtr hostdev)
 
 
 static int
-qemuAddRemoveSharedHostdevInternal(virQEMUDriverPtr driver,
+qemuSharedHostdevAddRemoveInternal(virQEMUDriverPtr driver,
                                    virDomainHostdevDefPtr hostdev,
                                    const char *name,
                                    bool addDevice)
@@ -1803,7 +1803,7 @@ qemuAddRemoveSharedHostdevInternal(virQEMUDriverPtr driver,
 }
 
 static int
-qemuAddRemoveSharedDeviceInternal(virQEMUDriverPtr driver,
+qemuSharedDeviceAddRemoveInternal(virQEMUDriverPtr driver,
                                   virDomainDeviceDefPtr dev,
                                   const char *name,
                                   bool addDevice)
@@ -1813,10 +1813,10 @@ qemuAddRemoveSharedDeviceInternal(virQEMUDriverPtr driver,
      * which is only valid for block disk and scsi host device.
      */
     if (dev->type == VIR_DOMAIN_DEVICE_DISK)
-        return qemuAddRemoveSharedDiskInternal(driver, dev->data.disk,
+        return qemuSharedDiskAddRemoveInternal(driver, dev->data.disk,
                                                name, addDevice);
     else if (dev->type == VIR_DOMAIN_DEVICE_HOSTDEV)
-        return qemuAddRemoveSharedHostdevInternal(driver, dev->data.hostdev,
+        return qemuSharedHostdevAddRemoveInternal(driver, dev->data.hostdev,
                                                   name, addDevice);
     else
         return 0;
@@ -1837,7 +1837,7 @@ qemuAddSharedDevice(virQEMUDriverPtr driver,
                     virDomainDeviceDefPtr dev,
                     const char *name)
 {
-    return qemuAddRemoveSharedDeviceInternal(driver, dev, name, true);
+    return qemuSharedDeviceAddRemoveInternal(driver, dev, name, true);
 }
 
 
@@ -1846,7 +1846,7 @@ qemuRemoveSharedDisk(virQEMUDriverPtr driver,
                      virDomainDiskDefPtr disk,
                      const char *name)
 {
-    return qemuAddRemoveSharedDiskInternal(driver, disk, name, false);
+    return qemuSharedDiskAddRemoveInternal(driver, disk, name, false);
 }
 
 
@@ -1864,7 +1864,7 @@ qemuRemoveSharedDevice(virQEMUDriverPtr driver,
                        virDomainDeviceDefPtr dev,
                        const char *name)
 {
-    return qemuAddRemoveSharedDeviceInternal(driver, dev, name, false);
+    return qemuSharedDeviceAddRemoveInternal(driver, dev, name, false);
 }
 
 
-- 
2.21.0




More information about the libvir-list mailing list