enlightenment/clouseau
enlightenment
/
clouseau
7
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Revert "Clouseau: Don't clear the list if an app has exited." 

It's not correct. It breaks cleaning up of apps that have been closed,
even if not currently open. Should be reworked.

This fixes T638.

This reverts commit 81d2d9d021.
This commit is contained in:
Tom Hacohen 2013-12-13 12:38:30 +00:00
parent 758e8e8249
commit 18ce861cff
1 changed files with 24 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
src/bin/clouseau_client.c
View File

@ -473,9 +473,30 @@ _free_app(App_Data_St *st)
}
static void
_remove_app(Gui_Elements *g EINA_UNUSED, app_closed_st *app EINA_UNUSED)
{
/* We don't really want to do anything with app removals atm. */
_remove_app(Gui_Elements *g, app_closed_st *app)
{ /* Handle the case that NO app is selected, set sel_app to NULL */
app_info_st *sel_app = (g->sel_app) ? g->sel_app->app: NULL;
App_Data_St *st = (App_Data_St *)
eina_list_search_unsorted(apps, _app_ptr_cmp,
(void *) (uintptr_t) app->ptr);
/* if NO app selected OR closing app is the selected one, reset display */
if ((!sel_app) || (app->ptr == sel_app->ptr))
_set_selected_app(NULL, g->dd_list, NULL);
if (st)
{ /* Remove from list and free all app info */
Eina_List *l;
apps = eina_list_remove(apps, st);
_free_app(st);
if (elm_hoversel_expanded_get(g->dd_list))
elm_hoversel_hover_end(g->dd_list);
elm_hoversel_clear(g->dd_list);
EINA_LIST_FOREACH(apps, l , st)
_add_app_to_dd_list(g->dd_list, st);
}
}
static void