X-Git-Url: https://codewiz.org/gitweb?a=blobdiff_plain;f=cfg%2Fdebug.h;h=979130888e017a5bc90de733191d89659dc1d994;hb=9ffcd0c4865a64d7fc713eb038dcba199d11fccb;hp=38029081aa5ae942f1f5c918223b1a34fae5d3e0;hpb=126666459b99272b966ba70e946a27e547ec4600;p=bertos.git diff --git a/cfg/debug.h b/cfg/debug.h index 38029081..97913088 100755 --- a/cfg/debug.h +++ b/cfg/debug.h @@ -17,6 +17,15 @@ /*#* *#* $Log$ + *#* Revision 1.10 2006/02/23 09:10:10 bernie + *#* Add even more code duplication until we properly refactor debug.h. + *#* + *#* Revision 1.9 2006/02/23 08:40:33 bernie + *#* TRACEMSG() support for compilers with no variadic macros. + *#* + *#* Revision 1.8 2006/02/23 08:33:04 bernie + *#* Fix for compilers without variadic macros support. + *#* *#* Revision 1.7 2006/02/20 02:01:56 bernie *#* Depend on cfg/os.h. *#* @@ -173,8 +182,22 @@ { fputs(str, stderr); } - /* G++ can't inline functions with variable arguments... */ - #define kprintf(fmt, ...) fprintf(stderr, fmt, ## __VA_ARGS__) + #if COMPILER_VARIADIC_MACROS + /* G++ can't inline functions with variable arguments... */ + #define kprintf(fmt, ...) fprintf(stderr, fmt, ## __VA_ARGS__) + #else + INLINE int kprintf(const char *fmt, ...) + { + va_list ap; + int result; + + va_start(ap, fmt); + result = kvprintf(fmt, ap); + va_end(ap); + + return result; + } + #endif #define kvprintf(fmt, ap) vfprintf(stderr, fmt, ap) void kdump(const void *buf, size_t len); /* UNIMPLEMENTED */ @@ -184,8 +207,8 @@ #endif /* ASSERT */ #define ASSERT2(x, help) ASSERT(help && x) - /*! - * Check that the given pointer is not NULL or pointing to raw memory. + /** + * Check that the given pointer is either NULL or pointing to valid memory. * * The assumption here is that valid pointers never point to low * memory regions. This helps catching pointers taken from @@ -193,11 +216,27 @@ */ #define ASSERT_VALID_PTR(p) ASSERT((unsigned long)(p) > 0x200) + /** + * Check that the given pointer is not pointing to invalid memory. + * + * \see ASSERT_VALID_PTR() + */ #define ASSERT_VALID_PTR_OR_NULL(p) ASSERT((((p) == NULL) || ((unsigned long)(p) >= 0x200))) #if !CONFIG_KDEBUG_DISABLE_TRACE - #define TRACE kporintf("%s()\n", __func__) - #define TRACEMSG(msg,...) kprintf("%s(): " msg, __func__, ## __VA_ARGS__) + #define TRACE kprintf("%s()\n", __func__) + #if COMPILER_VARIADIC_MACROS + #define TRACEMSG(msg,...) kprintf("%s(): " msg, __func__, ## __VA_ARGS__) + #else + INLINE void TRACEMSG(const char *fmt, ...) + { + va_list va; + va_start(va, fmt); + kprintf("%s(): ", __func__); + kvprintf(fmt, va); + va_end(va); + } + #endif #else #define TRACE do {} while(0) #define TRACEMSG(...) do {} while(0) @@ -348,7 +387,14 @@ #define ASSERT_VALID_PTR_OR_NULL(p) ((void)0) #define ASSERT_VALID_OBJ(_t, _o) ((void)0) #define TRACE do {} while (0) - #define TRACEMSG(x,...) do {} while (0) + #if COMPILER_VARIADIC_MACROS + #define TRACEMSG(x, ...) do {} while (0) + #else + INLINE void TRACEMSG(UNUSED_ARG(const char *, msg), ...) + { + /* NOP */ + } + #endif #define DECLARE_WALL(name, size) /* nothing */ #define FWD_DECLARE_WALL(name, size) /* nothing */ @@ -368,12 +414,12 @@ INLINE void kputs(UNUSED_ARG(const char *, str)) { /* nop */ } INLINE void kdump(UNUSED_ARG(const void *, buf), UNUSED_ARG(size_t, len)) { /* nop */ } - #ifdef __cplusplus + #if defined(__cplusplus) && COMPILER_VARIADIC_MACROS /* G++ can't inline functions with variable arguments... */ #define kprintf(fmt,...) do { (void)(fmt); } while(0) #else /* ...but GCC can. */ - INLINE void kprintf(UNUSED_ARG(const char *, fmt), ...) { /* nop */ } + INLINE void kprintf(UNUSED_ARG(const char *, fmt), ...) { /* nop */ } #endif #endif /* _DEBUG */