# HG changeset patch # User glx # Date 2007-12-09 21:20:21 # Node ID d67dab6255c1b152fb44699501f4a85a7c10f4db # Parent e9fae92dc613ea7b8e7592e8d42e4e8c06804d7b (svn r11611) -Codechange: it is now possible to use a define to enable asserts and show them in crash.log for MSVC release builds diff --git a/src/blitter/factory.hpp b/src/blitter/factory.hpp --- a/src/blitter/factory.hpp +++ b/src/blitter/factory.hpp @@ -41,7 +41,7 @@ protected: if (name == NULL) return; this->name = strdup(name); -#if !defined(NDEBUG) +#if !defined(NDEBUG) || defined(WITH_ASSERT) /* NDEBUG disables asserts and gives a warning: unused variable 'P' */ std::pair P = #endif /* !NDEBUG */ diff --git a/src/driver.h b/src/driver.h --- a/src/driver.h +++ b/src/driver.h @@ -79,7 +79,7 @@ protected: strecpy(buf, GetDriverTypeName(type), lastof(buf)); strecpy(buf + 5, name, lastof(buf)); -#if !defined(NDEBUG) +#if !defined(NDEBUG) || defined(WITH_ASSERT) /* NDEBUG disables asserts and gives a warning: unused variable 'P' */ std::pair P = #endif /* !NDEBUG */ diff --git a/src/stdafx.h b/src/stdafx.h --- a/src/stdafx.h +++ b/src/stdafx.h @@ -207,6 +207,13 @@ #define strcasecmp stricmp #define strncasecmp strnicmp #endif + + void SetExceptionString(const char* s, ...); + + #if defined(NDEBUG) && defined(WITH_ASSERT) + #undef assert + #define assert(expression) if (!(expression)) { SetExceptionString("Assertion failed at %s:%d: %s", __FILE__, __LINE__, #expression); *(byte*)0 = 0; } + #endif #endif /* defined(_MSC_VER) */ #if defined(WINCE) diff --git a/src/win32.cpp b/src/win32.cpp --- a/src/win32.cpp +++ b/src/win32.cpp @@ -81,6 +81,17 @@ bool LoadLibraryList(Function proc[], co #ifdef _MSC_VER static const char *_exception_string = NULL; +void SetExceptionString(const char *s, ...) +{ + va_list va; + char buf[512]; + + va_start(va, s); + vsnprintf(buf, lengthof(buf), s, va); + va_end(va); + + _exception_string = strdup(buf); +} #endif void ShowOSErrorBox(const char *buf)