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

Timestamp:

Nov 25, 2008 1:04:37 PM (16 years ago)

Author:

vboxsync

Message:

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

Location:

trunk/src/VBox/Main

Files:

• ConsoleImpl.cpp (modified) (4 diffs)
• idl/VirtualBox.xidl (modified) (1 diff)

trunk/src/VBox/Main/ConsoleImpl.cpp

@@ -1533 +1533 @@
 
     if (mMachineState != MachineState_Paused)
-        return setError (E_FAIL, tr ("Cannot resume the machine as it is "
-                                     "not paused (machine state: %d)"),
-                         mMachineState);
+        return setError (VBOX_E_INVALID_VM_STATE,
+            tr ("Cannot resume the machine as it is not paused "
+                "(machine state: %d)"), mMachineState);
 
     /* protect mpVM */
@@ -1553 +1553 @@
 
     HRESULT rc = VBOX_SUCCESS (vrc) ? S_OK :
-        setError (E_FAIL,
+        setError (VBOX_E_VM_ERROR,
             tr ("Could not resume the machine execution (%Rrc)"), vrc);
 
@@ -1571 +1571 @@
 
     if (mMachineState != MachineState_Running)
-        return setError (E_FAIL, tr ("Cannot power off the machine as it is "
-                                     "not running (machine state: %d)"),
-                         mMachineState);
+        return setError (VBOX_E_INVALID_VM_STATE,
+            tr ("Cannot power off the machine as it is not running "
+                "(machine state: %d)"), mMachineState);
 
     /* protect mpVM */
@@ -1590 +1590 @@
 
     HRESULT rc = VBOX_SUCCESS (vrc) ? S_OK :
-        setError (E_FAIL,
+        setError (VBOX_E_PDM_ERROR,
             tr ("Controlled power off failed (%Rrc)"), vrc);
 

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

@@ -185 +185 @@
   </result>
 
+  <result name="VBOX_E_PDM_ERROR" value="0x80BB0006">
+    <desc>
+       Pluggable Device Manager error.
+    </desc>
+  </result>
+
   <!--
   // all common enums