fix some syntax errors

initialize variables before using
erase unreacherble codes

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

diff --git a/include/tdm_list.h b/include/tdm_list.h
index 6255cd01d6991189e55f7b97a74b03167ac798f0..2fd1db0688522f4c94b9a21f4e06a2a7f5b28f9d 100644 (file)
--- a/include/tdm_list.h
+++ b/include/tdm_list.h
@@ -150,7 +150,7 @@ static inline int list_length(struct list_head *item)
 
 #define LIST_FIND_ITEM(item, head, type, member, found) \
        do {    \
-               type *pos;      \
+               type *pos = NULL;       \
                found = NULL;   \
                LIST_FOR_EACH_ENTRY(pos, head, member)  \
                        if (pos == item) { found = item; break; }       \

diff --git a/src/tdm.c b/src/tdm.c
index 996fb9acb38a45d9a17e80ab5a7cc58d6a10e5db..be9d26b939c0b59443c3372f7ea52fd81d36452a 100644 (file)
--- a/src/tdm.c
+++ b/src/tdm.c
@@ -610,7 +610,7 @@ _tdm_display_update_internal(tdm_private_display *private_display,
 {
        tdm_output **outputs = NULL;
        int output_count = 0, i;
-       tdm_error ret;
+       tdm_error ret = TDM_ERROR_NONE;
 
        LIST_INITHEAD(&private_display->output_list);
        LIST_INITHEAD(&private_display->pp_list);

diff --git a/src/tdm_display.c b/src/tdm_display.c
index 11e09bc460708bbd368a428381a683085707e1ad..e691501a54906a3b44922a7464834ecb35a7d9b5 100644 (file)
--- a/src/tdm_display.c
+++ b/src/tdm_display.c
@@ -1150,9 +1150,7 @@ tdm_output_set_dpms(tdm_output *output, tdm_output_dpms dpms_value)
        tdm_func_output *func_output;
        OUTPUT_FUNC_ENTRY();
 
-       if (dpms_value < TDM_OUTPUT_DPMS_ON)
-               dpms_value = TDM_OUTPUT_DPMS_ON;
-       else if (dpms_value > TDM_OUTPUT_DPMS_OFF)
+       if (dpms_value > TDM_OUTPUT_DPMS_OFF)
                dpms_value = TDM_OUTPUT_DPMS_OFF;
 
        _pthread_mutex_lock(&private_display->lock);
@@ -1238,7 +1236,7 @@ tdm_output_call_change_handler_internal(tdm_private_output *private_output,
                                                                                tdm_value value)
 {
        tdm_private_display *private_display;
-       tdm_private_change_handler *change_handler;
+       tdm_private_change_handler *change_handler = NULL;
 
        TDM_RETURN_IF_FAIL(TDM_MUTEX_IS_LOCKED());
        TDM_RETURN_IF_FAIL(private_output);

diff --git a/src/tdm_helper.c b/src/tdm_helper.c
index 1363979328356dbd3ae6a245728cb4dedc8ace33..4172fe891eb450fd928f8845a71ca53ab7b8cc3f 100644 (file)
--- a/src/tdm_helper.c
+++ b/src/tdm_helper.c
@@ -408,8 +408,6 @@ _tdm_helper_buffer_convert(tbm_surface_h srcbuf, tbm_surface_h dstbuf,
 cant_convert:
        if (src_img)
                pixman_image_unref(src_img);
-       if (dst_img)
-               pixman_image_unref(dst_img);
 
        tbm_surface_unmap(srcbuf);
        tbm_surface_unmap(dstbuf);

diff --git a/src/tdm_pp.c b/src/tdm_pp.c
index a440c41d44457a425d31a206f080ee41f9c2923e..9075c8b6f7a9fc9f348334f89963cc37048a23fe 100644 (file)
--- a/src/tdm_pp.c
+++ b/src/tdm_pp.c
@@ -112,7 +112,7 @@ tdm_pp_cb_done(tdm_pp *pp_backend, tbm_surface_h src, tbm_surface_h dst,
 {
        tdm_private_pp *private_pp = user_data;
        tdm_private_display *private_display = private_pp->private_display;
-       tdm_pp_private_buffer *pp_buffer, *first_entry;
+       tdm_pp_private_buffer *pp_buffer = NULL, *first_entry = NULL;
 
        TDM_RETURN_IF_FAIL(TDM_MUTEX_IS_LOCKED());
 

diff --git a/src/tdm_thread.c b/src/tdm_thread.c
index aff3f39c207648c1f65eec358095a184b6a37c04..0d24561d0bc010520569c24cf6f8b0610856bd7b 100644 (file)
--- a/src/tdm_thread.c
+++ b/src/tdm_thread.c
@@ -240,7 +240,7 @@ tdm_thread_handle_cb(tdm_private_loop *private_loop)
        tdm_private_thread *private_thread;
        tdm_thread_cb_base *base;
        char buffer[1024];
-       int len, i;
+       unsigned int len, i;
 
        /* DON'T check TDM_MUTEX_IS_LOCKED here */