From a0f37fad24445c2669c249fd96fbbec1a2c8d365 Mon Sep 17 00:00:00 2001 From: Youngbok Shin Date: Fri, 18 Mar 2016 11:54:00 -0700 Subject: [PATCH] modules: remove error message from dlsym() when a module is loaded Summary: dlsym() could print error message when it tried to load a nonexistent symbol. Whenever eina_module_load is called, it checks __eina_module_init symbol. Even if there is no symbol for init, module loading could be done well. But, it will print an error message. So, we need to use EINA_MODULE_INIT, EINA_MODULE_SHUTDOWN in every modules for removing error messages. Test Plan: N/A Reviewers: woohyun, raster, Hermet, seoz, cedric Reviewed By: cedric Differential Revision: https://phab.enlightenment.org/D3805 Signed-off-by: Cedric Bail --- legacy/elementary/src/modules/access_output/mod.c | 14 ++++++++++++++ .../datetime_input_ctxpopup.c | 14 ++++++++++++++ legacy/elementary/src/modules/test_entry/mod.c | 14 ++++++++++++++ 3 files changed, 42 insertions(+) diff --git a/legacy/elementary/src/modules/access_output/mod.c b/legacy/elementary/src/modules/access_output/mod.c index 3f15f32923..0dff159885 100644 --- a/legacy/elementary/src/modules/access_output/mod.c +++ b/legacy/elementary/src/modules/access_output/mod.c @@ -121,3 +121,17 @@ out_done_callback_set(void (*func) (void *data), const void *data) cb_func = func; cb_data = (void *)data; } + +static Eina_Bool +_module_init(void) +{ + return EINA_TRUE; +} + +static void +_module_shutdown(void) +{ +} + +EINA_MODULE_INIT(_module_init); +EINA_MODULE_SHUTDOWN(_module_shutdown); diff --git a/legacy/elementary/src/modules/datetime_input_ctxpopup/datetime_input_ctxpopup.c b/legacy/elementary/src/modules/datetime_input_ctxpopup/datetime_input_ctxpopup.c index b8dc56cff0..198f095a28 100644 --- a/legacy/elementary/src/modules/datetime_input_ctxpopup/datetime_input_ctxpopup.c +++ b/legacy/elementary/src/modules/datetime_input_ctxpopup/datetime_input_ctxpopup.c @@ -382,3 +382,17 @@ elm_modapi_shutdown(void *m EINA_UNUSED) { return 1; // succeed always } + +static Eina_Bool +_module_init(void) +{ + return EINA_TRUE; +} + +static void +_module_shutdown(void) +{ +} + +EINA_MODULE_INIT(_module_init); +EINA_MODULE_SHUTDOWN(_module_shutdown); diff --git a/legacy/elementary/src/modules/test_entry/mod.c b/legacy/elementary/src/modules/test_entry/mod.c index 025d9b4355..d2d5171af2 100644 --- a/legacy/elementary/src/modules/test_entry/mod.c +++ b/legacy/elementary/src/modules/test_entry/mod.c @@ -35,3 +35,17 @@ obj_longpress(Evas_Object *obj) { printf("longpress: %p\n", obj); } + +static Eina_Bool +_module_init(void) +{ + return EINA_TRUE; +} + +static void +_module_shutdown(void) +{ +} + +EINA_MODULE_INIT(_module_init); +EINA_MODULE_SHUTDOWN(_module_shutdown);