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

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.

File:

• trunk/src/VBox/Main/src-server/MediumFormatImpl.cpp (modified) (1 diff)

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;