[waffle] [PATCH 10/10] android: add missing pragma once guard

2015-03-24 Thread Emil Velikov
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/waffle/android/droid_surfaceflingerlink.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/waffle/android/droid_surfaceflingerlink.h b/src/waffle/android/droid_surfaceflingerlink.h index 53aeb4f..67c1dc2 100644 ---

Re: [waffle] [PATCH 02/10] nacl: move dlfcn.h inclusion to where it's needed

2015-03-24 Thread Tapani
Reviewed-by: Tapani Palli tapani.pa...@intel.com On 03/24/2015 05:56 PM, Emil Velikov wrote: Allows us to remove the extern C { #include system_header.h } pattern which can cause subtle issues. Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/waffle/nacl/nacl_container.cpp | 2 ++

[waffle] [PATCH v2 2/3] wcore: rework non-compatible mutex initialization

2015-03-24 Thread Emil Velikov
C11 does not specify a static initializer, based on the idea that the a mutex will be platform and/or implementation dependent. As such the alternative solution is to initialize the mutex with call_once/mtx_init. This will allow us to remove the transition hack, and in due time move to the

[waffle] [PATCH 00/10] nacl dl rework and header cleanups et all

2015-03-24 Thread Emil Velikov
Hi all, Was pondering on the trivial nacl_dl* nitpick I was talking about earlier and I've noticed something more interesting along the way. Namely: in some places in waffle we include headers explicitly wrapped for C linkage (extern C { #include foo.h }). As mentioned by Jose over at

[waffle] [PATCH 04/10] nacl: rework nacl_dl functions

2015-03-24 Thread Emil Velikov
This is a direct copy from cgl with the following tweaks - s/cgl/nacl/ - s/cgl_dl_gl_path/NACL_GLES2_LIBRARY/ - Add move the nacl_prefix function from nacl_platform.c Doing this allows us to have more consistent codebase and additional error messages for the user. As a bonus it squashes a bug

[waffle] [PATCH 02/10] nacl: move dlfcn.h inclusion to where it's needed

2015-03-24 Thread Emil Velikov
Allows us to remove the extern C { #include system_header.h } pattern which can cause subtle issues. Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- src/waffle/nacl/nacl_container.cpp | 2 ++ src/waffle/nacl/nacl_container.h | 2 -- 2 files changed, 2 insertions(+), 2 deletions(-)