Make clang happy

SVN revision: 44257

diff --git a/legacy/ecore/src/lib/ecore/ecore_exe.c b/legacy/ecore/src/lib/ecore/ecore_exe.c
index 86a1f5c..4ce9afb 100644 (file)
--- a/legacy/ecore/src/lib/ecore/ecore_exe.c
+++ b/legacy/ecore/src/lib/ecore/ecore_exe.c
@@ -1422,16 +1422,12 @@ _ecore_exe_data_generic_handler(void *data, Ecore_Fd_Handler *fd_handler,
        flags = ECORE_EXE_PIPE_READ;
        event_type = ECORE_EXE_EVENT_DATA;
        child_fd = exe->child_fd_read;
-       if (exe->flags & ECORE_EXE_PIPE_READ_LINE_BUFFERED)
-          is_buffered = 1;
      }
    else
      {
        flags = ECORE_EXE_PIPE_ERROR;
        event_type = ECORE_EXE_EVENT_ERROR;
        child_fd = exe->child_fd_error;
-       if (exe->flags & ECORE_EXE_PIPE_ERROR_LINE_BUFFERED)
-          is_buffered = 1;
      }
 
    if ((fd_handler)

diff --git a/legacy/ecore/src/lib/ecore/ecore_getopt.c b/legacy/ecore/src/lib/ecore/ecore_getopt.c
index 472e6dc..eea801f 100644 (file)
--- a/legacy/ecore/src/lib/ecore/ecore_getopt.c
+++ b/legacy/ecore/src/lib/ecore/ecore_getopt.c
@@ -557,10 +557,10 @@ _ecore_getopt_help_desc(FILE *fp, const Ecore_Getopt_Desc *desc)
    switch (desc->action)
      {
       case ECORE_GETOPT_ACTION_STORE:
-        used = _ecore_getopt_help_desc_store(fp, helpcol, cols, used, desc);
+        _ecore_getopt_help_desc_store(fp, helpcol, cols, used, desc);
         break;
       case ECORE_GETOPT_ACTION_CHOICE:
-        used = _ecore_getopt_help_desc_choices(fp, helpcol, cols, used, desc);
+        _ecore_getopt_help_desc_choices(fp, helpcol, cols, used, desc);
         break;
       default:
         break;
@@ -746,7 +746,6 @@ _ecore_getopt_parse_find_nonargs_base(const Ecore_Getopt *parser, int argc, char
        base = dst;
        if (src != dst)
          argv[dst] = argv[src];
-       src++;
        dst++;
      }
 

diff --git a/legacy/ecore/src/lib/ecore/ecore_main.c b/legacy/ecore/src/lib/ecore/ecore_main.c
index e593f93..a22336f 100644 (file)
--- a/legacy/ecore/src/lib/ecore/ecore_main.c
+++ b/legacy/ecore/src/lib/ecore/ecore_main.c
@@ -662,7 +662,6 @@ _ecore_main_loop_iterate_internal(int once_only)
    if (_ecore_event_exist())
      {
        have_event = 1;
-       have_signal = 1;
        _ecore_main_select(0.0);
        goto process_events;
      }
@@ -688,7 +687,6 @@ _ecore_main_loop_iterate_internal(int once_only)
    if (_ecore_event_exist())
      {
        have_event = 1;
-       have_signal = 1;
        _ecore_main_select(0.0);
        goto process_events;
      }
@@ -725,7 +723,6 @@ _ecore_main_loop_iterate_internal(int once_only)
             if (!_ecore_idler_exist())
               {
                  if (_ecore_main_select(-1.0) > 0) have_event = 1;
-                 if (_ecore_signal_count_get() > 0) have_signal = 1;
               }
             /* idlers */
             else
@@ -751,7 +748,6 @@ _ecore_main_loop_iterate_internal(int once_only)
             if (!_ecore_idler_exist())
               {
                  if (_ecore_main_select(next_time) > 0) have_event = 1;
-                 if (_ecore_signal_count_get() > 0) have_signal = 1;
               }
             /* idlers */
             else

diff --git a/legacy/ecore/src/lib/ecore/ecore_tree.c b/legacy/ecore/src/lib/ecore/ecore_tree.c
index 59c63a2..3e7da17 100644 (file)
--- a/legacy/ecore/src/lib/ecore/ecore_tree.c
+++ b/legacy/ecore/src/lib/ecore/ecore_tree.c
@@ -650,9 +650,7 @@ tree_node_find_parent(Ecore_Tree *tree, const void *key)
 
    while (!parent)
      {
-       int compare;
-
-       if ((compare = tree->compare_func(key, travel->key)) < 0)
+       if (tree->compare_func(key, travel->key) < 0)
          {
             if (!travel->right_child)
               parent = travel;

diff --git a/legacy/ecore/src/lib/ecore_config/ecore_config.c b/legacy/ecore/src/lib/ecore_config/ecore_config.c
index 919af86..8e6650f 100644 (file)
--- a/legacy/ecore/src/lib/ecore_config/ecore_config.c
+++ b/legacy/ecore/src/lib/ecore_config/ecore_config.c
@@ -449,7 +449,6 @@ ecore_config_type_guess(const char *key, const char *val)
 {
    Ecore_Config_Prop  *p;
    char               *l;
-   long                v;
 
    l = NULL;
 
@@ -460,7 +459,7 @@ ecore_config_type_guess(const char *key, const char *val)
       return ECORE_CONFIG_NIL;
    if (val[0] == '#')
       return ECORE_CONFIG_RGB;
-   v = strtol(val, &l, 10);
+   strtol(val, &l, 10);
    if (*l)
      {
        float               f;
@@ -1639,7 +1638,6 @@ ecore_config_init_global(const char *name)
    char               *p;
    int global;
    char               *buf;
-   global = 0;
 
    if ((p = getenv("HOME")))
      {                         /* debug-only ### FIXME */

diff --git a/legacy/ecore/src/lib/ecore_config/ecore_config_db.c b/legacy/ecore/src/lib/ecore_config/ecore_config_db.c
index b509b9a..96bcb0b 100644 (file)
--- a/legacy/ecore/src/lib/ecore_config/ecore_config_db.c
+++ b/legacy/ecore/src/lib/ecore_config/ecore_config_db.c
@@ -122,7 +122,6 @@ _ecore_config_db_read(Ecore_Config_DB_File *db, const char *key)
 {
    char *data, *value;
    int size;
-   Ecore_Config_Prop *prop;
    Ecore_Config_Type type;
    
    data = eet_read(db->ef, (char*)key, &size);
@@ -151,8 +150,7 @@ _ecore_config_db_read(Ecore_Config_DB_File *db, const char *key)
    
        type = data[0];
        value = data + l + 1;
-       prop = ecore_config_get(key);
-       
+
        switch (type) 
          {
             case ECORE_CONFIG_INT:

diff --git a/legacy/ecore/src/lib/ecore_config/ecore_config_ipc_ecore.c b/legacy/ecore/src/lib/ecore_config/ecore_config_ipc_ecore.c
index bd2ec03..ebe3df6 100644 (file)
--- a/legacy/ecore/src/lib/ecore_config/ecore_config_ipc_ecore.c
+++ b/legacy/ecore/src/lib/ecore_config/ecore_config_ipc_ecore.c
@@ -158,7 +158,6 @@ _ecore_config_ipc_ecore_handle_request(Ecore_Ipc_Server * server,
 
    srv = _ecore_config_server_convert(server);
    serial = e->minor;
-   ret = ECORE_CONFIG_ERR_FAIL;
    r = NULL;
    m = (char *)e->data;
    E(1, "IPC/eCore: client sent: [%d,%d] #%d (%d) @ %p\n", e->major, e->minor,

diff --git a/legacy/ecore/src/lib/ecore_config/ecore_config_ipc_main.c b/legacy/ecore/src/lib/ecore_config/ecore_config_ipc_main.c
index 909dff6..704878f 100644 (file)
--- a/legacy/ecore/src/lib/ecore_config/ecore_config_ipc_main.c
+++ b/legacy/ecore/src/lib/ecore_config/ecore_config_ipc_main.c
@@ -93,12 +93,9 @@ _ecore_config_ipc_prop_desc(Ecore_Config_Server * srv, const long serial,
                            const char *key)
 {
 #ifdef HAVE_EVAS2
-   Ecore_Config_Bundle *theme;
    Ecore_Config_Prop  *e;
 
-   theme = ecore_config_bundle_by_serial_get(srv, serial);
    e = ecore_config_get(key);
-
    if (e)
      {
        estring            *s = estring_new(512);
@@ -118,11 +115,8 @@ _ecore_config_ipc_prop_get(Ecore_Config_Server * srv, const long serial,
 {
 #ifdef HAVE_EVAS2
    char               *ret;
-   Ecore_Config_Bundle *theme;
 
-   ret = NULL;
-   theme = ecore_config_bundle_by_serial_get(srv, serial);
-   if ((ret = ecore_config_as_string_get( /*theme, */ key)))
+   if ((ret = ecore_config_as_string_get(key)))
       return ret;
 #endif
    return strdup("<undefined>");

diff --git a/legacy/ecore/src/lib/ecore_evas/ecore_evas_x.c b/legacy/ecore/src/lib/ecore_evas/ecore_evas_x.c
index 045727d..2200663 100644 (file)
--- a/legacy/ecore/src/lib/ecore_evas/ecore_evas_x.c
+++ b/legacy/ecore/src/lib/ecore_evas/ecore_evas_x.c
@@ -3038,9 +3038,6 @@ ecore_evas_software_x11_16_new(const char *disp_name, Ecore_X_Window parent,
 
    if (einfo)
      {
-       int screen;
-
-       screen = DefaultScreen(ecore_x_display_get());
        if (ScreenCount(ecore_x_display_get()) > 1)
          {
             Ecore_X_Window *roots;
@@ -3058,10 +3055,7 @@ ecore_evas_software_x11_16_new(const char *disp_name, Ecore_X_Window parent,
                       for (i = 0; i < num; i++)
                         {
                            if (at.root == roots[i])
-                             {
-                                screen = i;
-                                break;
-                             }
+                             break;
                         }
                    }
                  free(roots);

diff --git a/legacy/ecore/src/lib/ecore_file/ecore_file_monitor_poll.c b/legacy/ecore/src/lib/ecore_file/ecore_file_monitor_poll.c
index fe3be7b..990dc2c 100644 (file)
--- a/legacy/ecore/src/lib/ecore_file/ecore_file_monitor_poll.c
+++ b/legacy/ecore/src/lib/ecore_file/ecore_file_monitor_poll.c
@@ -209,10 +209,8 @@ static void
 _ecore_file_monitor_poll_check(Ecore_File_Monitor *em)
 {
    int mtime;
-   int is_dir;
 
    mtime = ecore_file_mod_time(em->path);
-   is_dir = ecore_file_is_dir(em->path);
    if (mtime < ECORE_FILE_MONITOR_POLL(em)->mtime)
      {
        Ecore_File *l;

diff --git a/legacy/ecore/src/lib/ecore_ipc/ecore_ipc.c b/legacy/ecore/src/lib/ecore_ipc/ecore_ipc.c
index 6e1f0b2..f74fedd 100644 (file)
--- a/legacy/ecore/src/lib/ecore_ipc/ecore_ipc.c
+++ b/legacy/ecore/src/lib/ecore_ipc/ecore_ipc.c
@@ -284,7 +284,7 @@ ecore_ipc_init(void)
                                          _ecore_ipc_event_server_del, NULL);
    handler[i++] = ecore_event_handler_add(ECORE_CON_EVENT_CLIENT_DATA,
                                          _ecore_ipc_event_client_data, NULL);
-   handler[i++] = ecore_event_handler_add(ECORE_CON_EVENT_SERVER_DATA,
+   handler[i] = ecore_event_handler_add(ECORE_CON_EVENT_SERVER_DATA,
                                          _ecore_ipc_event_server_data, NULL);
    return _ecore_ipc_init_count;
 }

diff --git a/legacy/ecore/src/lib/ecore_txt/ecore_txt.c b/legacy/ecore/src/lib/ecore_txt/ecore_txt.c
index 29608af..eb9201a 100644 (file)
--- a/legacy/ecore/src/lib/ecore_txt/ecore_txt.c
+++ b/legacy/ecore/src/lib/ecore_txt/ecore_txt.c
@@ -37,7 +37,6 @@ ecore_txt_convert(const char *enc_from, const char *enc_to, const char *text)
    outp     = new_txt;
    outalloc = 64;
    outlen   = 0;
-   tob      = 0;
 
    for (;;)
      {

diff --git a/legacy/ecore/src/lib/ecore_x/xlib/ecore_x_composite.c b/legacy/ecore/src/lib/ecore_x/xlib/ecore_x_composite.c
index 7ef3589..f9e091c 100644 (file)
--- a/legacy/ecore/src/lib/ecore_x/xlib/ecore_x_composite.c
+++ b/legacy/ecore/src/lib/ecore_x/xlib/ecore_x_composite.c
@@ -35,7 +35,7 @@ EAPI void
 ecore_x_composite_redirect_window(Ecore_X_Window win, Ecore_X_Composite_Update_Type type)
 {
 #ifdef ECORE_XCOMPOSITE
-   int update;
+   int update = CompositeRedirectAutomatic;
 
    switch(type)
      {
@@ -54,7 +54,7 @@ EAPI void
 ecore_x_composite_redirect_subwindows(Ecore_X_Window win, Ecore_X_Composite_Update_Type type)
 {
 #ifdef ECORE_XCOMPOSITE
-   int update;
+   int update = CompositeRedirectAutomatic;
 
    switch(type)
      {
@@ -73,7 +73,7 @@ EAPI void
 ecore_x_composite_unredirect_window(Ecore_X_Window win, Ecore_X_Composite_Update_Type type)
 {
 #ifdef ECORE_XCOMPOSITE
-   int update;
+   int update = CompositeRedirectAutomatic;
 
    switch(type)
      {
@@ -92,7 +92,7 @@ EAPI void
 ecore_x_composite_unredirect_subwindows(Ecore_X_Window win, Ecore_X_Composite_Update_Type type)
 {
 #ifdef ECORE_XCOMPOSITE
-   int update;
+   int update = CompositeRedirectAutomatic;
 
    switch(type)
      {

diff --git a/legacy/ecore/src/lib/ecore_x/xlib/ecore_x_events.c b/legacy/ecore/src/lib/ecore_x/xlib/ecore_x_events.c
index dd5b408..7ac72ee 100644 (file)
--- a/legacy/ecore/src/lib/ecore_x/xlib/ecore_x_events.c
+++ b/legacy/ecore/src/lib/ecore_x/xlib/ecore_x_events.c
@@ -1410,7 +1410,6 @@ _ecore_x_event_handle_client_message(XEvent *xevent)
      {
        Ecore_X_Event_Xdnd_Enter *e;
        Ecore_X_DND_Target *target;
-       unsigned long three;
 
        e = calloc(1, sizeof(Ecore_X_Event_Xdnd_Enter));
        if (!e) return;
@@ -1427,7 +1426,7 @@ _ecore_x_event_handle_client_message(XEvent *xevent)
             return;
          }
 
-       if ((three = xevent->xclient.data.l[1] & 0x1UL))
+       if (xevent->xclient.data.l[1] & 0x1UL)
          {
             /* source supports more than 3 types, fetch property */
             unsigned char *data;

diff --git a/legacy/ecore/src/lib/ecore_x/xlib/ecore_x_netwm.c b/legacy/ecore/src/lib/ecore_x/xlib/ecore_x_netwm.c
index f330fe2..844702d 100644 (file)
--- a/legacy/ecore/src/lib/ecore_x/xlib/ecore_x_netwm.c
+++ b/legacy/ecore/src/lib/ecore_x/xlib/ecore_x_netwm.c
@@ -1145,7 +1145,7 @@ ecore_x_netwm_desktop_request_send(Ecore_X_Window win, Ecore_X_Window root, unsi
 }
 
 int
-_ecore_x_netwm_startup_info_begin(Ecore_X_Window win, char *data)
+_ecore_x_netwm_startup_info_begin(Ecore_X_Window win __UNUSED__, char *data __UNUSED__)
 {
 #if 0
    Ecore_X_Startup_Info *info;
@@ -1182,15 +1182,12 @@ _ecore_x_netwm_startup_info_begin(Ecore_X_Window win, char *data)
        /* We have a '\0' in there, the message is done */
        _ecore_x_netwm_startup_info_process(info);
      }
-#else
-   win = 0;
-   data = NULL;
 #endif
    return 1;
 }
 
 int
-_ecore_x_netwm_startup_info(Ecore_X_Window win, char *data)
+_ecore_x_netwm_startup_info(Ecore_X_Window win __UNUSED__, char *data __UNUSED__)
 {
 #if 0
    Ecore_X_Startup_Info *info;
@@ -1219,9 +1216,6 @@ _ecore_x_netwm_startup_info(Ecore_X_Window win, char *data)
        /* We have a '\0' in there, the message is done */
        _ecore_x_netwm_startup_info_process(info);
      }
-#else
-   win = 0;
-   data = NULL;
 #endif
    return 1;
 }

diff --git a/legacy/ecore/src/lib/ecore_x/xlib/ecore_x_randr.c b/legacy/ecore/src/lib/ecore_x/xlib/ecore_x_randr.c
index 580b49d..5cba091 100644 (file)
--- a/legacy/ecore/src/lib/ecore_x/xlib/ecore_x_randr.c
+++ b/legacy/ecore/src/lib/ecore_x/xlib/ecore_x_randr.c
@@ -85,9 +85,9 @@ EAPI Ecore_X_Randr_Rotation
 ecore_x_randr_screen_rotation_get(Ecore_X_Window root)
 {
 #ifdef ECORE_XRANDR
-   Rotation rot, crot = 0;
+   Rotation crot = 0;
    
-   rot = XRRRotations(_ecore_x_disp, XRRRootToScreen(_ecore_x_disp, root), &crot);
+   XRRRotations(_ecore_x_disp, XRRRootToScreen(_ecore_x_disp, root), &crot);
    return crot;
 #else
    return 0;

diff --git a/legacy/ecore/src/lib/ecore_x/xlib/ecore_x_screensaver.c b/legacy/ecore/src/lib/ecore_x/xlib/ecore_x_screensaver.c
index 9b769fd..bfae17b 100644 (file)
--- a/legacy/ecore/src/lib/ecore_x/xlib/ecore_x_screensaver.c
+++ b/legacy/ecore/src/lib/ecore_x/xlib/ecore_x_screensaver.c
@@ -144,7 +144,5 @@ ecore_x_screensaver_event_listen_set(int on)
      XScreenSaverSelectInput(_ecore_x_disp, root, ScreenSaverNotifyMask);
    else
      XScreenSaverSelectInput(_ecore_x_disp, root, 0);
-#else
-   on = 0;
 #endif
 }