Changeset 14972 in vbox for trunk/src/VBox/Main/BIOSSettingsImpl.cpp

Timestamp:

Dec 4, 2008 12:10:37 PM (16 years ago)

Author:

vboxsync

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

40340

Message:

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

The mega commit that implements Main-wide usage of new CheckCom*
macros, mostly CheckComArgNotNull, CheckComArgStrNotEmptyOrNull,
CheckComArgOutSafeArrayPointerValid, CheckComArgExpr.
Note that some methods incorrectly returned E_INVALIDARG where they
should have returned E_POINTER and vice versa. If any higher level
function tests these, they will behave differently now...

Special thanks to: vi macros, making it easy to semi-automatically
find and replace several hundred instances of if (!aName) ...

File:

• trunk/src/VBox/Main/BIOSSettingsImpl.cpp (modified) (1 diff)

trunk/src/VBox/Main/BIOSSettingsImpl.cpp

@@ -412 +412 @@
 STDMETHODIMP BIOSSettings::COMGETTER(IDEControllerType)(IDEControllerType_T *aControllerType)
 {
-    if (!aControllerType)
-        return E_POINTER;
+    CheckComArgOutPointerValid(aControllerType);
 
     AutoCaller autoCaller (this);