domain_conf: drop unused ref-count in snapshot object

The ref count was assigned to 1 at creation, then never modified again
until it was decremented just before freeing the object.

* src/conf/domain_conf.h (_virDomainSnapshotObj): Delete unused
field.
(virDomainSnapshotObjUnref): Delete unused prototype.
* src/libvirt_private.syms: Likewise.
* src/conf/domain_conf.c (virDomainSnapshotObjNew)
(virDomainSnapshotObjListDataFree): Update users.
(virDomainSnapshotObjUnref): Delete.

diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index c6373005eb1d061811ccb63f98bdd7f6628fc9d7..1b02c25bb278526604786cbbcf403926781b2483 100644 (file)
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
@@ -8722,8 +8722,6 @@ static virDomainSnapshotObjPtr virDomainSnapshotObjNew(void)
         return NULL;
     }
 
-    snapshot->refs = 1;
-
     VIR_DEBUG("obj=%p", snapshot);
 
     return snapshot;
@@ -8740,17 +8738,6 @@ static void virDomainSnapshotObjFree(virDomainSnapshotObjPtr snapshot)
     VIR_FREE(snapshot);
 }
 
-int virDomainSnapshotObjUnref(virDomainSnapshotObjPtr snapshot)
-{
-    snapshot->refs--;
-    VIR_DEBUG("obj=%p refs=%d", snapshot, snapshot->refs);
-    if (snapshot->refs == 0) {
-        virDomainSnapshotObjFree(snapshot);
-        return 0;
-    }
-    return snapshot->refs;
-}
-
 virDomainSnapshotObjPtr virDomainSnapshotAssignDef(virDomainSnapshotObjListPtr snapshots,
                                                    const virDomainSnapshotDefPtr def)
 {
@@ -8782,7 +8769,7 @@ virDomainSnapshotObjListDataFree(void *payload,
 {
     virDomainSnapshotObjPtr obj = payload;
 
-    virDomainSnapshotObjUnref(obj);
+    virDomainSnapshotObjFree(obj);
 }
 
 int virDomainSnapshotObjListInit(virDomainSnapshotObjListPtr snapshots)

diff --git a/src/conf/domain_conf.h b/src/conf/domain_conf.h
index cbd0f98d790847ba65b0465ef8fc479b8f66e920..9f595d6d9c7b69f9cc9e14210cf648925657e704 100644 (file)
--- a/src/conf/domain_conf.h
+++ b/src/conf/domain_conf.h
@@ -998,8 +998,6 @@ struct _virDomainSnapshotDef {
 typedef struct _virDomainSnapshotObj virDomainSnapshotObj;
 typedef virDomainSnapshotObj *virDomainSnapshotObjPtr;
 struct _virDomainSnapshotObj {
-    int refs;
-
     virDomainSnapshotDefPtr def;
 };
 
@@ -1028,7 +1026,6 @@ virDomainSnapshotObjPtr virDomainSnapshotFindByName(const virDomainSnapshotObjLi
                                                     const char *name);
 void virDomainSnapshotObjListRemove(virDomainSnapshotObjListPtr snapshots,
                                     virDomainSnapshotObjPtr snapshot);
-int virDomainSnapshotObjUnref(virDomainSnapshotObjPtr snapshot);
 int virDomainSnapshotHasChildren(virDomainSnapshotObjPtr snap,
                                 virDomainSnapshotObjListPtr snapshots);
 

diff --git a/src/libvirt_private.syms b/src/libvirt_private.syms
index 55be36e7d8571c4cd9b9f4981e02a1359c2fde83..b4b6c63f6a677bdd7cef00e0900163921b295857 100644 (file)
--- a/src/libvirt_private.syms
+++ b/src/libvirt_private.syms
@@ -305,7 +305,6 @@ virDomainSnapshotHasChildren;
 virDomainSnapshotObjListGetNames;
 virDomainSnapshotObjListNum;
 virDomainSnapshotObjListRemove;
-virDomainSnapshotObjUnref;
 virDomainSoundDefFree;
 virDomainSoundModelTypeFromString;
 virDomainSoundModelTypeToString;