Sorry, proper file patched on this one. It's eina_log.c
dh
Christopher Michael wrote:
Attached is a patch for Eina_Amalgamation. This patch fixes a compiler
warning that I recently noticed: "Warning return with a value in
function returning void". Basically, I created a second macro
"CHECK_MAIN_NO_RETURN" which can be used in functions that return void.
Rather than the normal "CHECK_MAIN" which returns VA_ARGS.
dh
Index: eina_log.c
===================================================================
--- eina_log.c (revision 43427)
+++ eina_log.c (working copy)
@@ -372,6 +372,15 @@
return __VA_ARGS__; \
} \
} while (0)
+#define CHECK_MAIN_NO_RETURN(...) \
+ do { \
+ if (!IS_MAIN(pthread_self())) { \
+ fprintf(stderr, \
+ "ERR: not main thread! current=%lu, main=%lu\n", \
+ pthread_self(), _main_thread); \
+ return; \
+ } \
+ } while (0)
#ifdef EINA_PTHREAD_SPIN
static pthread_spinlock_t _log_lock;
@@ -409,6 +418,7 @@
#define IS_MAIN(t) (1)
#define IS_OTHER(t) (0)
#define CHECK_MAIN(...) do {} while (0)
+#define CHECK_MAIN_NO_RETURN(...) do {} while (0)
#define INIT() do {} while (0)
#define SHUTDOWN() do {} while (0)
#endif
@@ -1085,7 +1095,7 @@
void
eina_log_threads_shutdown(void)
{
- CHECK_MAIN(0);
+ CHECK_MAIN_NO_RETURN(0);
SHUTDOWN();
_threads_enabled = EINA_FALSE;
}
------------------------------------------------------------------------------
Come build with us! The BlackBerry(R) Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay
ahead of the curve. Join us from November 9 - 12, 2009. Register now!
http://p.sf.net/sfu/devconference
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
