Are you saying C++ does not support anonymous structs/unions? They're one of the most useful features of C and make for vastly more readable code.
On Tue, Feb 24, 2015 at 1:01 AM, Simon Kågström < [email protected]> wrote: > On 2015-02-23 21:17, [email protected] wrote: > > Some googling suggests that this is because the ODP API files do not > have a > > named namespace and as a result default to the namespace 'anonymous'. > However, > > anonymous is limited to a single translation unit. > > [...] > > > #ifdef __cplusplus > > extern "C" { > > namespace ODP { > > #endif > > > > ...ODP definitions > > > > #ifdef __cplusplus > > } > > } > > #endif > > I think the proper fix is something like this: > > diff --git a/include/odp/api/pool.h b/include/odp/api/pool.h > index 6fe52fa..5eb8448 100644 > --- a/include/odp/api/pool.h > +++ b/include/odp/api/pool.h > @@ -82,7 +82,7 @@ typedef struct odp_pool_param_t { > struct { > uint32_t num; /**< Number of timeouts in the > pool */ > } tmo; > - }; > + } u; > > int type; /**< Pool type */ > } odp_pool_param_t; > > > And then changing all the uses of them. > > // Simon > > _______________________________________________ > 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
