Changeset 31312 in vbox for trunk

Timestamp:

Aug 2, 2010 3:17:45 PM (15 years ago)

Author:

vboxsync

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

64339

Message:

Main: undo rename, readability

File:

• trunk/src/VBox/Main/MachineImpl.cpp (modified) (8 diffs)

trunk/src/VBox/Main/MachineImpl.cpp

@@ -3216 +3216 @@
                         medium->getLocationFull().raw());
 
-    bool fImplicit = false;
+    bool fIndirect = false;
     if (!medium.isNull())
-        fImplicit = medium->isReadOnly();
+        fIndirect = medium->isReadOnly();
     bool associate = true;
 
     do
     {
-        if (aType == DeviceType_HardDisk && mMediaData.isBackedUp())
+        if (    aType == DeviceType_HardDisk
+             && mMediaData.isBackedUp())
         {
             const MediaData::AttachmentList &oldAtts = mMediaData.backedUpData()->mAttachments;
@@ -3232 +3233 @@
             if ((pAttachTemp = findAttachment(oldAtts, medium)))
             {
-                AssertReturn(!fImplicit, E_FAIL);
+                AssertReturn(!fIndirect, E_FAIL);
 
                 /* see if it's the same bus/channel/device */
@@ -3251 +3252 @@
 
         /* go further only if the attachment is to be indirect */
-        if (!fImplicit)
+        if (!fIndirect)
             break;
 
@@ -3319 +3320 @@
                     mediumLock.attach(medium);
                     /* not implicit, doesn't require association with this VM */
-                    fImplicit = false;
+                    fIndirect = false;
                     associate = false;
                     /* go right to the MediumAttachment creation */
@@ -3342 +3343 @@
                 const MediaData::AttachmentList &snapAtts = snap->getSnapshotMachine()->mMediaData->mAttachments;
 
-                MediaData::AttachmentList::const_iterator foundIt = snapAtts.end();
+                MediumAttachment *pAttachFound = NULL;
                 uint32_t foundLevel = 0;
 
@@ -3363 +3364 @@
                          * descendant of medium will be used
                          */
-                        if (    (*it)->getDevice() == aDevice
-                             && (*it)->getPort() == aControllerPort
-                             && (*it)->getControllerName() == aControllerName
+                        if (    pAttach->getDevice() == aDevice
+                             && pAttach->getPort() == aControllerPort
+                             && pAttach->getControllerName() == aControllerName
                            )
                         {
-                            foundIt = it;
+                            pAttachFound = pAttach;
                             break;
                         }
-                        else if (    foundIt == snapAtts.end()
+                        else if (    !pAttachFound
                                   || level > foundLevel /* prefer younger */
                                 )
                         {
-                            foundIt = it;
+                            pAttachFound = pAttach;
                             foundLevel = level;
                         }
@@ -3381 +3382 @@
                 }
 
-                if (foundIt != snapAtts.end())
+                if (pAttachFound)
                 {
-                    base = (*foundIt)->getMedium();
+                    base = pAttachFound->getMedium();
                     break;
                 }
@@ -3469 +3470 @@
                           aDevice,
                           aType,
-                          fImplicit,
+                          fIndirect,
                           0 /* No bandwidth limit */);
     if (FAILED(rc)) return rc;