[PATCH 01/17] qemuSnapshotRedefine: Rename 'def' to 'snapdef'

Peter Krempa pkrempa at redhat.com
Wed Jan 12 18:10:01 UTC 2022


'def' is commonly used to refer to domain definition. Most of the
snapshot code uses 'snapdef' for the snapshot definition.

Signed-off-by: Peter Krempa <pkrempa at redhat.com>
---
 src/qemu/qemu_snapshot.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/qemu/qemu_snapshot.c b/src/qemu/qemu_snapshot.c
index 5d383279b0..624ace0314 100644
--- a/src/qemu/qemu_snapshot.c
+++ b/src/qemu/qemu_snapshot.c
@@ -1709,7 +1709,7 @@ qemuSnapshotCreateWriteMetadata(virDomainObj *vm,
 static virDomainSnapshotPtr
 qemuSnapshotRedefine(virDomainObj *vm,
                      virDomainPtr domain,
-                     virDomainSnapshotDef *def,
+                     virDomainSnapshotDef *snapdef,
                      virQEMUDriver *driver,
                      virQEMUDriverConfig *cfg,
                      unsigned int flags)
@@ -1717,13 +1717,13 @@ qemuSnapshotRedefine(virDomainObj *vm,
     virDomainMomentObj *snap = NULL;
     virDomainSnapshotPtr ret = NULL;

-    if (virDomainSnapshotRedefinePrep(vm, &def, &snap,
+    if (virDomainSnapshotRedefinePrep(vm, &snapdef, &snap,
                                       driver->xmlopt,
                                       flags) < 0)
         return NULL;

     if (!snap) {
-        if (!(snap = virDomainSnapshotAssignDef(vm->snapshots, def)))
+        if (!(snap = virDomainSnapshotAssignDef(vm->snapshots, snapdef)))
             return NULL;
     }
     /* XXX Should we validate that the redefined snapshot even
-- 
2.31.1




More information about the libvir-list mailing list