Changeset 35115 in vbox for trunk/src

Timestamp:

Dec 15, 2010 9:37:13 AM (14 years ago)

Author:

vboxsync

Message:

Frontends/VBoxManage: reorder code so that the medium type is set before attaching it to a storage controller

File:

• trunk/src/VBox/Frontends/VBoxManage/VBoxManageStorageController.cpp (modified) (2 diffs)

trunk/src/VBox/Frontends/VBoxManage/VBoxManageStorageController.cpp

@@ -556 +556 @@
                 if (!pMedium2Mount)
                     throw Utf8StrFmt("Invalid UUID or filename \"%s\"", pszMedium);
+            }
+
+            // set medium type, if so desired
+            if (pMedium2Mount && mediumType != MediumType_Normal)
+            {
+                CHECK_ERROR(pMedium2Mount, COMSETTER(Type)(mediumType));
+            }
+
+            if (pMedium2Mount && !bstrComment.isEmpty())
+            {
+                CHECK_ERROR(pMedium2Mount, COMSETTER(Description)(bstrComment.raw()));
             }
 
@@ -621 +632 @@
                 break;
             }
-
-            // set medium type, if so desired
-            if (pMedium2Mount && mediumType != MediumType_Normal)
-            {
-                CHECK_ERROR(pMedium2Mount, COMSETTER(Type)(mediumType));
-            }
-
-            if (!bstrComment.isEmpty())
-            {
-                CHECK_ERROR(pMedium2Mount, COMSETTER(Description)(bstrComment.raw()));
-            }
         }