erase tdm log macros semicolon

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

diff --git a/include/tdm_log.h b/include/tdm_log.h
index 02943c0..c55acf6 100644 (file)
--- a/include/tdm_log.h
+++ b/include/tdm_log.h
@@ -79,14 +79,15 @@ extern int tdm_debug;
 #define LOG_TAG "TDM"
 
 #define TDM_DBG(fmt, args...) \
-       if (tdm_debug) \
        do { \
-               struct timespec ts;     \
-               clock_gettime(CLOCK_MONOTONIC, &ts);    \
-               LOGD("[%d.%06d] "fmt"\n", (int)ts.tv_sec, (int)ts.tv_nsec / 1000, ##args);      \
-               printf("[TDM_DBG][%d.%06d][%d][%s %d] "fmt"\n", (int)ts.tv_sec, \
-                       (int)ts.tv_nsec / 1000, (int)syscall(SYS_gettid), __func__, __LINE__, ##args); \
-       } while (0);
+               if (tdm_debug) { \
+                       struct timespec ts;     \
+                       clock_gettime(CLOCK_MONOTONIC, &ts);    \
+                       LOGD("[%d.%06d] "fmt"\n", (int)ts.tv_sec, (int)ts.tv_nsec / 1000, ##args);      \
+                       printf("[TDM_DBG][%d.%06d][%d][%s %d] "fmt"\n", (int)ts.tv_sec, \
+                               (int)ts.tv_nsec / 1000, (int)syscall(SYS_gettid), __func__, __LINE__, ##args); \
+               } \
+       } while (0)
 
 #define TDM_INFO(fmt, args...) \
        do { \
@@ -95,7 +96,7 @@ extern int tdm_debug;
                LOGI("[%d.%06d] "fmt"\n", (int)ts.tv_sec, (int)ts.tv_nsec / 1000, ##args);      \
                printf("[TDM_INF][%d.%06d][%d][%s %d] "fmt"\n", (int)ts.tv_sec, \
                        (int)ts.tv_nsec / 1000, (int)syscall(SYS_gettid), __func__, __LINE__, ##args); \
-       } while (0);
+       } while (0)
 
 #define TDM_WRN(fmt, args...) \
        do { \
@@ -104,7 +105,7 @@ extern int tdm_debug;
                LOGI("[%d.%06d] "fmt"\n", (int)ts.tv_sec, (int)ts.tv_nsec / 1000, ##args);      \
                printf("[TDM_WRN][%d.%06d][%d][%s %d] "fmt"\n", (int)ts.tv_sec, \
                        (int)ts.tv_nsec / 1000, (int)syscall(SYS_gettid), __func__, __LINE__, ##args); \
-       } while (0);
+       } while (0)
 
 #define TDM_ERR(fmt, args...) \
        do { \
@@ -113,7 +114,7 @@ extern int tdm_debug;
                LOGE("[%d.%06d] "fmt"\n", (int)ts.tv_sec, (int)ts.tv_nsec / 1000, ##args);      \
                printf("[TDM_ERR][%d.%06d][%d][%s %d] "fmt"\n", (int)ts.tv_sec, \
                        (int)ts.tv_nsec / 1000, (int)syscall(SYS_gettid), __func__, __LINE__, ##args); \
-       } while (0);
+       } while (0)
 
 #else /* TDM_CONFIG_DLOG */
 
@@ -125,13 +126,14 @@ extern int tdm_debug;
 #define COLOR_RESET "\x1b[0m"
 
 #define TDM_DBG(fmt, args...) \
-       if (tdm_debug) \
        do { \
-               struct timespec ts;     \
-               clock_gettime(CLOCK_MONOTONIC, &ts);    \
-               printf("[TDM_DBG][%d.%06d][%d][%s %d] "fmt"\n", (int)ts.tv_sec, \
-                       (int)ts.tv_nsec / 1000, (int)syscall(SYS_gettid), __func__, __LINE__, ##args); \
-       } while (0);
+               if (tdm_debug) { \
+                       struct timespec ts;     \
+                       clock_gettime(CLOCK_MONOTONIC, &ts);    \
+                       printf("[TDM_DBG][%d.%06d][%d][%s %d] "fmt"\n", (int)ts.tv_sec, \
+                               (int)ts.tv_nsec / 1000, (int)syscall(SYS_gettid), __func__, __LINE__, ##args); \
+               } \
+       } while (0)
 
 #define TDM_INFO(fmt, args...) \
        do { \
@@ -139,7 +141,7 @@ extern int tdm_debug;
                clock_gettime(CLOCK_MONOTONIC, &ts);    \
                printf(COLOR_GREEN"[TDM_INF]"COLOR_RESET"[%d.%06d][%d][%s %d] "fmt"\n", (int)ts.tv_sec, \
                        (int)ts.tv_nsec / 1000, (int)syscall(SYS_gettid), __func__, __LINE__, ##args); \
-       } while (0);
+       } while (0)
 
 #define TDM_WRN(fmt, args...) \
        do { \
@@ -148,7 +150,7 @@ extern int tdm_debug;
                printf(COLOR_YELLOW"[TDM_WRN]"COLOR_RESET"[%d.%06d][%d][%s %d] "fmt"\n", (int)ts.tv_sec,        \
                        (int)ts.tv_nsec / 1000, (int)syscall(SYS_gettid), __func__, __LINE__, ##args); \
                TDM_ASSERT(0); \
-       } while (0);
+       } while (0)
 
 #define TDM_ERR(fmt, args...) \
        do { \
@@ -157,7 +159,7 @@ extern int tdm_debug;
                printf(COLOR_RED"[TDM_ERR]"COLOR_RESET"[%d.%06d][%d][%s %d] "fmt"\n", (int)ts.tv_sec,   \
                        (int)ts.tv_nsec / 1000, (int)syscall(SYS_gettid), __func__, __LINE__, ##args); \
                TDM_ASSERT(0); \
-       } while (0);
+       } while (0)
 
 #endif /* TDM_CONFIG_DLOG */
 

diff --git a/src/tdm_display.c b/src/tdm_display.c
index 969b227..2fdc486 100644 (file)
--- a/src/tdm_display.c
+++ b/src/tdm_display.c
@@ -154,11 +154,10 @@ _tdm_display_lock(tdm_display *dpy, const char *func)
 
        ret = pthread_mutex_trylock(&private_display->lock);
        if (ret < 0) {
-               if (ret == EBUSY) {
+               if (ret == EBUSY)
                        TDM_ERR("mutex lock busy: %s", func);
-               } else {
+               else
                        TDM_ERR("mutex lock failed: %s(%m)", func);
-               }
                return TDM_ERROR_OPERATION_FAILED;
        }
 
@@ -1578,7 +1577,7 @@ tdm_layer_get_displaying_buffer(tdm_layer *layer, tdm_error *error)
                if (error)
                        *error = TDM_ERROR_OPERATION_FAILED;
                _pthread_mutex_unlock(&private_display->lock);
-               TDM_ERR("layer(%p) showing_buffer is null", private_layer);
+               TDM_DBG("layer(%p) showing_buffer is null", private_layer);
                return NULL;
        }
        _pthread_mutex_unlock(&private_display->lock);