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

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

Files:

• idl/VirtualBox.xidl (modified) (2 diffs)
• include/MediumFormatImpl.h (modified) (1 diff)
• src-client/ConsoleImpl2.cpp (modified) (2 diffs)
• src-server/ApplianceImplImport.cpp (modified) (1 diff)
• src-server/MachineImplCloneVM.cpp (modified) (1 diff)
• src-server/MediumFormatImpl.cpp (modified) (1 diff)
• src-server/SnapshotImpl.cpp (modified) (1 diff)

trunk/src/VBox/Main/idl/VirtualBox.xidl

@@ -13412 +13412 @@
   <interface
     name="IMediumFormat" extends="$unknown"
-    uuid="9bd5b655-ea47-4637-99f3-aad0948be35b"
+    uuid="6238e1cf-a17d-4ec1-8172-418bfb22b93a"
     wsmap="managed"
     >
@@ -13460 +13460 @@
     </attribute>
 
-    <attribute name="capabilities" type="unsigned long" readonly="yes">
+    <attribute name="capabilities" type="MediumFormatCapabilities" safearray="yes" readonly="yes">
       <desc>
         Capabilities of the format as a set of bit flags.

trunk/src/VBox/Main/include/MediumFormatImpl.h

@@ -91 +91 @@
     STDMETHOD(COMGETTER(Id))(BSTR *aId);
     STDMETHOD(COMGETTER(Name))(BSTR *aName);
-    STDMETHOD(COMGETTER(Capabilities))(ULONG *aCaps);
+    STDMETHOD(COMGETTER(Capabilities))(ComSafeArrayOut(MediumFormatCapabilities_T, aCaps));
 
     // IMediumFormat methods

trunk/src/VBox/Main/src-client/ConsoleImpl2.cpp

@@ -3244 +3244 @@
             ComPtr<IMediumFormat> pMediumFormat;
             hrc = pMedium->COMGETTER(MediumFormat)(pMediumFormat.asOutParam());             H();
-            ULONG uCaps;
-            hrc = pMediumFormat->COMGETTER(Capabilities)(&uCaps);                           H();
+            ULONG uCaps = 0;
+            com::SafeArray <MediumFormatCapabilities_T> mediumFormatCap;
+            hrc = pMediumFormat->COMGETTER(Capabilities)(ComSafeArrayAsOutParam(mediumFormatCap));    H();
+
+            for (ULONG j = 0; j < mediumFormatCap.size(); j++)
+                uCaps |= mediumFormatCap[j];
+
             if (uCaps & MediumFormatCapabilities_File)
             {
@@ -3418 +3423 @@
             ComPtr<IMediumFormat> pMediumFormat;
             hrc = pMedium->COMGETTER(MediumFormat)(pMediumFormat.asOutParam());             H();
-            ULONG uCaps;
-            hrc = pMediumFormat->COMGETTER(Capabilities)(&uCaps);                           H();
+            ULONG uCaps = 0;
+            com::SafeArray <MediumFormatCapabilities_T> mediumFormatCap;
+            hrc = pMediumFormat->COMGETTER(Capabilities)(ComSafeArrayAsOutParam(mediumFormatCap));     H();
+
+            for (ULONG j = 0; j < mediumFormatCap.size(); j++)
+                uCaps |= mediumFormatCap[j];
+
             if (uCaps & MediumFormatCapabilities_File)
             {

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)