From: agiersch Date: Thu, 24 Mar 2011 16:49:35 +0000 (+0000) Subject: Define xbt_assert as a variadic macro. X-Git-Tag: v3.6_beta2~112 X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/commitdiff_plain/1fda4cc79d808bb4ca659d056c5e67be99ce47ca?ds=sidebyside Define xbt_assert as a variadic macro. git-svn-id: svn+ssh://scm.gforge.inria.fr/svn/simgrid/simgrid/trunk@9857 48e7efb5-ca39-0410-a469-dd3cf9ba447f --- diff --git a/ChangeLog b/ChangeLog index 6f98f69275..71d3a60fba 100644 --- a/ChangeLog +++ b/ChangeLog @@ -56,6 +56,7 @@ SimGrid (3.6) unstable; urgency=low http://lists.gforge.inria.fr/pipermail/simgrid-user/2010-December/002206.html * Add new macros with variable number of arguments. - in xbt/log.h: XBT_DEBUG, XBT_VERB, XBT_INFO, etc. + - in xbt/asserts.h: xbt_assert - in xbt/cunit.h: xbt_test_{add,fail,assert,log} - in xbt/ex.h: THROWF and RETHROWF. Define XBT_USE_DEPRECATED if you want to use the old numbered macros like diff --git a/doc/FAQ.doc b/doc/FAQ.doc index 129f926c3d..a6ec582641 100644 --- a/doc/FAQ.doc +++ b/doc/FAQ.doc @@ -1342,7 +1342,7 @@ workstation_ptask_L07.c: /* Parse the file */ surf_parse_open(file); - xbt_assert1((!surf_parse()), "Parse error in %s", file); + xbt_assert(!surf_parse(), "Parse error in %s", file); surf_parse_close(); \endverbatim diff --git a/include/xbt/asserts.h b/include/xbt/asserts.h index f3157f5349..386c7c82dd 100644 --- a/include/xbt/asserts.h +++ b/include/xbt/asserts.h @@ -21,7 +21,7 @@ SG_BEGIN_DECL() * * You can pass them a format message and arguments, just as if it where a * printf. - * It is converted to a CRITICALn logging request. + * It is converted to a XBT_CRITICAL logging request. * Be careful: the boolean expression that you want to test should not have * side effects, because assertions are disabled at compile time if NDEBUG * is set. @@ -29,32 +29,28 @@ SG_BEGIN_DECL() * @{ */ #ifdef NDEBUG -#define xbt_assert(cond) -#define xbt_assert0(cond,msg) -#define xbt_assert1(cond,msg,a) -#define xbt_assert2(cond,msg,a,b) -#define xbt_assert3(cond,msg,a,b,c) -#define xbt_assert4(cond,msg,a,b,c,d) -#define xbt_assert5(cond,msg,a,b,c,d,e) -#define xbt_assert6(cond,msg,a,b,c,d,e,f) +#define xbt_assert(...) ((void)0) #else /** @brief The condition which failed will be displayed. @hideinitializer */ -#define xbt_assert(cond) do { if (!(cond)) THROWF(0,0,"Assertion %s failed", #cond); } while (0) - /** @hideinitializer */ -#define xbt_assert0(cond,msg) do { if (!(cond)) THROWF(0,0,msg); } while (0) - /** @hideinitializer */ -#define xbt_assert1(cond,msg,a) do { if (!(cond)) THROWF(0,0,msg,a); } while (0) - /** @hideinitializer */ -#define xbt_assert2(cond,msg,a,b) do { if (!(cond)) THROWF(0,0,msg,a,b); } while (0) - /** @hideinitializer */ -#define xbt_assert3(cond,msg,a,b,c) do { if (!(cond)) THROWF(0,0,msg,a,b,c); } while (0) - /** @hideinitializer */ -#define xbt_assert4(cond,msg,a,b,c,d) do { if (!(cond)) THROWF(0,0,msg,a,b,c,d); } while (0) - /** @hideinitializer */ -#define xbt_assert5(cond,msg,a,b,c,d,e) do { if (!(cond)) THROWF(0,0,msg,a,b,c,d,e); } while (0) - /** @hideinitializer */ -#define xbt_assert6(cond,msg,a,b,c,d,e,f) do { if (!(cond)) THROWF(0,0,msg,a,b,c,d,e,f); } while (0) +#define xbt_assert(...) \ + _XBT_IF_ONE_ARG(_xbt_assert_ARG1, _xbt_assert_ARGN, __VA_ARGS__)(__VA_ARGS__) +#define _xbt_assert_ARG1(cond) \ + _xbt_assert_ARGN(cond, "Assertion %s failed", #cond) +#define _xbt_assert_ARGN(cond, ...) \ + do { if (!(cond)) THROWF(0, 0, __VA_ARGS__); } while (0) +#endif + +#if 1 || defined(XBT_USE_DEPRECATED) + +#define xbt_assert0(...) xbt_assert(__VA_ARGS__) +#define xbt_assert1(...) xbt_assert(__VA_ARGS__) +#define xbt_assert2(...) xbt_assert(__VA_ARGS__) +#define xbt_assert3(...) xbt_assert(__VA_ARGS__) +#define xbt_assert4(...) xbt_assert(__VA_ARGS__) +#define xbt_assert5(...) xbt_assert(__VA_ARGS__) +#define xbt_assert6(...) xbt_assert(__VA_ARGS__) + #endif /** @} */ SG_END_DECL()