Changeset 8372

Timestamp:

May 1, 2011, 9:02:38 PM (13 years ago)

Author:

rgrieder

Message:

Improved output of OrxAssert (line number, function name and file) and added OrxVerify.
You can use the latter like OrxAssert, but the condition will also be evaluated in release mode. However abort() will still only be called in debug mode.

File:

• code/trunk/src/libraries/util/OrxAssert.h (modified) (2 diffs)

code/trunk/src/libraries/util/OrxAssert.h

@@ -40 +40 @@
 
 #include <cassert>
-#include "OutputHandler.h"
+#include "Debug.h"
 
 #ifndef NDEBUG
+
 /** Run time assertion like assert(), but with an embedded message.
 @details
@@ -51 +52 @@
     @endcode
 */
-#define OrxAssert(Assertion, ErrorMessage) \
-    Assertion ? ((void)0) : (void)(orxonox::OutputHandler::getOutStream(1) << ErrorMessage << std::endl); \
-    assert(Assertion)
+#define OrxAssert(condition, errorMessage) \
+    do \
+    { \
+        if (!(condition)) \
+        { \
+            COUT(1) << __FILE__ << "(" << __LINE__ << "): "; \
+            COUT(1) << "Assertion failed in " << __FUNCTIONNAME__ << std::endl; \
+            COUT(1) << "Expression: " << #condition << std::endl; \
+            COUT(1) << "Message   : " << errorMessage << std::endl; \
+            /* Don't use the condition again to avoid double evaluation */ \
+            /* Instead, stringify the expression and negate it */ \
+            assert(!#condition); \
+        } \
+    } while (false)
+
+/** Works like OrxAssert in debug mode, but also checks the condition in release
+    mode (no abort() triggered then).
+@details
+    The message will be printed as error with COUT(1). <br>
+    You can use the same magic here as you can with \ref ThrowException
+    @code
+        OrxVerify(condition, "Text: " << number << " more text");
+    @endcode
+*/
+#define OrxVerify(condition, errorMessage) \
+    do \
+    { \
+        if (!(condition)) \
+        { \
+            COUT(1) << __FILE__ << "(" << __LINE__ << "): "; \
+            COUT(1) << "Verification failed in " << __FUNCTIONNAME__ << std::endl; \
+            COUT(1) << "Expression: " << #condition << std::endl; \
+            COUT(1) << "Message   : " << errorMessage << std::endl; \
+            /* Don't use the condition again to avoid double evaluation */ \
+            /* Instead, stringify the expression and negate it */ \
+            assert(!#condition); \
+        } \
+    } while (false)
+
 #else
+
 #define OrxAssert(condition, errorMessage)  ((void)0)
+
+#define OrxVerify(condition, errorMessage) \
+    do \
+    { \
+        if (!(condition)) \
+        { \
+            COUT(1) << __FILE__ << "(" << __LINE__ << "): "; \
+            COUT(1) << "Verification failed in " << __FUNCTIONNAME__ << std::endl; \
+            COUT(1) << "Expression: " << #condition << std::endl; \
+            COUT(1) << "Message   : " << errorMessage << std::endl; \
+            /* No assert() in release configuration */ \
+        } \
+    } while (false)
+
 #endif