Changeset 107240 in vbox for trunk/src/VBox/Main/src-all

Timestamp:

Dec 6, 2024 10:46:24 AM (3 months ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

166254

Message:

bugref:10806. New TrackedObjectState enum usage. Added member m_deletionTime. Added the functions updateState, deletionTime. Added getters idleTime, lifeTime, deletionTime, creationTime. Fixed the function resetState and getter state.

File:

• trunk/src/VBox/Main/src-all/ObjectsTracker.cpp (modified) (9 diffs)

trunk/src/VBox/Main/src-all/ObjectsTracker.cpp

@@ -47 +47 @@
     m_lifeTime(0),
     m_idleTime(1),
-    m_State(TrackedObjectData::Invalid),
+    m_state(TrackedObjectState_None),
     m_pIface(NULL)
     {
@@ -68 +68 @@
     RTTimeNow(unconst(&m_creationTime));
     m_lastAccessTime = m_creationTime;
-    m_State = TrackedObjectData::Valid;
+    m_state = TrackedObjectState_Alive;
 }
 
@@ -83 +83 @@
         m_pIface = that.m_pIface;
         m_creationTime = that.m_creationTime;
+        m_deletionTime = that.m_deletionTime;
         m_lastAccessTime = that.m_lastAccessTime;
         m_idleTimeStart = that.m_idleTimeStart;
         m_fIdleTimeStart = that.m_fIdleTimeStart;
-        m_State = that.m_State;
+        m_state = that.m_state;
     }
 }
@@ -111 +112 @@
         m_pIface = that.m_pIface;
         m_creationTime = that.m_creationTime;
+        m_deletionTime = that.m_deletionTime;
         m_lastAccessTime = that.m_lastAccessTime;
         m_idleTimeStart = that.m_idleTimeStart;
         m_fIdleTimeStart = that.m_fIdleTimeStart;
-        m_State = that.m_State;
+        m_state = that.m_state;
     }
 
@@ -134 +136 @@
     if (!m_fIdleTimeStart)
     {
+        RTTimeNow(unconst(&m_deletionTime));
+        updateState(TrackedObjectState_Deleted);//Alive -> Deleted
         RTTimeNow(unconst(&m_idleTimeStart));
         m_fIdleTimeStart = true;
@@ -149 +153 @@
 
     return com::Utf8Str(szCreationTime);
+}
+
+TrackedObjectState_T TrackedObjectData::deletionTime(PRTTIMESPEC aTime) const
+{
+    if (m_state != TrackedObjectState_Alive)
+        *aTime = m_deletionTime;
+    return m_state;
 }
 
@@ -167 +178 @@
     }
     return cRefs;
+}
+
+TrackedObjectState_T TrackedObjectData::updateState(TrackedObjectState_T aNewState)
+{
+    return m_state < aNewState ? m_state = aNewState : m_state;
 }
 
@@ -345 +361 @@
         if ( i_getObj(aObjId).getInterface().isNotNull() )
         {
-            /* Excessive check because user may get only the valid objects Ids. But for 200% assurance it's here */
-            if (i_getObj(aObjId).state() == TrackedObjectData::Valid)
+            if (i_getObj(aObjId).state() != TrackedObjectState_Invalid)
             {
                 aObjData = i_getObj(aObjId);
@@ -666 +681 @@
     if (pIter != m_trackedObjectsData.end())
     {
-        pIter->second.resetState();
+        pIter->second.updateState(TrackedObjectState_Invalid);//Deleted -> Invalid
         m_trackedInvalidObjectIds.insert(aObjId);
         hrc = S_OK;