Changeset 35445 in vbox for trunk/src/VBox/Runtime

Timestamp:

Jan 10, 2011 11:05:23 AM (14 years ago)

Author:

vboxsync

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

69316

Message:

Runtime/errmsgwin: add a heuristics for providing a human readable error message in more cases, avoiding the "Unknown Status" all over the place.

File:

• trunk/src/VBox/Runtime/win/errmsgwin.cpp (modified) (2 diffs)

trunk/src/VBox/Runtime/win/errmsgwin.cpp

@@ -5 +5 @@
 
 /*
- * Copyright (C) 2006-2007 Oracle Corporation
+ * Copyright (C) 2006-2011 Oracle Corporation
  *
  * This file is part of VirtualBox Open Source Edition (OSE), as
@@ -81 +81 @@
             return &g_aStatusMsgs[i];
 
+    /* The g_aStatusMsgs table contains a wild mix of error codes with and
+     * without included facility and severity. So the chance is high that there
+     * was no exact match. Try to find a non-exact match, and include the
+     * actual value in case we pick the wrong entry. Better than always using
+     * the "Unknown Status" case. */
+    for (i = 0; i < RT_ELEMENTS(g_aStatusMsgs); i++)
+        if (g_aStatusMsgs[i].iCode == HRESULT_CODE(rc))
+        {
+            int32_t iMsg = (ASMAtomicIncU32(&g_iUnknownMsgs) - 1) % RT_ELEMENTS(g_aUnknownMsgs);
+            RTStrPrintf(&g_aszUnknownStr[iMsg][0], sizeof(g_aszUnknownStr[iMsg]), "%s 0x%X", g_aStatusMsgs[i].pszDefine, rc);
+            return &g_aUnknownMsgs[iMsg];
+        }
+
     /*
      * Need to use the temporary stuff.