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

@@ -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:

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;

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

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)

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);
      }

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 *

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;
 }