Changeset 17260 in vbox for trunk/src/VBox/Frontends/VBoxManage

Timestamp:

Mar 3, 2009 9:16:15 AM (16 years ago)

Author:

vboxsync

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

43608

Message:

#3551: “Main: Replace remaining collections with safe arrays”
Replace SnapshotCollection. Reviewed by dmik. Tested with GUI.

File:

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

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

@@ -66 +66 @@
 
     /* get the children */
-    ComPtr<ISnapshotCollection> coll;
-    rootSnapshot->COMGETTER(Children)(coll.asOutParam());
-    if (coll)
-    {
-        ComPtr<ISnapshotEnumerator> enumerator;
-        coll->Enumerate(enumerator.asOutParam());
-        ULONG index = 0;
-        BOOL hasMore = FALSE;
-        while (enumerator->HasMore(&hasMore), hasMore)
-        {
-            ComPtr<ISnapshot> snapshot;
-            enumerator->GetNext(snapshot.asOutParam());
+    SafeIfaceArray <ISnapshot> coll;
+    rootSnapshot->COMGETTER(Children)(ComSafeArrayAsOutParam(coll));
+    if (!coll.isNull())
+    {
+        for (size_t index = 0; index < coll.size(); ++index)
+        {
+            ComPtr<ISnapshot> snapshot = coll[index];
             if (snapshot)
             {
@@ -88 +83 @@
                 showSnapshots(snapshot, details, newPrefix, level + 1);
             }
-            index++;
         }
     }