From 30965869f0e3aaea6adb2765e98a6797e9f03749 Mon Sep 17 00:00:00 2001 From: Sebastian Dransfeld Date: Tue, 13 Nov 2012 22:51:11 +0000 Subject: [PATCH] efreet: don't double test for data in list SVN revision: 79254 --- legacy/efreet/src/bin/efreetd_cache.c | 17 ++++++++--------- 1 file changed, 8 insertions(+), 9 deletions(-) diff --git a/legacy/efreet/src/bin/efreetd_cache.c b/legacy/efreet/src/bin/efreetd_cache.c index 30ab231378..2b53060d52 100644 --- a/legacy/efreet/src/bin/efreetd_cache.c +++ b/legacy/efreet/src/bin/efreetd_cache.c @@ -422,21 +422,20 @@ cache_desktop_dir_add(const char *dir) san = eina_file_path_sanitize(dir); if (!san) return; - if ((!eina_list_search_unsorted_list(desktop_system_dirs, strcmplen, san)) && - (!eina_list_search_unsorted_list(desktop_extra_dirs, EINA_COMPARE_CB(strcmp), san))) - { - /* Not a registered path */ - desktop_extra_dirs = eina_list_append(desktop_extra_dirs, eina_stringshare_add(san)); - save_list("extra_desktop.dirs", desktop_extra_dirs); - cache_desktop_update(); - } - else if ((l = eina_list_search_unsorted_list(desktop_system_dirs, strcmplen, san))) + if ((l = eina_list_search_unsorted_list(desktop_system_dirs, strcmplen, san))) { /* Path is registered, but maybe not monitored */ const char *path = eina_list_data_get(l); if (!eina_hash_find(change_monitors, path)) cache_desktop_update(); } + else if (!eina_list_search_unsorted_list(desktop_extra_dirs, EINA_COMPARE_CB(strcmp), san)) + { + /* Not a registered path */ + desktop_extra_dirs = eina_list_append(desktop_extra_dirs, eina_stringshare_add(san)); + save_list("extra_desktop.dirs", desktop_extra_dirs); + cache_desktop_update(); + } free(san); }