This patch removes nearly all the warnings, I observed one straggler though

In file included from ../../platform/linux-generic/include/odp/init.h:28:
../../include/odp/api/init.h:121:9: warning: empty struct has size 0 in C,
size 1 in C++ [-Wextern-c-compat]
typedef struct odp_platform_init_t {

my procedure:-

./bootstrap
./configure --enable-test-perf --enable-test-vald --enable-test-cpp
make clean
make CXX=clang++


On 12 March 2015 at 17:36, Bill Fischofer <[email protected]> wrote:

> C++ doesn't like null structs so add a dummy variable to make it happy.
> Note that we only use these types as pseudo-pointers for strong typing
> so the contents of what they apparently point to is irrelevant since
> we never reference it.
>
> This addresses Bug https://bugs.linaro.org/show_bug.cgi?id=1267
>
> Signed-off-by: Bill Fischofer <[email protected]>
> ---
>  platform/linux-generic/include/odp/plat/strong_types.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/platform/linux-generic/include/odp/plat/strong_types.h
> b/platform/linux-generic/include/odp/plat/strong_types.h
> index 7fe73d8..72db7c4 100644
> --- a/platform/linux-generic/include/odp/plat/strong_types.h
> +++ b/platform/linux-generic/include/odp/plat/strong_types.h
> @@ -16,7 +16,7 @@
>  #define STRONG_TYPES_H_
>
>  /** Use strong typing for ODP types */
> -#define odp_handle_t struct {} *
> +#define odp_handle_t struct { uint8_t unused_dummy_var; } *
>
>  /** Internal macro to get value of an ODP handle */
>  #define _odp_typeval(handle) ((uint32_t)(uintptr_t)(handle))
> --
> 2.1.0
>
>


-- 
Mike Holmes
Technical Manager - Linaro Networking Group
Linaro.org <http://www.linaro.org/> *│ *Open source software for ARM SoCs
_______________________________________________
lng-odp mailing list
[email protected]
http://lists.linaro.org/mailman/listinfo/lng-odp

Reply via email to