X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/bddd7a8d8953cf4c09e56c13e1bfcc594ba0267e..f29fe16932ede752589ef9ed34a757342387932f:/include/wx/debug.h diff --git a/include/wx/debug.h b/include/wx/debug.h index 10e902d5c6..f353646e8b 100644 --- a/include/wx/debug.h +++ b/include/wx/debug.h @@ -1,13 +1,15 @@ -///////////////////////////////////////////////////////////////////////////// -// Name: wx/debug.h -// Purpose: Misc debug functions and macros -// Author: Vadim Zeitlin -// Modified by: -// Created: 29/01/98 -// RCS-ID: $Id$ -// Copyright: (c) 1998 Vadim Zeitlin -// Licence: wxWindows licence -///////////////////////////////////////////////////////////////////////////// +/** +* Name: wx/debug.h +* Purpose: Misc debug functions and macros +* Author: Vadim Zeitlin +* Modified by: Ryan Norton (Converted to C) +* Created: 29/01/98 +* RCS-ID: $Id$ +* Copyright: (c) 1998 Vadim Zeitlin +* Licence: wxWindows licence +*/ + +/* THIS IS A C FILE, DON'T USE C++ FEATURES (IN PARTICULAR COMMENTS) IN IT */ #ifndef _WX_DEBUG_H_ #define _WX_DEBUG_H_ @@ -15,60 +17,60 @@ #ifndef __WXWINCE__ #include #endif -#include // for CHAR_BIT used below +#include /* for CHAR_BIT used below */ -#include "wx/wxchar.h" // for __TFILE__ and wxChar +#include "wx/wxchar.h" /* for __TFILE__ and wxChar */ -// ---------------------------------------------------------------------------- -// Defines controlling the debugging macros -// ---------------------------------------------------------------------------- +/* ---------------------------------------------------------------------------- */ +/* Defines controlling the debugging macros */ +/* ---------------------------------------------------------------------------- */ -// if _DEBUG is defined (MS VC++ and others use it in debug builds), define -// __WXDEBUG__ too +/* if _DEBUG is defined (MS VC++ and others use it in debug builds), define */ +/* __WXDEBUG__ too */ #ifdef _DEBUG #ifndef __WXDEBUG__ #define __WXDEBUG__ - #endif // !__WXDEBUG__ -#endif // _DEBUG + #endif /* !__WXDEBUG__ */ +#endif /* _DEBUG */ -// if NDEBUG is defined ( uses it), undef __WXDEBUG__ and WXDEBUG +/* if NDEBUG is defined ( uses it), undef __WXDEBUG__ and WXDEBUG */ #ifdef NDEBUG #undef __WXDEBUG__ #undef WXDEBUG -#endif // NDEBUG +#endif /* NDEBUG */ -// if __WXDEBUG__ is defined, make sure that WXDEBUG is defined and >= 1 +/* if __WXDEBUG__ is defined, make sure that WXDEBUG is defined and >= 1 */ #ifdef __WXDEBUG__ #if !defined(WXDEBUG) || !WXDEBUG #undef WXDEBUG #define WXDEBUG 1 - #endif // !WXDEBUG -#endif // __WXDEBUG__ - -// ---------------------------------------------------------------------------- -// Debugging macros -// -// All debugging macros rely on ASSERT() which in turn calls user-defined -// OnAssert() function. To keep things simple, it's called even when the -// expression is TRUE (i.e. everything is ok) and by default does nothing: just -// returns the same value back. But if you redefine it to do something more sexy -// (popping up a message box in your favourite GUI, sending you e-mail or -// whatever) it will affect all ASSERTs, FAILs and CHECKs in your code. -// -// Warning: if you don't like advice on programming style, don't read -// further! ;-) -// -// Extensive use of these macros is recommended! Remember that ASSERTs are -// disabled in final build (without __WXDEBUG__ defined), so they add strictly -// nothing to your program's code. On the other hand, CHECK macros do stay -// even in release builds, but in general are not much of a burden, while -// a judicious use of them might increase your program's stability. -// ---------------------------------------------------------------------------- - -// Macros which are completely disabled in 'release' mode -// -// NB: these functions are implemented in src/common/appcmn.cpp -#ifdef __WXDEBUG__ + #endif /* !WXDEBUG */ +#endif /* __WXDEBUG__ */ + +/* ---------------------------------------------------------------------------- */ +/* Debugging macros */ +/* */ +/* All debugging macros rely on ASSERT() which in turn calls user-defined */ +/* OnAssert() function. To keep things simple, it's called even when the */ +/* expression is TRUE (i.e. everything is ok) and by default does nothing: just */ +/* returns the same value back. But if you redefine it to do something more sexy */ +/* (popping up a message box in your favourite GUI, sending you e-mail or */ +/* whatever) it will affect all ASSERTs, FAILs and CHECKs in your code. */ +/* */ +/* Warning: if you don't like advice on programming style, don't read */ +/* further! ;-) */ +/* */ +/* Extensive use of these macros is recommended! Remember that ASSERTs are */ +/* disabled in final build (without __WXDEBUG__ defined), so they add strictly */ +/* nothing to your program's code. On the other hand, CHECK macros do stay */ +/* even in release builds, but in general are not much of a burden, while */ +/* a judicious use of them might increase your program's stability. */ +/* ---------------------------------------------------------------------------- */ + +/* Macros which are completely disabled in 'release' mode */ +/* */ +/* NB: these functions are implemented in src/common/appcmn.cpp */ +#if defined(__cplusplus) && defined(__WXDEBUG__) /* this function may be redefined to do something non trivial and is called whenever one of debugging macros fails (i.e. condition is false in an @@ -83,57 +85,60 @@ const wxChar *szCond, const wxChar *szMsg = NULL); - // call this function to break into the debugger unconditionally (assuming - // the program is running under debugger, of course) + /* call this function to break into the debugger unconditionally (assuming */ + /* the program is running under debugger, of course) */ extern void WXDLLIMPEXP_BASE wxTrap(); - // helper function used to implement wxASSERT and wxASSERT_MSG - // - // note using "int" and not "bool" for cond to avoid VC++ warnings about - // implicit conversions when doing "wxAssert( pointer )" and also use of - // "!!cond" below to ensure that everything is converted to int + /* helper function used to implement wxASSERT and wxASSERT_MSG */ + /* */ + /* note using "int" and not "bool" for cond to avoid VC++ warnings about */ + /* implicit conversions when doing "wxAssert( pointer )" and also use of */ + /* "!!cond" below to ensure that everything is converted to int */ extern void WXDLLIMPEXP_BASE wxAssert(int cond, const wxChar *szFile, int nLine, const wxChar *szCond, const wxChar *szMsg = NULL) ; - // generic assert macro + /* generic assert macro */ #define wxASSERT(cond) wxAssert(!!(cond), __TFILE__, __LINE__, _T(#cond)) - // assert with additional message explaining it's cause + /* assert with additional message explaining it's cause */ #define wxASSERT_MSG(cond, msg) \ wxAssert(!!(cond), __TFILE__, __LINE__, _T(#cond), msg) - // an assert helper used to avoid warning when testing constant expressions, - // i.e. wxASSERT( sizeof(int) == 4 ) can generate a compiler warning about - // expression being always true, but not using - // wxASSERT( wxAssertIsEqual(sizeof(int), 4) ) - // - // NB: this is made obsolete by wxCOMPILE_TIME_ASSERT() and shouldn't be - // used any longer + /* an assert helper used to avoid warning when testing constant expressions, */ + /* i.e. wxASSERT( sizeof(int) == 4 ) can generate a compiler warning about */ + /* expression being always true, but not using */ + /* wxASSERT( wxAssertIsEqual(sizeof(int), 4) ) */ + /* */ + /* NB: this is made obsolete by wxCOMPILE_TIME_ASSERT() and shouldn't be */ + /* used any longer */ extern bool WXDLLIMPEXP_BASE wxAssertIsEqual(int x, int y); #else #define wxTrap() - // nothing to do in release modes (hopefully at this moment there are - // no more bugs ;-) + /* nothing to do in release modes (hopefully at this moment there are */ + /* no more bugs ;-) */ #define wxASSERT(cond) #define wxASSERT_MSG(x, m) -#endif //__WXDEBUG__ +#endif /* __WXDEBUG__ */ + +#ifdef __cplusplus + /* Use of wxFalse instead of FALSE suppresses compiler warnings about testing */ + /* constant expression */ + WXDLLIMPEXP_DATA_BASE(extern const bool) wxFalse; +#endif -// Use of wxFalse instead of FALSE suppresses compiler warnings about testing -// constant expression -WXDLLIMPEXP_DATA_BASE(extern const bool) wxFalse; #define wxAssertFailure wxFalse -// special form of assert: always triggers it (in debug mode) +/* special form of assert: always triggers it (in debug mode) */ #define wxFAIL wxASSERT(wxAssertFailure) -// FAIL with some message +/* FAIL with some message */ #define wxFAIL_MSG(msg) wxASSERT_MSG(wxAssertFailure, msg) -// NB: the following macros work also in release mode! +/* NB: the following macros work also in release mode! */ /* These macros must be used only in invalid situation: for example, an @@ -142,36 +147,36 @@ WXDLLIMPEXP_DATA_BASE(extern const bool) wxFalse; CHECK( p != NULL ) or CHECK( p != NULL, return LogError("p is NULL!!") ) */ -// check that expression is true, "return" if not (also FAILs in debug mode) +/* check that expression is true, "return" if not (also FAILs in debug mode) */ #define wxCHECK(x, rc) if (!(x)) {wxFAIL; return rc; } -// as wxCHECK but with a message explaining why we fail +/* as wxCHECK but with a message explaining why we fail */ #define wxCHECK_MSG(x, rc, msg) if (!(x)) {wxFAIL_MSG(msg); return rc; } -// check that expression is true, perform op if not +/* check that expression is true, perform op if not */ #define wxCHECK2(x, op) if (!(x)) {wxFAIL; op; } -// as wxCHECK2 but with a message explaining why we fail +/* as wxCHECK2 but with a message explaining why we fail */ #define wxCHECK2_MSG(x, op, msg) if (!(x)) {wxFAIL_MSG(msg); op; } -// special form of wxCHECK2: as wxCHECK, but for use in void functions -// -// NB: there is only one form (with msg parameter) and it's intentional: -// there is no other way to tell the caller what exactly went wrong -// from the void function (of course, the function shouldn't be void -// to begin with...) +/* special form of wxCHECK2: as wxCHECK, but for use in void functions */ +/* */ +/* NB: there is only one form (with msg parameter) and it's intentional: */ +/* there is no other way to tell the caller what exactly went wrong */ +/* from the void function (of course, the function shouldn't be void */ +/* to begin with...) */ #define wxCHECK_RET(x, msg) if (!(x)) {wxFAIL_MSG(msg); return; } -// ---------------------------------------------------------------------------- -// Compile time asserts -// -// Unlike the normal assert and related macros above which are checked during -// the program tun-time the macros below will result in a compilation error if -// the condition they check is false. This is usually used to check the -// expressions containing sizeof()s which cannot be tested with the -// preprocessor. If you can use the #if's, do use them as you can give a more -// detailed error message then. -// ---------------------------------------------------------------------------- +/* ---------------------------------------------------------------------------- */ +/* Compile time asserts */ +/* */ +/* Unlike the normal assert and related macros above which are checked during */ +/* the program tun-time the macros below will result in a compilation error if */ +/* the condition they check is false. This is usually used to check the */ +/* expressions containing sizeof()s which cannot be tested with the */ +/* preprocessor. If you can use the #if's, do use them as you can give a more */ +/* detailed error message then. */ +/* ---------------------------------------------------------------------------- */ /* How this works (you don't have to understand it to be able to use the @@ -181,19 +186,9 @@ WXDLLIMPEXP_DATA_BASE(extern const bool) wxFalse; particular, this is why we define a struct and not an object (which would result in a warning about unused variable) and a named struct (otherwise we'd get a warning about an unnamed struct not used to define an object!). - The _n__ part is to stop VC++ 7 being confused since it encloses __LINE++ in - parentheses. Unfortunately this does not work with other compilers, so - we will only enable it when we know the _precise_ symbols to test for. */ -#define wxMAKE_ASSERT_NAME_HELPER(line) wxAssert_ ## line -#define wxMAKE_ASSERT_NAME(line) wxMAKE_ASSERT_NAME_HELPER(line) -#if 0 -#define wxMAKE_UNIQUE_ASSERT_NAME wxMAKE_ASSERT_NAME(_n___ ## __LINE__) -#else -#define wxMAKE_UNIQUE_ASSERT_NAME wxMAKE_ASSERT_NAME(__LINE__) -#endif -#define wxMAKE_UNIQUE_ASSERT_NAME2(text) wxMAKE_ASSERT_NAME(text) +#define wxMAKE_UNIQUE_ASSERT_NAME wxMAKE_UNIQUE_NAME(wxAssert_) /* The second argument of this macro must be a valid C++ identifier and not a @@ -206,31 +201,40 @@ WXDLLIMPEXP_DATA_BASE(extern const bool) wxFalse; #define wxCOMPILE_TIME_ASSERT(expr, msg) \ struct wxMAKE_UNIQUE_ASSERT_NAME { unsigned int msg: expr; } +/* + When using VC++ 6 with "Edit and Continue" on, the compiler completely + mishandles __LINE__ and so wxCOMPILE_TIME_ASSERT() doesn't work, provide a + way to make "unique" assert names by specifying a unique prefix explicitly + */ +#define wxMAKE_UNIQUE_ASSERT_NAME2(text) wxCONCAT(wxAssert_, text) + #define wxCOMPILE_TIME_ASSERT2(expr, msg, text) \ struct wxMAKE_UNIQUE_ASSERT_NAME2(text) { unsigned int msg: expr; } -// helpers for wxCOMPILE_TIME_ASSERT below, for private use only +/* helpers for wxCOMPILE_TIME_ASSERT below, for private use only */ #define wxMAKE_BITSIZE_MSG(type, size) type ## SmallerThan ## size ## Bits -// a special case of compile time assert: check that the size of the given type -// is at least the given number of bits +/* a special case of compile time assert: check that the size of the given type */ +/* is at least the given number of bits */ #define wxASSERT_MIN_BITSIZE(type, size) \ wxCOMPILE_TIME_ASSERT(sizeof(type) * CHAR_BIT >= size, \ wxMAKE_BITSIZE_MSG(type, size)) -// ---------------------------------------------------------------------------- -// other miscellaneous debugger-related functions -// ---------------------------------------------------------------------------- - -// return true if we're running under debugger -// -// currently this only really works under Mac in CodeWarrior builds, it always -// returns false otherwise -#ifdef __WXMAC__ - extern bool WXDLLIMPEXP_BASE wxIsDebuggerRunning(); -#else // !Mac - inline bool wxIsDebuggerRunning() { return false; } -#endif // Mac/!Mac - -#endif // _WX_DEBUG_H_ +/* ---------------------------------------------------------------------------- */ +/* other miscellaneous debugger-related functions */ +/* ---------------------------------------------------------------------------- */ + +/* return true if we're running under debugger */ +/* */ +/* currently this only really works under Mac in CodeWarrior builds, it always */ +/* returns false otherwise */ +#ifdef __cplusplus + #ifdef __WXMAC__ + extern bool WXDLLIMPEXP_BASE wxIsDebuggerRunning(); + #else /* !Mac */ + inline bool wxIsDebuggerRunning() { return false; } + #endif /* Mac/!Mac */ +#endif /* __cplusplus */ + +#endif /* _WX_DEBUG_H_ */