From 427b6c34dfb1d7fa724262f23178841c3d8ba5e3 Mon Sep 17 00:00:00 2001
From: Sebastian Dransfeld <sd@tango.flipp.net>
Date: Thu, 4 Oct 2012 23:26:48 +0000
Subject: [PATCH] e: Cleanup

Removes unused variables, drops duplicate return or break
     statements and drops unnecessary checks.

Patch by Igor Murzov

SVN revision: 77490
---
 src/bin/e_zone.c                            | 1 -
 src/modules/battery/batget.c                | 3 ---
 src/modules/conf_theme/e_int_config_scale.c | 1 -
 src/modules/cpufreq/e_mod_main.c            | 2 +-
 src/modules/fileman/e_mod_menu.c            | 2 +-
 src/modules/illume-keyboard/e_kbd_dict.c    | 1 -
 src/modules/xkbswitch/e_mod_main.c          | 1 -
 7 files changed, 2 insertions(+), 9 deletions(-)

diff --git a/src/bin/e_zone.c b/src/bin/e_zone.c
index d091a1904..14f4378bd 100644
--- a/src/bin/e_zone.c
+++ b/src/bin/e_zone.c
@@ -1296,7 +1296,6 @@ _e_zone_useful_geometry_calc(E_Zone *zone)
               if (yy1 > zone->h - shelf->h)
                 yy1 = zone->h - shelf->h;
               break;
-              break;
 
             case E_GADCON_ORIENT_LEFT:
             case E_GADCON_ORIENT_CORNER_LT:
diff --git a/src/modules/battery/batget.c b/src/modules/battery/batget.c
index 538b74ccb..94430cd0d 100644
--- a/src/modules/battery/batget.c
+++ b/src/modules/battery/batget.c
@@ -272,8 +272,6 @@ static void
 bsd_apm_check(void)
 {
    int ac_stat, bat_stat, bat_val, time_val;
-   char buf[4096];
-   int hours, minutes;
    int apm_fd = -1;
    struct apm_info info;
 
@@ -357,7 +355,6 @@ darwin_check(void)
    const void *values;
    int device_num, device_count;
    int currentval = 0, maxval = 0;
-   char buf[4096];
    CFTypeRef blob;
    CFArrayRef sources;
    CFDictionaryRef device_dict;
diff --git a/src/modules/conf_theme/e_int_config_scale.c b/src/modules/conf_theme/e_int_config_scale.c
index 7c19a8ee6..50d2f26c0 100644
--- a/src/modules/conf_theme/e_int_config_scale.c
+++ b/src/modules/conf_theme/e_int_config_scale.c
@@ -386,7 +386,6 @@ _adv_changed(E_Config_Dialog *cfd __UNUSED__, E_Config_Dialog_Data *cfdata)
           (cfdata->max != e_config->scale.max) ||
           (cfdata->factor != e_config->scale.factor) ||
           (cfdata->base_dpi != e_config->scale.base_dpi);
-   return 1;
 }
 
 static void
diff --git a/src/modules/cpufreq/e_mod_main.c b/src/modules/cpufreq/e_mod_main.c
index 31b11e22b..870520dd2 100644
--- a/src/modules/cpufreq/e_mod_main.c
+++ b/src/modules/cpufreq/e_mod_main.c
@@ -597,7 +597,7 @@ _cpufreq_status_check_available(Status *s)
 #elif defined (__FreeBSD__)
    int freq;
    size_t len = sizeof(buf);
-   char *freqs, *pos, *q;
+   char *pos, *q;
 
    /* read freq_levels sysctl and store it in freq */
    if (sysctlbyname("dev.cpu.0.freq_levels", buf, &len, NULL, 0) == 0)
diff --git a/src/modules/fileman/e_mod_menu.c b/src/modules/fileman/e_mod_menu.c
index 1a474b94e..54f973f76 100644
--- a/src/modules/fileman/e_mod_menu.c
+++ b/src/modules/fileman/e_mod_menu.c
@@ -274,7 +274,7 @@ _e_mod_fileman_parse_gtk_bookmarks(E_Menu   *m,
              e_menu_item_callback_set(mi, _e_mod_menu_gtk_cb,
                                       (void *)eina_stringshare_add(uri->path));
              e_menu_item_submenu_pre_callback_set(mi, _e_mod_menu_populate, eina_stringshare_add("/"));
-             if (uri) efreet_uri_free(uri);
+             efreet_uri_free(uri);
           }
         fclose(fp);
      }
diff --git a/src/modules/illume-keyboard/e_kbd_dict.c b/src/modules/illume-keyboard/e_kbd_dict.c
index 5a4218a59..eeb71a295 100644
--- a/src/modules/illume-keyboard/e_kbd_dict.c
+++ b/src/modules/illume-keyboard/e_kbd_dict.c
@@ -153,7 +153,6 @@ _e_kbd_dict_writes_cb_sort(const void *d1, const void *d2)
    kw1 = d1;
    kw2 = d2;
    return _e_kbd_dict_normalized_strcmp(kw1->word, kw2->word);
-   return 0;
 }
 
 static const char *
diff --git a/src/modules/xkbswitch/e_mod_main.c b/src/modules/xkbswitch/e_mod_main.c
index 6ceba125d..3bac3cb2b 100644
--- a/src/modules/xkbswitch/e_mod_main.c
+++ b/src/modules/xkbswitch/e_mod_main.c
@@ -496,7 +496,6 @@ _e_xkb_cb_lmenu_post(void *data, E_Menu *menu __UNUSED__)
 
    if (!inst) return;
    e_gadcon_locked_set(inst->gcc->gadcon, 0);
-   if (!(inst) || !inst->lmenu) return;
    inst->lmenu = NULL;
 }