Changeset 35175 in vbox for trunk/src/VBox/Main/SnapshotImpl.cpp

Timestamp:

Dec 16, 2010 12:36:00 PM (14 years ago)

Author:

vboxsync

Message:

Main/Machine: rename mSnapshotData to mConsoleTaskData (together with the struct name), since in the future this will hold information for all long-term operations triggered via Console methods

File:

• trunk/src/VBox/Main/SnapshotImpl.cpp (modified) (10 diffs)

trunk/src/VBox/Main/SnapshotImpl.cpp

@@ -1367 +1367 @@
                   || mData->mMachineState == MachineState_Running
                   || mData->mMachineState == MachineState_Paused, E_FAIL);
-    AssertReturn(mSnapshotData.mLastState == MachineState_Null, E_FAIL);
-    AssertReturn(mSnapshotData.mSnapshot.isNull(), E_FAIL);
+    AssertReturn(mConsoleTaskData.mLastState == MachineState_Null, E_FAIL);
+    AssertReturn(mConsoleTaskData.mSnapshot.isNull(), E_FAIL);
 
     if (    !fTakingSnapshotOnline
@@ -1422 +1422 @@
 
     /* fill in the snapshot data */
-    mSnapshotData.mLastState = mData->mMachineState;
-    mSnapshotData.mSnapshot = pSnapshot;
+    mConsoleTaskData.mLastState = mData->mMachineState;
+    mConsoleTaskData.mSnapshot = pSnapshot;
+    /// @todo in the long run the progress object should be moved to
+    // VBoxSVC to avoid trouble with monitoring the progress object state
+    // when the process where it lives is terminating shortly after the
+    // operation completed.
 
     try
@@ -1436 +1440 @@
 
         /* Console::fntTakeSnapshotWorker and friends expects this. */
-        if (mSnapshotData.mLastState == MachineState_Running)
+        if (mConsoleTaskData.mLastState == MachineState_Running)
             setMachineState(MachineState_LiveSnapshotting);
         else
@@ -1449 +1453 @@
             throw rc;
 
-        if (mSnapshotData.mLastState == MachineState_Saved)
+        if (mConsoleTaskData.mLastState == MachineState_Saved)
         {
             Utf8Str stateFrom = mSSData->mStateFilePath;
-            Utf8Str stateTo = mSnapshotData.mSnapshot->stateFilePath();
+            Utf8Str stateTo = mConsoleTaskData.mSnapshot->stateFilePath();
 
             LogFlowThisFunc(("Copying the execution state from '%s' to '%s'...\n",
@@ -1484 +1488 @@
     {
         LogThisFunc(("Caught %Rhrc [%s]\n", hrc, Global::stringifyMachineState(mData->mMachineState) ));
-        if (    mSnapshotData.mLastState != mData->mMachineState
-             && (   mSnapshotData.mLastState == MachineState_Running
+        if (    mConsoleTaskData.mLastState != mData->mMachineState
+             && (   mConsoleTaskData.mLastState == MachineState_Running
                   ? mData->mMachineState == MachineState_LiveSnapshotting
                   : mData->mMachineState == MachineState_Saving)
            )
-            setMachineState(mSnapshotData.mLastState);
+            setMachineState(mConsoleTaskData.mLastState);
 
         pSnapshot->uninit();
         pSnapshot.setNull();
-        mSnapshotData.mLastState = MachineState_Null;
-        mSnapshotData.mSnapshot.setNull();
+        mConsoleTaskData.mLastState = MachineState_Null;
+        mConsoleTaskData.mSnapshot.setNull();
 
         rc = hrc;
@@ -1540 +1544 @@
                  || (    (    mData->mMachineState == MachineState_Saving
                            || mData->mMachineState == MachineState_LiveSnapshotting)
-                      && mSnapshotData.mLastState != MachineState_Null
-                      && !mSnapshotData.mSnapshot.isNull()
+                      && mConsoleTaskData.mLastState != MachineState_Null
+                      && !mConsoleTaskData.mSnapshot.isNull()
                     )
                  , E_FAIL);
@@ -1551 +1555 @@
      * all to avoid races.
      */
-    if (    mData->mMachineState != mSnapshotData.mLastState
-         && mSnapshotData.mLastState != MachineState_Running
+    if (    mData->mMachineState != mConsoleTaskData.mLastState
+         && mConsoleTaskData.mLastState != MachineState_Running
        )
-        setMachineState(mSnapshotData.mLastState);
+        setMachineState(mConsoleTaskData.mLastState);
 
     ComObjPtr<Snapshot> pOldFirstSnap = mData->mFirstSnapshot;
     ComObjPtr<Snapshot> pOldCurrentSnap = mData->mCurrentSnapshot;
 
-    bool fOnline = Global::IsOnline(mSnapshotData.mLastState);
+    bool fOnline = Global::IsOnline(mConsoleTaskData.mLastState);
 
     HRESULT rc = S_OK;
@@ -1566 +1570 @@
     {
         // new snapshot becomes the current one
-        mData->mCurrentSnapshot = mSnapshotData.mSnapshot;
+        mData->mCurrentSnapshot = mConsoleTaskData.mSnapshot;
 
         /* memorize the first snapshot if necessary */
@@ -1591 +1595 @@
         /* inform callbacks */
         mParent->onSnapshotTaken(mData->mUuid,
-                                 mSnapshotData.mSnapshot->getId());
+                                 mConsoleTaskData.mSnapshot->getId());
     }
     else
@@ -1603 +1607 @@
 
         /* delete the saved state file (it might have been already created) */
-        if (mSnapshotData.mSnapshot->stateFilePath().length())
-            RTFileDelete(mSnapshotData.mSnapshot->stateFilePath().c_str());
-
-        mSnapshotData.mSnapshot->uninit();
+        if (mConsoleTaskData.mSnapshot->stateFilePath().length())
+            RTFileDelete(mConsoleTaskData.mSnapshot->stateFilePath().c_str());
+
+        mConsoleTaskData.mSnapshot->uninit();
     }
 
     /* clear out the snapshot data */
-    mSnapshotData.mLastState = MachineState_Null;
-    mSnapshotData.mSnapshot.setNull();
+    mConsoleTaskData.mLastState = MachineState_Null;
+    mConsoleTaskData.mSnapshot.setNull();
 
     // save VirtualBox.xml (media registry most probably changed with diff image);