x11: Do not swallow XSettings events
authorEmmanuele Bassi <ebassi@linux.intel.com>
Tue, 6 Jul 2010 16:13:55 +0000 (17:13 +0100)
committerEmmanuele Bassi <ebassi@linux.intel.com>
Wed, 7 Jul 2010 13:52:19 +0000 (14:52 +0100)
We might want pieces higher in the stack (like Mx) to handle XSettings
events as well, and swallowing them by removing them from the events
queue would make it impossible.

clutter/x11/clutter-backend-x11.c

index b810463..81ce4d8 100644 (file)
@@ -108,10 +108,13 @@ xsettings_filter (XEvent       *xevent,
 {
   ClutterBackendX11 *backend_x11 = data;
 
-  if (_clutter_xsettings_client_process_event (backend_x11->xsettings, xevent))
-    return CLUTTER_X11_FILTER_REMOVE;
-  else
-    return CLUTTER_X11_FILTER_CONTINUE;
+  _clutter_xsettings_client_process_event (backend_x11->xsettings, xevent);
+
+  /* we always want the rest of the stack to get XSettings events, even
+   * if Clutter already handled them
+   */
+
+  return CLUTTER_X11_FILTER_CONTINUE;
 }
 
 static Bool