Changeset 15649 in vbox

Timestamp:

Dec 18, 2008 12:37:11 PM (16 years ago)

Author:

vboxsync

Message:

#3285: Improve error handling API to include unique error numbers
Document

• IHardDiskFormat::describeProperties

Location:

trunk/src/VBox/Main

Files:

• HardDiskFormatImpl.cpp (modified) (1 diff)
• idl/VirtualBox.xidl (modified) (1 diff)

trunk/src/VBox/Main/HardDiskFormatImpl.cpp

@@ -243 +243 @@
                                                 ComSafeArrayOut (BSTR, aDefaults))
 {
-    if (ComSafeArrayOutIsNull (aNames) ||
-        ComSafeArrayOutIsNull (aDescriptions) ||
-        ComSafeArrayOutIsNull (aTypes) ||
-        ComSafeArrayOutIsNull (aFlags) ||
-        ComSafeArrayOutIsNull (aDefaults))
-        return E_POINTER;
+    CheckComArgSafeArrayNotNull(aNames);
+    CheckComArgSafeArrayNotNull(aDescriptions);
+    CheckComArgSafeArrayNotNull(aTypes);
+    CheckComArgSafeArrayNotNull(aFlags);
+    CheckComArgSafeArrayNotNull(aDefaults);
 
     AutoCaller autoCaller (this);

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

@@ -8672 +8672 @@
         same and corresponds to the number of supported properties.
 
-        The returned arrays are not empty only if the
+        The returned arrays are filled in only if the
         <link to="HardDiskFormatCapabilities_Properties"/> flag is set.
+        All arguments must be non-NULL.
 
         <see>DataType</see>