brl/mutter
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

screen: Make meta_screen_foreach_window scan Wayland windows

Browse Source 
Scanning over the hash table of XIDs is a terrible idea. Not only were
we excluding Wayland windows, but we were also looking at alarms and
barriers, too. We were lucky that that only contained GObjects where
our checks would work.
This commit is contained in:
Jasper St. Pierre 2014-08-15 20:02:21 -04:00
parent a119ea96a3
commit 9c62a907c5
1 changed files with 7 additions and 41 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

48
src/core/screen.c
View File

@ -958,27 +958,6 @@ get_screen_name (MetaDisplay *display,
return scr;
}
static gint
ptrcmp (gconstpointer a, gconstpointer b)
{
if (a < b)
return -1;
else if (a > b)
return 1;
else
return 0;
}
static void
listify_func (gpointer key, gpointer value, gpointer data)
{
GSList **listp;
listp = data;
*listp = g_slist_prepend (*listp, value);
}
/**
* meta_screen_foreach_window:
* @screen: a #MetaScreen
@ -993,34 +972,21 @@ meta_screen_foreach_window (MetaScreen *screen,
MetaScreenWindowFunc func,
gpointer data)
{
GSList *winlist, *l;
GSList *windows, *l;
/* If we end up doing this often, just keeping a list
* of windows might be sensible.
*/
winlist = NULL;
g_hash_table_foreach (screen->display->xids,
listify_func,
&winlist);
windows = meta_display_list_windows (screen->display, META_LIST_DEFAULT);
winlist = g_slist_sort (winlist, ptrcmp);
for (l = winlist; l != NULL; l = l->next)
for (l = windows; l != NULL; l = l->next)
{
/* If the next node doesn't contain this window
* a second time, delete the window.
*/
if (l->next == NULL ||
(l->next && l->next->data != l->data))
{
MetaWindow *window = l->data;
if (META_IS_WINDOW (window) && !window->override_redirect)
(* func) (screen, window, data);
}
MetaWindow *window = l->data;
func (screen, window, data);
}
g_slist_free (winlist);
g_slist_free (windows);
}
int