On Wed, Mar 18, 2015 at 06:17:39PM -0400, Mike Holmes wrote: > ODP implementations should not call assert directly. > > Signed-off-by: Mike Holmes <mike.hol...@linaro.org> > --- > platform/linux-generic/odp_timer.c | 20 +++++++++++--------- > 1 file changed, 11 insertions(+), 9 deletions(-) > > diff --git a/platform/linux-generic/odp_timer.c > b/platform/linux-generic/odp_timer.c > index b7cb04f..39d7064 100644 > --- a/platform/linux-generic/odp_timer.c > +++ b/platform/linux-generic/odp_timer.c > @@ -23,9 +23,7 @@ > > /* For snprint, POSIX timers and sigevent */ > #define _POSIX_C_SOURCE 200112L > -#include <assert.h> > #include <errno.h> > -#include <string.h> > #include <stdlib.h> > #include <time.h> > #include <signal.h> > @@ -123,8 +121,9 @@ static void timer_init(odp_timer *tim, > /* Teardown when timer is freed */ > static void timer_fini(odp_timer *tim, tick_buf_t *tb) > { > - assert(tb->exp_tck.v == TMO_UNUSED); > - assert(tb->tmo_buf == ODP_BUFFER_INVALID); > + ODP_ASSERT(tb->exp_tck.v == TMO_UNUSED, "tb->exp_tck.v == TMO_UNUSED"); > + ODP_ASSERT(tb->tmo_buf == ODP_BUFFER_INVALID, > + "tb->tmo_buf == ODP_BUFFER_INVALID");
This isn't too pleasant, and the comments could become stale. How about we just modify ODP_ASSERT() to do this? (there are no users of it yet) Something like this perhaps; #define ODP_ASSERT(cond) ODP_ASSERT_STR(cond, " assert failed: " #cond) #define ODP_ASSERT_STR(cond, msg) \ do { if ((ODP_DEBUG == 1) && (!(cond))) { \ ODP_ERR("%s\n", msg); \ odp_global_data.abort_fn(); } \ } while (0) -- Stuart. _______________________________________________ lng-odp mailing list lng-odp@lists.linaro.org http://lists.linaro.org/mailman/listinfo/lng-odp