Changeset 44503 in vbox for trunk/src/VBox/Main/src-server

Timestamp:

Feb 1, 2013 6:28:53 AM (12 years ago)

Author:

vboxsync

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

83539

Message:

Main: change API. Interface IMediumFormat was touched. Attribute capabilities got the type MediumFormatCapabilities and option safeArray. PR5434.

Location:

trunk/src/VBox/Main/src-server

Files:

• ApplianceImplImport.cpp (modified) (1 diff)
• MachineImplCloneVM.cpp (modified) (1 diff)
• MediumFormatImpl.cpp (modified) (1 diff)
• SnapshotImpl.cpp (modified) (1 diff)

trunk/src/VBox/Main/src-server/ApplianceImplImport.cpp

@@ -1835 +1835 @@
             /* Check the capabilities. We need create capabilities. */
             ULONG lCabs = 0;
-            rc = trgFormat->COMGETTER(Capabilities)(&lCabs);
+            com::SafeArray <MediumFormatCapabilities_T> mediumFormatCap;
+            rc = trgFormat->COMGETTER(Capabilities)(ComSafeArrayAsOutParam(mediumFormatCap));
+
             if (FAILED(rc)) throw rc;
+            else
+            {
+                for (ULONG j = 0; j < mediumFormatCap.size(); j++)
+                    lCabs |= mediumFormatCap[j];
+            }
+
             if (!(   ((lCabs & MediumFormatCapabilities_CreateFixed) == MediumFormatCapabilities_CreateFixed)
                   || ((lCabs & MediumFormatCapabilities_CreateDynamic) == MediumFormatCapabilities_CreateDynamic)))

trunk/src/VBox/Main/src-server/MachineImplCloneVM.cpp

@@ -1090 +1090 @@
                         rc = pMedium->COMGETTER(MediumFormat)(pSrcFormat.asOutParam());
                         ULONG uSrcCaps = 0;
-                        rc = pSrcFormat->COMGETTER(Capabilities)(&uSrcCaps);
+                        com::SafeArray <MediumFormatCapabilities_T> mediumFormatCap;
+                        rc = pSrcFormat->COMGETTER(Capabilities)(ComSafeArrayAsOutParam(mediumFormatCap));
+
                         if (FAILED(rc)) throw rc;
+                        else
+                        {
+                            for (ULONG j = 0; j < mediumFormatCap.size(); j++)
+                                uSrcCaps |= mediumFormatCap[j];
+                        }
 
                         /* Default format? */

trunk/src/VBox/Main/src-server/MediumFormatImpl.cpp

@@ -218 +218 @@
 }
 
-STDMETHODIMP MediumFormat::COMGETTER(Capabilities)(ULONG *aCaps)
-{
-    CheckComArgOutPointerValid(aCaps);
-
-    AutoCaller autoCaller(this);
-    if (FAILED(autoCaller.rc())) return autoCaller.rc();
-
-    /* m.capabilities is const, no need to lock */
-
-    /// @todo add COMGETTER(ExtendedCapabilities) when we reach the 32 bit
-    /// limit (or make the argument ULONG64 after checking that COM is capable
-    /// of defining enums (used to represent bit flags) that contain 64-bit
-    /// values). Or go away from the enum/ulong hack for bit sets and use
-    /// a safearray like elsewhere.
-    ComAssertRet((uint64_t)m.capabilities == ((ULONG)m.capabilities), E_FAIL);
-
-    *aCaps = (ULONG)m.capabilities;
+STDMETHODIMP MediumFormat::COMGETTER(Capabilities)(ComSafeArrayOut(MediumFormatCapabilities_T, aCaps))
+{
+    CheckComArgOutSafeArrayPointerValid(aCaps);
+
+    AutoCaller autoCaller(this);
+    if (FAILED(autoCaller.rc())) return autoCaller.rc();
+
+    AutoReadLock alock(this COMMA_LOCKVAL_SRC_POS);
+
+    SafeArray<MediumFormatCapabilities_T> capabilities(sizeof(MediumFormatCapabilities_T)*8);
+
+    for (ULONG i = 0; i < capabilities.size(); ++i)
+    {
+        ULONG temp = m.capabilities;
+        temp &= 1<<i;
+        capabilities [i] = (MediumFormatCapabilities_T)temp;
+    }
+
+    capabilities.detachTo(ComSafeArrayOutArg(aCaps));
 
     return S_OK;

trunk/src/VBox/Main/src-server/SnapshotImpl.cpp

@@ -2536 +2536 @@
                     throw rc;
                 ULONG uTargetCaps = 0;
-                rc = pTargetFormat->COMGETTER(Capabilities)(&uTargetCaps);
-                if (FAILED(rc))
-                    throw rc;
+                com::SafeArray <MediumFormatCapabilities_T> mediumFormatCap;
+                rc = pTargetFormat->COMGETTER(Capabilities)(ComSafeArrayAsOutParam(mediumFormatCap));
+
+                if (FAILED(rc)) throw rc;
+                else
+                {
+                    for (ULONG j = 0; j < mediumFormatCap.size(); j++)
+                        uTargetCaps |= mediumFormatCap[j];
+                }
 
                 if (uTargetCaps & MediumFormatCapabilities_File)