#include "xsettings-client.h"
-#include <limits.h>
-#include <stdio.h>
-#include <stdlib.h>
+#include <gdk/x11/gdkx11display.h>
+#include <gdk/x11/gdkx11screen.h>
+#include <gdk/x11/gdkx11window.h>
+#include <gdk/x11/gdkscreen-x11.h>
+
+#include <gdkinternals.h>
+
#include <string.h>
#include <X11/Xlib.h>
#include <X11/Xmd.h> /* For CARD16 */
+#include "gdksettings.c"
+
+typedef struct _XSettingsBuffer XSettingsBuffer;
+
+struct _XSettingsBuffer
+{
+ char byte_order;
+ size_t len;
+ unsigned char *data;
+ unsigned char *pos;
+};
+
struct _XSettingsClient
{
+ GdkScreen *screen;
Display *display;
- int screen;
- XSettingsNotifyFunc notify;
- XSettingsWatchFunc watch;
- void *cb_data;
-
- XSettingsGrabFunc grab;
- XSettingsGrabFunc ungrab;
Window manager_window;
Atom manager_atom;
Atom selection_atom;
Atom xsettings_atom;
- XSettingsList *settings;
+ GHashTable *settings; /* string => XSettingsSetting */
};
+static void
+gdk_xsettings_notify (const char *name,
+ GdkSettingAction action,
+ XSettingsSetting *setting,
+ GdkScreen *screen)
+{
+ GdkEvent new_event;
+ GdkX11Screen *x11_screen = GDK_X11_SCREEN (screen);
+
+ if (x11_screen->xsettings_in_init)
+ return;
+
+ new_event.type = GDK_SETTING;
+ new_event.setting.window = gdk_screen_get_root_window (screen);
+ new_event.setting.send_event = FALSE;
+ new_event.setting.action = action;
+ new_event.setting.name = (char*) gdk_from_xsettings_name (name);
+
+ if (!new_event.setting.name)
+ return;
+
+ gdk_event_put (&new_event);
+}
+
static void
notify_changes (XSettingsClient *client,
- XSettingsList *old_list)
+ GHashTable *old_list)
{
GHashTableIter iter;
XSettingsSetting *setting, *old_setting;
-
- if (!client->notify)
- return;
+ const char *name;
if (client->settings != NULL)
{
g_hash_table_iter_init (&iter, client->settings);
- while (g_hash_table_iter_next (&iter, NULL, (gpointer*) &setting))
+ while (g_hash_table_iter_next (&iter, (gpointer *) &name, (gpointer*) &setting))
{
- old_setting = xsettings_list_lookup (old_list, setting->name);
+ old_setting = old_list ? g_hash_table_lookup (old_list, name) : NULL;
if (old_setting == NULL)
- client->notify (setting->name, XSETTINGS_ACTION_NEW, setting, client->cb_data);
+ gdk_xsettings_notify (name, GDK_SETTING_ACTION_NEW, setting, client->screen);
else if (!xsettings_setting_equal (setting, old_setting))
- client->notify (setting->name, XSETTINGS_ACTION_CHANGED, setting, client->cb_data);
+ gdk_xsettings_notify (name, GDK_SETTING_ACTION_CHANGED, setting, client->screen);
/* remove setting from old_list */
if (old_setting != NULL)
- g_hash_table_remove (old_list, setting->name);
+ g_hash_table_remove (old_list, name);
}
}
{
/* old_list now contains only deleted settings */
g_hash_table_iter_init (&iter, old_list);
- while (g_hash_table_iter_next (&iter, NULL, (gpointer*) &old_setting))
- client->notify (old_setting->name, XSETTINGS_ACTION_DELETED, NULL, client->cb_data);
+ while (g_hash_table_iter_next (&iter, (gpointer *) &name, (gpointer*) &old_setting))
+ gdk_xsettings_notify (name, GDK_SETTING_ACTION_DELETED, NULL, client->screen);
}
}
-static int
-ignore_errors (Display *display, XErrorEvent *event)
-{
- return True;
-}
-
-static char local_byte_order = '\0';
-
#define BYTES_LEFT(buffer) ((buffer)->data + (buffer)->len - (buffer)->pos)
-static XSettingsResult
+#define return_if_fail_bytes(buffer, n_bytes) G_STMT_START{ \
+ if (BYTES_LEFT (buffer) < (n_bytes)) \
+ { \
+ g_warning ("Invalid XSETTINGS property (read off end: Expected %u bytes, only %ld left", \
+ (n_bytes), BYTES_LEFT (buffer)); \
+ return FALSE; \
+ } \
+}G_STMT_END
+
+static gboolean
fetch_card16 (XSettingsBuffer *buffer,
CARD16 *result)
{
CARD16 x;
- if (BYTES_LEFT (buffer) < 2)
- return XSETTINGS_ACCESS;
+ return_if_fail_bytes (buffer, 2);
x = *(CARD16 *)buffer->pos;
buffer->pos += 2;
- if (buffer->byte_order == local_byte_order)
- *result = x;
+ if (buffer->byte_order == MSBFirst)
+ *result = GUINT16_FROM_BE (x);
else
- *result = (x << 8) | (x >> 8);
+ *result = GUINT16_FROM_LE (x);
- return XSETTINGS_SUCCESS;
+ return TRUE;
}
-static XSettingsResult
+static gboolean
fetch_ushort (XSettingsBuffer *buffer,
unsigned short *result)
{
CARD16 x;
- XSettingsResult r;
+ gboolean r;
r = fetch_card16 (buffer, &x);
- if (r == XSETTINGS_SUCCESS)
+ if (r)
*result = x;
return r;
}
-static XSettingsResult
+static gboolean
fetch_card32 (XSettingsBuffer *buffer,
CARD32 *result)
{
CARD32 x;
- if (BYTES_LEFT (buffer) < 4)
- return XSETTINGS_ACCESS;
+ return_if_fail_bytes (buffer, 4);
x = *(CARD32 *)buffer->pos;
buffer->pos += 4;
- if (buffer->byte_order == local_byte_order)
- *result = x;
+ if (buffer->byte_order == MSBFirst)
+ *result = GUINT32_FROM_BE (x);
else
- *result = (x << 24) | ((x & 0xff00) << 8) | ((x & 0xff0000) >> 8) | (x >> 24);
+ *result = GUINT32_FROM_LE (x);
- return XSETTINGS_SUCCESS;
+ return TRUE;
}
-static XSettingsResult
+static gboolean
fetch_card8 (XSettingsBuffer *buffer,
CARD8 *result)
{
- if (BYTES_LEFT (buffer) < 1)
- return XSETTINGS_ACCESS;
+ return_if_fail_bytes (buffer, 1);
*result = *(CARD8 *)buffer->pos;
buffer->pos += 1;
- return XSETTINGS_SUCCESS;
+ return TRUE;
}
#define XSETTINGS_PAD(n,m) ((n + m - 1) & (~(m-1)))
-static XSettingsList *
+static gboolean
+fetch_string (XSettingsBuffer *buffer,
+ guint length,
+ char **result)
+{
+ guint pad_len;
+
+ pad_len = XSETTINGS_PAD (length, 4);
+ if (pad_len < length) /* guard against overflow */
+ {
+ g_warning ("Invalid XSETTINGS property (overflow in string length)");
+ return FALSE;
+ }
+
+ return_if_fail_bytes (buffer, pad_len);
+
+ *result = g_strndup ((char *) buffer->pos, length);
+ buffer->pos += pad_len;
+
+ return TRUE;
+}
+
+static GHashTable *
parse_settings (unsigned char *data,
size_t len)
{
XSettingsBuffer buffer;
- XSettingsResult result = XSETTINGS_SUCCESS;
- XSettingsList *settings = NULL;
+ GHashTable *settings = NULL;
CARD32 serial;
CARD32 n_entries;
CARD32 i;
XSettingsSetting *setting = NULL;
+ char *name;
- local_byte_order = xsettings_byte_order ();
-
buffer.pos = buffer.data = data;
buffer.len = len;
- result = fetch_card8 (&buffer, (unsigned char *)&buffer.byte_order);
+ if (!fetch_card8 (&buffer, (unsigned char *)&buffer.byte_order))
+ goto out;
+
if (buffer.byte_order != MSBFirst &&
buffer.byte_order != LSBFirst)
{
- fprintf (stderr, "Invalid byte order in XSETTINGS property\n");
- result = XSETTINGS_FAILED;
+ g_warning ("Invalid XSETTINGS property (unknown byte order %u)", buffer.byte_order);
goto out;
}
buffer.pos += 3;
- result = fetch_card32 (&buffer, &serial);
- if (result != XSETTINGS_SUCCESS)
+ if (!fetch_card32 (&buffer, &serial) ||
+ !fetch_card32 (&buffer, &n_entries))
goto out;
- result = fetch_card32 (&buffer, &n_entries);
- if (result != XSETTINGS_SUCCESS)
- goto out;
+ GDK_NOTE(SETTINGS, g_print("reading %u settings (serial %u byte order %u)\n", n_entries, serial, buffer.byte_order));
for (i = 0; i < n_entries; i++)
{
CARD8 type;
CARD16 name_len;
CARD32 v_int;
- size_t pad_len;
- result = fetch_card8 (&buffer, &type);
- if (result != XSETTINGS_SUCCESS)
+ if (!fetch_card8 (&buffer, &type))
goto out;
buffer.pos += 1;
- result = fetch_card16 (&buffer, &name_len);
- if (result != XSETTINGS_SUCCESS)
+ if (!fetch_card16 (&buffer, &name_len))
goto out;
- pad_len = XSETTINGS_PAD(name_len, 4);
- if (BYTES_LEFT (&buffer) < pad_len)
- {
- result = XSETTINGS_ACCESS;
- goto out;
- }
-
- setting = malloc (sizeof *setting);
- if (!setting)
- {
- result = XSETTINGS_NO_MEM;
- goto out;
- }
+ setting = g_new (XSettingsSetting, 1);
setting->type = XSETTINGS_TYPE_INT; /* No allocated memory */
- setting->name = malloc (name_len + 1);
- if (!setting->name)
- {
- result = XSETTINGS_NO_MEM;
- goto out;
- }
-
- memcpy (setting->name, buffer.pos, name_len);
- setting->name[name_len] = '\0';
- buffer.pos += pad_len;
-
- result = fetch_card32 (&buffer, &v_int);
- if (result != XSETTINGS_SUCCESS)
+ if (!fetch_string (&buffer, name_len, &name) ||
+ /* last change serial (we ignore it) */
+ !fetch_card32 (&buffer, &v_int))
goto out;
- setting->last_change_serial = v_int;
switch (type)
{
case XSETTINGS_TYPE_INT:
- result = fetch_card32 (&buffer, &v_int);
- if (result != XSETTINGS_SUCCESS)
+ if (!fetch_card32 (&buffer, &v_int))
goto out;
setting->data.v_int = (INT32)v_int;
+ GDK_NOTE(SETTINGS, g_print(" %s = %d\n", name, (gint) setting->data.v_int));
break;
case XSETTINGS_TYPE_STRING:
- result = fetch_card32 (&buffer, &v_int);
- if (result != XSETTINGS_SUCCESS)
+ if (!fetch_card32 (&buffer, &v_int) ||
+ !fetch_string (&buffer, v_int, &setting->data.v_string))
goto out;
-
- pad_len = XSETTINGS_PAD (v_int, 4);
- if (v_int + 1 == 0 || /* Guard against wrap-around */
- BYTES_LEFT (&buffer) < pad_len)
- {
- result = XSETTINGS_ACCESS;
- goto out;
- }
-
- setting->data.v_string = malloc (v_int + 1);
- if (!setting->data.v_string)
- {
- result = XSETTINGS_NO_MEM;
- goto out;
- }
- memcpy (setting->data.v_string, buffer.pos, v_int);
- setting->data.v_string[v_int] = '\0';
- buffer.pos += pad_len;
-
+ GDK_NOTE(SETTINGS, g_print(" %s = \"%s\"\n", name, setting->data.v_string));
break;
case XSETTINGS_TYPE_COLOR:
- result = fetch_ushort (&buffer, &setting->data.v_color.red);
- if (result != XSETTINGS_SUCCESS)
- goto out;
- result = fetch_ushort (&buffer, &setting->data.v_color.green);
- if (result != XSETTINGS_SUCCESS)
- goto out;
- result = fetch_ushort (&buffer, &setting->data.v_color.blue);
- if (result != XSETTINGS_SUCCESS)
- goto out;
- result = fetch_ushort (&buffer, &setting->data.v_color.alpha);
- if (result != XSETTINGS_SUCCESS)
+ if (!fetch_ushort (&buffer, &setting->data.v_color.red) ||
+ !fetch_ushort (&buffer, &setting->data.v_color.green) ||
+ !fetch_ushort (&buffer, &setting->data.v_color.blue) ||
+ !fetch_ushort (&buffer, &setting->data.v_color.alpha))
goto out;
+ GDK_NOTE(SETTINGS, g_print(" %s = #%02X%02X%02X%02X\n", name,
+ setting->data.v_color.alpha, setting->data.v_color.red,
+ setting->data.v_color.green, setting->data.v_color.blue));
break;
default:
/* Quietly ignore unknown types */
+ GDK_NOTE(SETTINGS, g_print(" %s = ignored (unknown type %u)\n", name, type));
break;
}
setting->type = type;
- result = xsettings_list_insert (&settings, setting);
- if (result != XSETTINGS_SUCCESS)
- goto out;
+ if (settings == NULL)
+ settings = g_hash_table_new_full (g_str_hash, g_str_equal,
+ g_free,
+ (GDestroyNotify) xsettings_setting_free);
+
+ if (g_hash_table_lookup (settings, name) != NULL)
+ {
+ g_warning ("Invalid XSETTINGS property (Duplicate entry for '%s')", name);
+ goto out;
+ }
+ g_hash_table_insert (settings, name, setting);
setting = NULL;
+ name = NULL;
}
- out:
-
- if (result != XSETTINGS_SUCCESS)
- {
- switch (result)
- {
- case XSETTINGS_NO_MEM:
- fprintf(stderr, "Out of memory reading XSETTINGS property\n");
- break;
- case XSETTINGS_ACCESS:
- fprintf(stderr, "Invalid XSETTINGS property (read off end)\n");
- break;
- case XSETTINGS_DUPLICATE_ENTRY:
- fprintf (stderr, "Duplicate XSETTINGS entry for '%s'\n", setting->name);
- case XSETTINGS_FAILED:
- case XSETTINGS_SUCCESS:
- case XSETTINGS_NO_ENTRY:
- break;
- }
+ return settings;
- if (setting)
- xsettings_setting_free (setting);
+ out:
- xsettings_list_free (settings);
- settings = NULL;
+ if (setting)
+ xsettings_setting_free (setting);
- }
+ if (settings)
+ g_hash_table_unref (settings);
- return settings;
+ return NULL;
}
static void
unsigned char *data;
int result;
- int (*old_handler) (Display *, XErrorEvent *);
-
- XSettingsList *old_list = client->settings;
+ GHashTable *old_list = client->settings;
client->settings = NULL;
if (client->manager_window)
{
- old_handler = XSetErrorHandler (ignore_errors);
+ gdk_x11_display_error_trap_push (gdk_screen_get_display (client->screen));
result = XGetWindowProperty (client->display, client->manager_window,
client->xsettings_atom, 0, LONG_MAX,
False, client->xsettings_atom,
&type, &format, &n_items, &bytes_after, &data);
- XSetErrorHandler (old_handler);
+ gdk_x11_display_error_trap_pop_ignored (gdk_screen_get_display (client->screen));
if (result == Success && type != None)
{
}
notify_changes (client, old_list);
- xsettings_list_free (old_list);
+ if (old_list)
+ g_hash_table_unref (old_list);
}
static void
XSelectInput (display, window, attr.your_event_mask | mask);
}
+static Bool
+gdk_xsettings_watch (Window window,
+ Bool is_start,
+ long mask,
+ GdkScreen *screen);
+
static void
check_manager_window (XSettingsClient *client)
{
- if (client->manager_window && client->watch)
- client->watch (client->manager_window, False, 0, client->cb_data);
+ if (client->manager_window)
+ gdk_xsettings_watch (client->manager_window, False, 0, client->screen);
- if (client->grab)
- client->grab (client->display);
- else
- XGrabServer (client->display);
+ gdk_x11_display_grab (gdk_screen_get_display (client->screen));
client->manager_window = XGetSelectionOwner (client->display,
client->selection_atom);
XSelectInput (client->display, client->manager_window,
PropertyChangeMask | StructureNotifyMask);
- if (client->ungrab)
- client->ungrab (client->display);
- else
- XUngrabServer (client->display);
+ gdk_x11_display_ungrab (gdk_screen_get_display (client->screen));
XFlush (client->display);
- if (client->manager_window && client->watch)
+ if (client->manager_window)
{
- if (!client->watch (client->manager_window, True,
- PropertyChangeMask | StructureNotifyMask,
- client->cb_data))
+ if (!gdk_xsettings_watch (client->manager_window, True,
+ PropertyChangeMask | StructureNotifyMask,
+ client->screen))
{
/* Inability to watch the window probably means that it was destroyed
* after we ungrabbed
}
}
-
read_settings (client);
}
-XSettingsClient *
-xsettings_client_new (Display *display,
- int screen,
- XSettingsNotifyFunc notify,
- XSettingsWatchFunc watch,
- void *cb_data)
+static GdkFilterReturn
+gdk_xsettings_client_event_filter (GdkXEvent *xevent,
+ GdkEvent *event,
+ gpointer data)
{
- return xsettings_client_new_with_grab_funcs (display, screen, notify, watch, cb_data,
- NULL, NULL);
+ GdkScreen *screen = data;
+ XSettingsClient *client = GDK_X11_SCREEN (screen)->xsettings_client;
+ XEvent *xev = xevent;
+
+ /* The checks here will not unlikely cause us to reread
+ * the properties from the manager window a number of
+ * times when the manager changes from A->B. But manager changes
+ * are going to be pretty rare.
+ */
+ if (xev->xany.window == gdk_x11_window_get_xid (gdk_screen_get_root_window (screen)))
+ {
+ if (xev->xany.type == ClientMessage &&
+ xev->xclient.message_type == client->manager_atom &&
+ xev->xclient.data.l[1] == client->selection_atom)
+ {
+ check_manager_window (client);
+ return GDK_FILTER_REMOVE;
+ }
+ }
+ else if (xev->xany.window == client->manager_window)
+ {
+ if (xev->xany.type == DestroyNotify)
+ {
+ check_manager_window (client);
+ /* let GDK do its cleanup */
+ return GDK_FILTER_CONTINUE;
+ }
+ else if (xev->xany.type == PropertyNotify)
+ {
+ read_settings (client);
+ return GDK_FILTER_REMOVE;
+ }
+ }
+
+ return GDK_FILTER_CONTINUE;;
+}
+
+static Bool
+gdk_xsettings_watch (Window window,
+ Bool is_start,
+ long mask,
+ GdkScreen *screen)
+{
+ GdkWindow *gdkwin;
+
+ gdkwin = gdk_x11_window_lookup_for_display (gdk_screen_get_display (screen), window);
+
+ if (is_start)
+ {
+ if (gdkwin)
+ g_object_ref (gdkwin);
+ else
+ {
+ gdkwin = gdk_x11_window_foreign_new_for_display (gdk_screen_get_display (screen), window);
+
+ /* gdk_window_foreign_new_for_display() can fail and return NULL if the
+ * window has already been destroyed.
+ */
+ if (!gdkwin)
+ return False;
+ }
+
+ gdk_window_add_filter (gdkwin, gdk_xsettings_client_event_filter, screen);
+ }
+ else
+ {
+ if (!gdkwin)
+ {
+ /* gdkwin should not be NULL here, since if starting the watch succeeded
+ * we have a reference on the window. It might mean that the caller didn't
+ * remove the watch when it got a DestroyNotify event. Or maybe the
+ * caller ignored the return value when starting the watch failed.
+ */
+ g_warning ("gdk_xsettings_watch_cb(): Couldn't find window to unwatch");
+ return False;
+ }
+
+ gdk_window_remove_filter (gdkwin, gdk_xsettings_client_event_filter, screen);
+ g_object_unref (gdkwin);
+ }
+
+ return True;
}
XSettingsClient *
-xsettings_client_new_with_grab_funcs (Display *display,
- int screen,
- XSettingsNotifyFunc notify,
- XSettingsWatchFunc watch,
- void *cb_data,
- XSettingsGrabFunc grab,
- XSettingsGrabFunc ungrab)
+xsettings_client_new (GdkScreen *screen)
{
XSettingsClient *client;
char buffer[256];
char *atom_names[3];
Atom atoms[3];
- client = malloc (sizeof *client);
+ client = g_new (XSettingsClient, 1);
if (!client)
return NULL;
- client->display = display;
client->screen = screen;
- client->notify = notify;
- client->watch = watch;
- client->cb_data = cb_data;
- client->grab = grab;
- client->ungrab = ungrab;
+ client->display = gdk_x11_display_get_xdisplay (gdk_screen_get_display (screen));
+ client->screen = screen;
client->manager_window = None;
client->settings = NULL;
- sprintf(buffer, "_XSETTINGS_S%d", screen);
+ sprintf(buffer, "_XSETTINGS_S%d", gdk_x11_screen_get_screen_number (screen));
atom_names[0] = buffer;
atom_names[1] = "_XSETTINGS_SETTINGS";
atom_names[2] = "MANAGER";
- XInternAtoms (display, atom_names, 3, False, atoms);
+ XInternAtoms (client->display, atom_names, 3, False, atoms);
client->selection_atom = atoms[0];
client->xsettings_atom = atoms[1];
/* Select on StructureNotify so we get MANAGER events
*/
- add_events (display, RootWindow (display, screen), StructureNotifyMask);
+ add_events (client->display, gdk_x11_window_get_xid (gdk_screen_get_root_window (screen)), StructureNotifyMask);
- if (client->watch)
- client->watch (RootWindow (display, screen), True, StructureNotifyMask,
- client->cb_data);
+ gdk_xsettings_watch (gdk_x11_window_get_xid (gdk_screen_get_root_window (screen)), True, StructureNotifyMask,
+ client->screen);
check_manager_window (client);
return client;
}
-
void
-xsettings_client_set_grab_func (XSettingsClient *client,
- XSettingsGrabFunc grab)
+xsettings_client_destroy (XSettingsClient *client)
{
- client->grab = grab;
+ gdk_xsettings_watch (gdk_x11_window_get_xid (gdk_screen_get_root_window (client->screen)),
+ False, 0, client->screen);
+ if (client->manager_window)
+ gdk_xsettings_watch (client->manager_window, False, 0, client->screen);
+
+ if (client->settings)
+ g_hash_table_unref (client->settings);
+ g_free (client);
}
-void
-xsettings_client_set_ungrab_func (XSettingsClient *client,
- XSettingsGrabFunc ungrab)
+const XSettingsSetting *
+xsettings_client_get_setting (XSettingsClient *client,
+ const char *name)
{
- client->ungrab = ungrab;
+ return g_hash_table_lookup (client->settings, name);
}
-void
-xsettings_client_destroy (XSettingsClient *client)
+int
+xsettings_setting_equal (XSettingsSetting *setting_a,
+ XSettingsSetting *setting_b)
{
- if (client->watch)
- client->watch (RootWindow (client->display, client->screen),
- False, 0, client->cb_data);
- if (client->manager_window && client->watch)
- client->watch (client->manager_window, False, 0, client->cb_data);
-
- xsettings_list_free (client->settings);
- free (client);
-}
+ if (setting_a->type != setting_b->type)
+ return 0;
-XSettingsResult
-xsettings_client_get_setting (XSettingsClient *client,
- const char *name,
- XSettingsSetting **setting)
-{
- XSettingsSetting *search = xsettings_list_lookup (client->settings, name);
- if (search)
+ switch (setting_a->type)
{
- *setting = xsettings_setting_copy (search);
- return *setting ? XSETTINGS_SUCCESS : XSETTINGS_NO_MEM;
+ case XSETTINGS_TYPE_INT:
+ return setting_a->data.v_int == setting_b->data.v_int;
+ case XSETTINGS_TYPE_COLOR:
+ return (setting_a->data.v_color.red == setting_b->data.v_color.red &&
+ setting_a->data.v_color.green == setting_b->data.v_color.green &&
+ setting_a->data.v_color.blue == setting_b->data.v_color.blue &&
+ setting_a->data.v_color.alpha == setting_b->data.v_color.alpha);
+ case XSETTINGS_TYPE_STRING:
+ return strcmp (setting_a->data.v_string, setting_b->data.v_string) == 0;
}
- else
- return XSETTINGS_NO_ENTRY;
+
+ return 0;
}
-Bool
-xsettings_client_process_event (XSettingsClient *client,
- XEvent *xev)
+void
+xsettings_setting_free (XSettingsSetting *setting)
{
- /* The checks here will not unlikely cause us to reread
- * the properties from the manager window a number of
- * times when the manager changes from A->B. But manager changes
- * are going to be pretty rare.
- */
- if (xev->xany.window == RootWindow (client->display, client->screen))
- {
- if (xev->xany.type == ClientMessage &&
- xev->xclient.message_type == client->manager_atom &&
- xev->xclient.data.l[1] == client->selection_atom)
- {
- check_manager_window (client);
- return True;
- }
- }
- else if (xev->xany.window == client->manager_window)
- {
- if (xev->xany.type == DestroyNotify)
- {
- check_manager_window (client);
- /* let GDK do its cleanup */
- return False;
- }
- else if (xev->xany.type == PropertyNotify)
- {
- read_settings (client);
- return True;
- }
- }
-
- return False;
+ if (setting->type == XSETTINGS_TYPE_STRING)
+ g_free (setting->data.v_string);
+
+ g_free (setting);
}
+