diff --git a/legacy/ecore/src/lib/ecore/ecore.c b/legacy/ecore/src/lib/ecore/ecore.c index 3430e974ac..7e0f9738d4 100644 --- a/legacy/ecore/src/lib/ecore/ecore.c +++ b/legacy/ecore/src/lib/ecore/ecore.c @@ -217,7 +217,6 @@ shutdown_evil: EAPI int ecore_shutdown(void) { - Ecore_Pipe *p; /* * take a lock here because _ecore_event_shutdown() does callbacks */ @@ -225,15 +224,21 @@ ecore_shutdown(void) if (--_ecore_init_count != 0) goto unlock; - /* this looks horrible - a hack for now, but something to note. as - * we delete the _thread_call pipe a thread COULD be doing - * ecore_pipe_write() or what not to it at the same time - we - * must ensure all possible users of this _thread_call are finished - * and exited before we delete it here */ + /* this looks horrible - a hack for now, but something to note. as + * we delete the _thread_call pipe a thread COULD be doing + * ecore_pipe_write() or what not to it at the same time - we + * must ensure all possible users of this _thread_call are finished + * and exited before we delete it here */ + /* + * ok - this causes other valgrind complaints regarding glib aquiring + * locks internally. so fix bug a or bug b. let's leave the original + * bug in then and leave this as a note for now + Ecore_Pipe *p; p = _thread_call; _thread_call = NULL; ecore_pipe_wait(p, 1, 0.1); ecore_pipe_del(p); + */ eina_lock_free(&_thread_safety); eina_condition_free(&_thread_cond); eina_lock_free(&_thread_mutex);