enlightenment/enlightenment
enlightenment
/
enlightenment
10
9
Fork 12
Code Issues 25 Pull Requests 2 Projects Releases Wiki Activity

strcmp using freed pointers is bad, mmmmmk? xsettings ui now selects the currently used theme if it is known 

SVN revision: 74377
This commit is contained in:
Mike Blumenkrantz 2012-07-25 09:15:27 +00:00
parent f45c212c42
commit cc6d298edd
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/modules/conf_theme/e_int_config_xsettings.c
View File

@ -273,11 +273,10 @@ _fill_files_ilist(void *data)
so val remains valid */
label = eina_stringshare_add(tmp);
e_widget_ilist_append(o, NULL, label, NULL, NULL, label /* val */);
if ((e_config->xsettings.net_theme_name_detected == label) || (cfdata->widget_theme == label))
e_widget_ilist_selected_set(cfdata->gui.widget_list, cnt);
eina_stringshare_del(label);
free(tmp);
if (cfdata->widget_theme && !strcmp(cfdata->widget_theme, tmp))
e_widget_ilist_selected_set(cfdata->gui.widget_list, cnt);
cnt++;
}