Changeset 17348 in vbox for trunk/src/VBox/Runtime/common

Timestamp:

Mar 4, 2009 3:34:40 PM (16 years ago)

Author:

vboxsync

Message:

IPRT/assert: don't create a debug log on assert if we've got a release log and logging isn't enabled.

File:

• trunk/src/VBox/Runtime/common/misc/assert.cpp (modified) (2 diffs)

trunk/src/VBox/Runtime/common/misc/assert.cpp

@@ -42 +42 @@
 
 
-#if defined(IN_GUEST_R0) && (defined(RT_OS_LINUX) || defined(RT_OS_WINDOWS)) 
-/* 
- * This is legacy that should be eliminated. OS specific code deals with 
+#if defined(IN_GUEST_R0) && (defined(RT_OS_LINUX) || defined(RT_OS_WINDOWS))
+/*
+ * This is legacy that should be eliminated. OS specific code deals with
  * R0 assertions now and it will do the backdoor printfs in addition to
  * proper OS specific printfs and panics / BSODs / IPEs.
@@ -134 +134 @@
     }
 
-    pLog = RTLogDefaultInstance();
-    if (pLog)
-    {
-        RTLogPrintf("\n!!Assertion Failed!!\n"
-                    "Expression: %s\n"
-                    "Location  : %s(%d) %s\n",
-                    pszExpr, pszFile, uLine, pszFunction);
-        RTLogFlush(pLog);
+#ifndef LOG_ENABLED
+    if (!pLog)
+#endif
+    {
+        pLog = RTLogDefaultInstance();
+        if (pLog)
+        {
+            RTLogPrintf("\n!!Assertion Failed!!\n"
+                        "Expression: %s\n"
+                        "Location  : %s(%d) %s\n",
+                        pszExpr, pszFile, uLine, pszFunction);
+            RTLogFlush(pLog);
+        }
     }