erase _wayland_tbm_check_dlog_enable function 39/231539/1
authorJunkyeong Kim <jk0430.kim@samsung.com>
Wed, 22 Apr 2020 08:29:23 +0000 (17:29 +0900)
committerJunkyeong Kim <jk0430.kim@samsung.com>
Wed, 22 Apr 2020 08:29:26 +0000 (17:29 +0900)
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>
src/wayland-tbm-client.c
src/wayland-tbm-int.h
src/wayland-tbm-server.c
src/wayland-tbm-util.c

index bd419f592511292500218c4e4ba6759c1f9ee12a..031e160a51b3fba7aa575d769a3d7cdd938b6c06 100644 (file)
@@ -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));
index f759429f7f296203716cfedaf27010fd252490fa..498c78cab61743a2dc37e63f7b65369ef8869ff2 100644 (file)
@@ -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);
 
index db68ee1c8def68231258a4ad8e387b94efd284c7..c41f3484091889652cc02ef9b683a96982c9737a 100644 (file)
@@ -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));
index 981c595238525d5bdbb1db1a7b6c8597d85a34da..3e9a8e013fa6a4cc5f3da7ef44680baca17d01ab 100644 (file)
@@ -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)
 {