Changeset 16040 in vbox

Timestamp:

Jan 19, 2009 11:11:12 AM (16 years ago)

Author:

vboxsync

Message:

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

• use VBOX_E_OBJECT_NOT_FOUND instead of VBOX_E_FILE_ERROR in findMachine() and update documentation accordingly.

Location:

trunk/src/VBox/Main

Files:

• VirtualBoxImpl.cpp (modified) (1 diff)
• idl/VirtualBox.xidl (modified) (5 diffs)

trunk/src/VBox/Main/VirtualBoxImpl.cpp

@@ -1018 +1018 @@
     HRESULT rc = machine
         ? S_OK
-        : setError (VBOX_E_FILE_ERROR,
+        : setError (VBOX_E_OBJECT_NOT_FOUND,
             tr ("Could not find a registered machine named '%ls'"), aName);
 

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

@@ -1647 +1647 @@
         </note>
 
+        <result name="VBOX_E_OBJECT_NOT_FOUND">
+          No matching virtual machine found.
+        </result>
         <result name="VBOX_E_INVALID_OBJECT_STATE">
           Virtual machine was not created within this VirtualBox instance.
@@ -1661 +1664 @@
         instead.
 
-        <result name="VBOX_E_FILE_ERROR">
+        <result name="VBOX_E_OBJECT_NOT_FOUND">
           Could not find registered machine matching @a id.
         </result>
@@ -1676 +1679 @@
         instead.
 
-        <result name="VBOX_E_FILE_ERROR">
+        <result name="VBOX_E_OBJECT_NOT_FOUND">
           Could not find registered machine matching @a name.
         </result>
@@ -1712 +1715 @@
         </note>
 
-        <result name="VBOX_E_FILE_ERROR">
+        <result name="VBOX_E_OBJECT_NOT_FOUND">
           Could not find registered machine matching @a id.
         </result>
@@ -2304 +2307 @@
         <result name="E_ACCESSDENIED">
           Process not started by OpenRemoteSession.
+        </result>
+        <result name="VBOX_E_OBJECT_NOT_FOUND">
+          No matching virtual machine found.
         </result>
         <result name="VBOX_E_INVALID_OBJECT_STATE">