erase _wayland_tbm_check_dlog_enable function

the function use getenv. but getenv is not thread safe.
so erase this function to get rid of getenv.

Change-Id: I93ce296adf2b9f005c70f113d4159f74b4c691ec
Signed-off-by: Junkyeong Kim <jk0430.kim@samsung.com>

diff --git a/src/wayland-tbm-client.c b/src/wayland-tbm-client.c
index bd419f5..031e160 100644 (file)
--- a/src/wayland-tbm-client.c
+++ b/src/wayland-tbm-client.c
@@ -323,8 +323,6 @@ wayland_tbm_client_init(struct wl_display *display)
        struct wl_event_queue *wl_queue;
        struct wl_registry *wl_registry;
 
-       _wayland_tbm_check_dlog_enable();
-
        bTrace = 0;
 
        tbm_client = calloc(1, sizeof(struct wayland_tbm_client));

diff --git a/src/wayland-tbm-int.h b/src/wayland-tbm-int.h
index f759429..498c78c 100644 (file)
--- a/src/wayland-tbm-int.h
+++ b/src/wayland-tbm-int.h
@@ -43,7 +43,6 @@ extern "C" {
 #ifdef HAVE_DLOG
 #include <dlog.h>
 
-extern int bDlog;
 extern int bTrace;
 
 #ifdef LOG_TAG
@@ -51,42 +50,11 @@ extern int bTrace;
 #endif
 
 #define LOG_TAG "WL_TBM"
-
-#define WL_TBM_C_LOG(fmt, ...) {\
-       if (bDlog) {\
-               LOGI(fmt, ##__VA_ARGS__);\
-       } \
-       else {\
-               fprintf(stderr, "[WL_TBM_C(%d):%s] " fmt"\n", getpid(), __func__, ##__VA_ARGS__);\
-       } \
-}
-#define WL_TBM_S_LOG(fmt, ...) {\
-       if (bDlog) {\
-               LOGI(fmt, ##__VA_ARGS__);\
-       } \
-       else{\
-               fprintf(stderr, "[WL_TBM_S(%d):%s] " fmt"\n", getpid(), __func__, ##__VA_ARGS__);\
-       } \
-}
-#define WL_TBM_LOG(fmt, ...) {\
-       if (bDlog) {\
-               LOGI(fmt, ##__VA_ARGS__);\
-       } \
-       else {\
-               fprintf(stderr, "[WL_TBM(%d)] " fmt"\n", getpid(), ##__VA_ARGS__);\
-       } \
-}
-
+#define WL_TBM_C_LOG(fmt, ...)   LOGI(fmt, ##__VA_ARGS__);
+#define WL_TBM_S_LOG(fmt, ...)   LOGI(fmt, ##__VA_ARGS__);
+#define WL_TBM_LOG(fmt, ...)     LOGI(fmt, ##__VA_ARGS__);
 /* temp log */
-#define WL_TBM_LOG_E(fmt, ...) {\
-       if (bDlog) {\
-               LOGE(fmt, ##__VA_ARGS__);\
-       } \
-       else {\
-               fprintf(stderr, "[WL_TBM(%d)] " fmt"\n", getpid(), ##__VA_ARGS__);\
-       } \
-}
-
+#define WL_TBM_LOG_E(fmt, ...)   LOGE(fmt, ##__VA_ARGS__);
 #else
 #define WL_TBM_C_LOG(fmt, ...)   fprintf(stderr, "[WL_TBM_C(%d):%s] " fmt"\n", getpid(), __func__, ##__VA_ARGS__)
 #define WL_TBM_S_LOG(fmt, ...)   fprintf(stderr, "[WL_TBM_S(%d):%s] " fmt"\n", getpid(), __func__, ##__VA_ARGS__)
@@ -187,9 +155,6 @@ _wayland_tbm_util_get_appname_brief(char *brief);
 void
 _wayland_tbm_util_get_appname_from_pid(long pid, char *str);
 
-void
-_wayland_tbm_check_dlog_enable(void);
-
 int
 _waylend_tbm_util_target_parse(char * target_str, struct wayland_tbm_monitor_target *target);
 

diff --git a/src/wayland-tbm-server.c b/src/wayland-tbm-server.c
index db68ee1..c41f348 100644 (file)
--- a/src/wayland-tbm-server.c
+++ b/src/wayland-tbm-server.c
@@ -716,8 +716,6 @@ wayland_tbm_server_init(struct wl_display *display, const char *device_name,
 {
        struct wayland_tbm_server *tbm_srv;
 
-       _wayland_tbm_check_dlog_enable();
-
        bTrace = 0;
 
        tbm_srv = calloc(1, sizeof(struct wayland_tbm_server));

diff --git a/src/wayland-tbm-util.c b/src/wayland-tbm-util.c
index 981c595..3e9a8e0 100644 (file)
--- a/src/wayland-tbm-util.c
+++ b/src/wayland-tbm-util.c
@@ -331,23 +331,6 @@ _wayland_tbm_util_get_appname_from_pid(long pid, char *str)
        snprintf(str, sizeof(cmdline), "%s", cmdline);
 }
 
-void
-_wayland_tbm_check_dlog_enable(void)
-{
-#ifdef HAVE_DLOG
-       char *env;
-
-       env = getenv("WL_TBM_DLOG");
-       if (env) {
-               bDlog = atoi(env);
-               WL_TBM_LOG("WL_TBM_DLOG=%s", env);
-       } else {
-               bDlog = 1;
-       }
-#endif
-       return;
-}
-
 char *
 _wayland_tbm_dump_directory_make(void)
 {