On 17 February 2015 at 08:10, Simon Kågström <[email protected]> wrote: > On 2015-02-16 18:19, [email protected] wrote: >> Mike Holmes <mailto:[email protected]> changed bug 300 >> <https://bugs.linaro.org/show_bug.cgi?id=300> >> What Removed Added >> Resolution --- WONTFIX >> Status CONFIRMED RESOLVED >> >> *Comment # 5 <https://bugs.linaro.org/show_bug.cgi?id=300#c5> on bug 300 >> <https://bugs.linaro.org/show_bug.cgi?id=300> from Mike Holmes >> <mailto:[email protected]> * >> >> pedantic has not been adopted for the ODP build flags, this will not be fixed >> at this time > > This is another thing which doesn't work when including ODP headers from > C++ code. When you write C++ I assume you mean C++ in general. But just like we require C99 for C code (I wanted C11 because of the atomics support), perhaps there is some suitable language level for C++ as well? I assume C++ has a similar feature growth as C.
-- Ola > A patch to fix it can be found below, but might not be what > you want to do. > > // Simon > > commit 6997dddc8658eee3de7a18de6a510365d2783253 > Author: Simon Kagstrom <[email protected]> > Date: Tue Feb 17 08:08:04 2015 +0100 > > debug: Use static_assert(cond,msg) for C++ code > > diff --git a/include/odp/api/debug.h b/include/odp/api/debug.h > index 5949680..cb9de64 100644 > --- a/include/odp/api/debug.h > +++ b/include/odp/api/debug.h > @@ -14,8 +14,8 @@ > > > #ifdef __cplusplus > -extern "C" { > -#endif > +#define _ODP_STATIC_ASSERT(cond, msg) static_assert(cond, msg) > +#else > > /** @addtogroup odp_ver_abt_log_dbg > * Macros that allows different messages. > @@ -50,9 +50,6 @@ extern "C" { > /** > * @} > */ > - > -#ifdef __cplusplus > -} > #endif > > #endif > > _______________________________________________ > lng-odp mailing list > [email protected] > http://lists.linaro.org/mailman/listinfo/lng-odp _______________________________________________ lng-odp mailing list [email protected] http://lists.linaro.org/mailman/listinfo/lng-odp
