TDE
/
tdebase
mirror of https://mirror.git.trinitydesktop.org/gitea/TDE/tdebase
0
0
Fork 0
Code Issues Releases Wiki Activity

Improvement: Add EWMH _NET_WM_ACTIVE_WINDOW support

Browse Source 
- Add support for using _NET_WM_ACTIVE_WINDOW to track focus (rather
  than using FocusIn/Out events) as --use-ewmh-active-win.

- Add wid_get_attr() and use it in place of XGetWindowProperty() in
  various places.

- Remove dpy parameter from find_win() and find_toplevel() as it's unused.

- Fix a writing-to-freed-memory issue in paint_preprocess().
pull/2/head
Richard Grenville 12 years ago
parent 53aeb801d4
commit 86103c5ceb
2 changed files with 215 additions and 94 deletions
Show Stats Download Patch File Download Diff File

210
compton.c
Unescape View File

@ -71,6 +71,9 @@ Bool has_name_pixmap;
int root_height, root_width; int root_height, root_width;
/// A region of the size of the screen. /// A region of the size of the screen.
XserverRegion screen_reg = None; XserverRegion screen_reg = None;
/// Current active window. Used by EWMH _NET_ACTIVE_WINDOW focus
/// detection.
win *active_win = NULL;
/// Pregenerated alpha pictures. /// Pregenerated alpha pictures.
Picture *alpha_picts = NULL; Picture *alpha_picts = NULL;
@ -153,14 +156,14 @@ int size_expose = 0;
int n_expose = 0; int n_expose = 0;
// atoms // atoms
Atom extents_atom; Atom opacity_atom = None;
Atom opacity_atom; Atom frame_extents_atom = None;
Atom frame_extents_atom; Atom client_atom = None;
Atom client_atom; Atom name_atom = None;
Atom name_atom; Atom name_ewmh_atom = None;
Atom name_ewmh_atom; Atom class_atom = None;
Atom class_atom; Atom transient_atom = None;
Atom transient_atom; Atom ewmh_active_win_atom = None;;
Atom win_type_atom; Atom win_type_atom;
Atom win_type[NUM_WINTYPES]; Atom win_type[NUM_WINTYPES];
@ -217,6 +220,7 @@ static options_t opts = {
.detect_client_opacity = False, .detect_client_opacity = False,
.inactive_dim = 0.0, .inactive_dim = 0.0,
.alpha_step = 0.03, .alpha_step = 0.03,
.use_ewmh_active_win = False,
.track_focus = False, .track_focus = False,
.track_wdata = False, .track_wdata = False,
@ -998,12 +1002,13 @@ static long
determine_evmask(Display *dpy, Window wid, win_evmode_t mode) { determine_evmask(Display *dpy, Window wid, win_evmode_t mode) {
long evmask = NoEventMask; long evmask = NoEventMask;
if (WIN_EVMODE_FRAME == mode || find_win(dpy, wid)) { if (WIN_EVMODE_FRAME == mode || find_win(wid)) {
evmask |= PropertyChangeMask; evmask |= PropertyChangeMask;
if (opts.track_focus) evmask |= FocusChangeMask; if (opts.track_focus && !opts.use_ewmh_active_win)
evmask |= FocusChangeMask;
} }
if (WIN_EVMODE_CLIENT == mode || find_toplevel(dpy, wid)) { if (WIN_EVMODE_CLIENT == mode || find_toplevel(wid)) {
if (opts.frame_opacity || opts.track_wdata if (opts.frame_opacity || opts.track_wdata
|| opts.detect_client_opacity) || opts.detect_client_opacity)
evmask |= PropertyChangeMask; evmask |= PropertyChangeMask;
@ -1013,7 +1018,7 @@ determine_evmask(Display *dpy, Window wid, win_evmode_t mode) {
} }
static win * static win *
find_win(Display *dpy, Window id) { find_win(Window id) {
win *w; win *w;
for (w = list; w; w = w->next) { for (w = list; w; w = w->next) {
@ -1027,12 +1032,11 @@ find_win(Display *dpy, Window id) {
/** /**
* Find out the WM frame of a client window using existing data. * Find out the WM frame of a client window using existing data.
* *
* @param dpy display to use
* @param w window ID * @param w window ID
* @return struct _win object of the found window, NULL if not found * @return struct _win object of the found window, NULL if not found
*/ */
static win * static win *
find_toplevel(Display *dpy, Window id) { find_toplevel(Window id) {
win *w; win *w;
for (w = list; w; w = w->next) { for (w = list; w; w = w->next) {
@ -1055,7 +1059,7 @@ find_toplevel2(Display *dpy, Window wid) {
win *w = NULL; win *w = NULL;
// We traverse through its ancestors to find out the frame // We traverse through its ancestors to find out the frame
while (wid && wid != root && !(w = find_win(dpy, wid))) { while (wid && wid != root && !(w = find_win(wid))) {
Window troot; Window troot;
Window parent; Window parent;
Window *tchildren; Window *tchildren;
@ -1087,6 +1091,12 @@ find_toplevel2(Display *dpy, Window wid) {
*/ */
static win * static win *
recheck_focus(Display *dpy) { recheck_focus(Display *dpy) {
// Use EWMH _NET_ACTIVE_WINDOW if enabled
if (opts.use_ewmh_active_win) {
update_ewmh_active_win(dpy);
return active_win;
}
// Determine the currently focused window so we can apply appropriate // Determine the currently focused window so we can apply appropriate
// opacity on it // opacity on it
Window wid = 0; Window wid = 0;
@ -1096,7 +1106,7 @@ recheck_focus(Display *dpy) {
XGetInputFocus(dpy, &wid, &revert_to); XGetInputFocus(dpy, &wid, &revert_to);
// Fallback to the old method if find_toplevel() fails // Fallback to the old method if find_toplevel() fails
if (!(w = find_toplevel(dpy, wid))) { if (!(w = find_toplevel(wid))) {
w = find_toplevel2(dpy, wid); w = find_toplevel2(dpy, wid);
} }
@ -1117,33 +1127,24 @@ root_tile_f(Display *dpy) {
} */ } */
Picture picture; Picture picture;
Atom actual_type;
Pixmap pixmap; Pixmap pixmap;
int actual_format;
unsigned long nitems;
unsigned long bytes_after;
unsigned char *prop;
Bool fill; Bool fill;
XRenderPictureAttributes pa; XRenderPictureAttributes pa;
int p; int p;
pixmap = None; pixmap = None;
// Get the values of background attributes
for (p = 0; background_props[p]; p++) { for (p = 0; background_props[p]; p++) {
prop = NULL; winattr_t attr = wid_get_attr(dpy, root,
if (XGetWindowProperty(dpy, root, XInternAtom(dpy, background_props[p], False), 1L, XA_PIXMAP, 32);
XInternAtom(dpy, background_props[p], False), if (attr.nitems) {
0, 4, False, AnyPropertyType, &actual_type, pixmap = *((long *) attr.data);
&actual_format, &nitems, &bytes_after, &prop
) == Success
&& actual_type == XInternAtom(dpy, "PIXMAP", False)
&& actual_format == 32 && nitems == 1) {
memcpy(&pixmap, prop, 4);
XFree(prop);
fill = False; fill = False;
free_winattr(&attr);
break; break;
} else if (prop) }
XFree(prop); free_winattr(&attr);
} }
if (!pixmap) { if (!pixmap) {
@ -1321,27 +1322,16 @@ find_client_win(Display *dpy, Window w) {
static void static void
get_frame_extents(Display *dpy, win *w, Window client) { get_frame_extents(Display *dpy, win *w, Window client) {
long *extents;
Atom type;
int format;
unsigned long nitems, after;
unsigned char *data = NULL;
int result;
w->left_width = 0; w->left_width = 0;
w->right_width = 0; w->right_width = 0;
w->top_width = 0; w->top_width = 0;
w->bottom_width = 0; w->bottom_width = 0;
result = XGetWindowProperty( winattr_t attr = wid_get_attr(dpy, client, frame_extents_atom,
dpy, client, frame_extents_atom, 4L, XA_CARDINAL, 32);
0L, 4L, False, AnyPropertyType,
&type, &format, &nitems, &after,
&data);
if (result == Success) { if (4 == attr.nitems) {
if (nitems == 4 && after == 0) { long *extents = (long *) attr.data;
extents = (long *) data;
w->left_width = extents[0]; w->left_width = extents[0];
w->right_width = extents[1]; w->right_width = extents[1];
w->top_width = extents[2]; w->top_width = extents[2];
@ -1350,8 +1340,8 @@ get_frame_extents(Display *dpy, win *w, Window client) {
if (opts.frame_opacity) if (opts.frame_opacity)
update_reg_ignore_expire(w); update_reg_ignore_expire(w);
} }
XFree(data);
} free_winattr(&attr);
} }
static inline Picture static inline Picture
@ -1539,13 +1529,19 @@ paint_preprocess(Display *dpy, win *list) {
if (to_paint) { if (to_paint) {
w->prev_trans = t; w->prev_trans = t;
t = w; t = w;
w->to_paint = to_paint;
} }
else { else {
// Avoid setting w->to_paint if w is to be freed
if (!(w->opacity_tgt == w->opacity && w->destroyed)) {
check_fade_fin(dpy, w); check_fade_fin(dpy, w);
}
w->to_paint = to_paint; w->to_paint = to_paint;
} }
else {
check_fade_fin(dpy, w);
}
}
}
return t; return t;
} }
@ -1916,7 +1912,7 @@ static void
map_win(Display *dpy, Window id, map_win(Display *dpy, Window id,
unsigned long sequence, Bool fade, unsigned long sequence, Bool fade,
Bool override_redirect) { Bool override_redirect) {
win *w = find_win(dpy, id); win *w = find_win(id);
// Don't care about window mapping if it's an InputOnly window // Don't care about window mapping if it's an InputOnly window
if (!w || InputOnly == w->a.class) return; if (!w || InputOnly == w->a.class) return;
@ -1980,14 +1976,15 @@ map_win(Display *dpy, Window id,
win_get_class(dpy, w); win_get_class(dpy, w);
} }
/*
* Occasionally compton does not seem able to get a FocusIn event from a
* window just mapped. I suspect it's a timing issue again when the
* XSelectInput() is called too late. We have to recheck the focused
* window here.
*/
if (opts.track_focus) { if (opts.track_focus) {
// Occasionally compton does not seem able to get a FocusIn event from
// a window just mapped. I suspect it's a timing issue again when the
// XSelectInput() is called too late. We have to recheck the focused
// window here. It makes no sense if we are using EWMH
// _NET_ACTIVE_WINDOW.
if (!opts.use_ewmh_active_win)
recheck_focus(dpy); recheck_focus(dpy);
// Consider a window without client window a WM window and mark it // Consider a window without client window a WM window and mark it
// focused if mark_wmwin_focused is on, or it's over-redirected and // focused if mark_wmwin_focused is on, or it's over-redirected and
// mark_ovredir_focused is on // mark_ovredir_focused is on
@ -2059,7 +2056,7 @@ unmap_callback(Display *dpy, win *w) {
static void static void
unmap_win(Display *dpy, Window id, Bool fade) { unmap_win(Display *dpy, Window id, Bool fade) {
win *w = find_win(dpy, id); win *w = find_win(id);
if (!w) return; if (!w) return;
@ -2326,7 +2323,7 @@ mark_client_win(Display *dpy, win *w, Window client) {
static void static void
add_win(Display *dpy, Window id, Window prev, Bool override_redirect) { add_win(Display *dpy, Window id, Window prev, Bool override_redirect) {
if (find_win(dpy, id)) { if (find_win(id)) {
return; return;
} }
@ -2500,7 +2497,7 @@ configure_win(Display *dpy, XConfigureEvent *ce) {
return; return;
} }
win *w = find_win(dpy, ce->window); win *w = find_win(ce->window);
XserverRegion damage = None; XserverRegion damage = None;
if (!w) if (!w)
@ -2568,7 +2565,7 @@ configure_win(Display *dpy, XConfigureEvent *ce) {
static void static void
circulate_win(Display *dpy, XCirculateEvent *ce) { circulate_win(Display *dpy, XCirculateEvent *ce) {
win *w = find_win(dpy, ce->window); win *w = find_win(ce->window);
Window new_above; Window new_above;
if (!w) return; if (!w) return;
@ -2591,6 +2588,10 @@ finish_destroy_win(Display *dpy, Window id) {
finish_unmap_win(dpy, w); finish_unmap_win(dpy, w);
*prev = w->next; *prev = w->next;
// Clear active_win if it's pointing to the destroyed window
if (active_win)
active_win = NULL;
free_picture(dpy, &w->shadow_pict); free_picture(dpy, &w->shadow_pict);
free_damage(dpy, &w->damage); free_damage(dpy, &w->damage);
free_region(dpy, &w->reg_ignore); free_region(dpy, &w->reg_ignore);
@ -2611,7 +2612,7 @@ destroy_callback(Display *dpy, win *w) {
static void static void
destroy_win(Display *dpy, Window id, Bool fade) { destroy_win(Display *dpy, Window id, Bool fade) {
win *w = find_win(dpy, id); win *w = find_win(id);
if (w) { if (w) {
w->destroyed = True; w->destroyed = True;
@ -2649,7 +2650,7 @@ damage_win(Display *dpy, XDamageNotifyEvent *de) {
return; return;
} */ } */
win *w = find_win(dpy, de->drawable); win *w = find_win(de->drawable);
if (!w) return; if (!w) return;
@ -2987,7 +2988,7 @@ ev_focus_in(XFocusChangeEvent *ev) {
if (!ev_focus_accept(ev)) if (!ev_focus_accept(ev))
return; return;
win *w = find_win(dpy, ev->window); win *w = find_win(ev->window);
// To deal with events sent from windows just destroyed // To deal with events sent from windows just destroyed
if (!w) return; if (!w) return;
@ -3004,7 +3005,7 @@ ev_focus_out(XFocusChangeEvent *ev) {
if (!ev_focus_accept(ev)) if (!ev_focus_accept(ev))
return; return;
win *w = find_win(dpy, ev->window); win *w = find_win(ev->window);
// To deal with events sent from windows just destroyed // To deal with events sent from windows just destroyed
if (!w) return; if (!w) return;
@ -3095,16 +3096,58 @@ ev_expose(XExposeEvent *ev) {
} }
} }
/**
* Update current active window based on EWMH _NET_ACTIVE_WIN.
*
* Does not change anything if we fail to get the attribute or the window
* returned could not be found.
*/
static void
update_ewmh_active_win(Display *dpy) {
// Get the attribute firstly
winattr_t attr = wid_get_attr(dpy, root, ewmh_active_win_atom,
1L, XA_WINDOW, 32);
if (!attr.nitems) {
free_winattr(&attr);
return;
}
// Search for the window
Window wid = *((long *) attr.data);
win *w = NULL;
free_winattr(&attr);
if (!(w = find_toplevel(wid)))
if (!(w = find_win(wid)))
w = find_toplevel2(dpy, wid);
// Mark the window focused
if (w) {
if (!w->focused)
set_focused(dpy, w, True);
if (active_win && w != active_win)
set_focused(dpy, active_win, False);
active_win = w;
}
}
inline static void inline static void
ev_property_notify(XPropertyEvent *ev) { ev_property_notify(XPropertyEvent *ev) {
// Destroy the root "image" if the wallpaper probably changed
if (root == ev->window) { if (root == ev->window) {
if (opts.track_focus && opts.use_ewmh_active_win
&& ewmh_active_win_atom == ev->atom) {
update_ewmh_active_win(dpy);
}
else {
// Destroy the root "image" if the wallpaper probably changed
for (int p = 0; background_props[p]; p++) { for (int p = 0; background_props[p]; p++) {
if (ev->atom == XInternAtom(dpy, background_props[p], False)) { if (ev->atom == XInternAtom(dpy, background_props[p], False)) {
root_damaged(); root_damaged();
break; break;
} }
} }
}
// Unconcerned about any other proprties on root window // Unconcerned about any other proprties on root window
return; return;
} }
@ -3112,10 +3155,10 @@ ev_property_notify(XPropertyEvent *ev) {
// If _NET_WM_OPACITY changes // If _NET_WM_OPACITY changes
if (ev->atom == opacity_atom) { if (ev->atom == opacity_atom) {
win *w = NULL; win *w = NULL;
if ((w = find_win(dpy, ev->window))) if ((w = find_win(ev->window)))
w->opacity_prop = wid_get_opacity_prop(dpy, w->id, OPAQUE); w->opacity_prop = wid_get_opacity_prop(dpy, w->id, OPAQUE);
else if (opts.detect_client_opacity else if (opts.detect_client_opacity
&& (w = find_toplevel(dpy, ev->window))) && (w = find_toplevel(ev->window)))
w->opacity_prop_client = wid_get_opacity_prop(dpy, w->client_win, w->opacity_prop_client = wid_get_opacity_prop(dpy, w->client_win,
OPAQUE); OPAQUE);
if (w) { if (w) {
@ -3124,8 +3167,8 @@ ev_property_notify(XPropertyEvent *ev) {
} }
// If frame extents property changes // If frame extents property changes
if (opts.frame_opacity && ev->atom == extents_atom) { if (opts.frame_opacity && ev->atom == frame_extents_atom) {
win *w = find_toplevel(dpy, ev->window); win *w = find_toplevel(ev->window);
if (w) { if (w) {
get_frame_extents(dpy, w, ev->window); get_frame_extents(dpy, w, ev->window);
// If frame extents change, the window needs repaint // If frame extents change, the window needs repaint
@ -3136,14 +3179,14 @@ ev_property_notify(XPropertyEvent *ev) {
// If name changes // If name changes
if (opts.track_wdata if (opts.track_wdata
&& (name_atom == ev->atom || name_ewmh_atom == ev->atom)) { && (name_atom == ev->atom || name_ewmh_atom == ev->atom)) {
win *w = find_toplevel(dpy, ev->window); win *w = find_toplevel(ev->window);
if (w && 1 == win_get_name(dpy, w)) if (w && 1 == win_get_name(dpy, w))
determine_shadow(dpy, w); determine_shadow(dpy, w);
} }
// If class changes // If class changes
if (opts.track_wdata && class_atom == ev->atom) { if (opts.track_wdata && class_atom == ev->atom) {
win *w = find_toplevel(dpy, ev->window); win *w = find_toplevel(ev->window);
if (w) { if (w) {
win_get_class(dpy, w); win_get_class(dpy, w);
determine_shadow(dpy, w); determine_shadow(dpy, w);
@ -3158,7 +3201,7 @@ ev_damage_notify(XDamageNotifyEvent *ev) {
inline static void inline static void
ev_shape_notify(XShapeEvent *ev) { ev_shape_notify(XShapeEvent *ev) {
win *w = find_win(dpy, ev->window); win *w = find_win(ev->window);
if (!w || IsUnmapped == w->a.map_state) return; if (!w || IsUnmapped == w->a.map_state) return;
/* /*
@ -3217,9 +3260,9 @@ ev_handle(XEvent *ev) {
else if (overlay == wid) else if (overlay == wid)
window_name = "(Overlay)"; window_name = "(Overlay)";
else { else {
win *w = find_win(dpy, wid); win *w = find_win(wid);
if (!w) if (!w)
w = find_toplevel(dpy, wid); w = find_toplevel(wid);
if (w && w->name) if (w && w->name)
window_name = w->name; window_name = w->name;
@ -3400,6 +3443,9 @@ usage(void) {
"--vsync-aggressive\n" "--vsync-aggressive\n"
" Attempt to send painting request before VBlank and do XFlush()\n" " Attempt to send painting request before VBlank and do XFlush()\n"
" during VBlank. This switch may be lifted out at any moment.\n" " during VBlank. This switch may be lifted out at any moment.\n"
"--use-ewmh-active-win\n"
" Use _NET_WM_ACTIVE_WINDOW on the root window to determine which\n"
" window is focused instead of using FocusIn/Out events.\n"
"\n" "\n"
"Format of a condition:\n" "Format of a condition:\n"
"\n" "\n"
@ -3751,6 +3797,9 @@ parse_config(char *cpath, struct options_tmp *pcfgtmp) {
lcfg_lookup_bool(&cfg, "paint-on-overlay", &opts.paint_on_overlay); lcfg_lookup_bool(&cfg, "paint-on-overlay", &opts.paint_on_overlay);
// --sw-opti // --sw-opti
lcfg_lookup_bool(&cfg, "sw-opti", &opts.sw_opti); lcfg_lookup_bool(&cfg, "sw-opti", &opts.sw_opti);
// --use-ewmh-active-win
lcfg_lookup_bool(&cfg, "use-ewmh-active-win",
&opts.use_ewmh_active_win);
// --shadow-exclude // --shadow-exclude
{ {
config_setting_t *setting = config_setting_t *setting =
@ -3821,6 +3870,7 @@ get_cfg(int argc, char *const *argv) {
{ "paint-on-overlay", no_argument, NULL, 273 }, { "paint-on-overlay", no_argument, NULL, 273 },
{ "sw-opti", no_argument, NULL, 274 }, { "sw-opti", no_argument, NULL, 274 },
{ "vsync-aggressive", no_argument, NULL, 275 }, { "vsync-aggressive", no_argument, NULL, 275 },
{ "use-ewmh-active-win", no_argument, NULL, 276 },
// Must terminate with a NULL entry // Must terminate with a NULL entry
{ NULL, 0, NULL, 0 }, { NULL, 0, NULL, 0 },
}; };
@ -4007,6 +4057,10 @@ get_cfg(int argc, char *const *argv) {
// --vsync-aggressive // --vsync-aggressive
opts.vsync_aggressive = True; opts.vsync_aggressive = True;
break; break;
case 276:
// --use-ewmh-active-win
opts.use_ewmh_active_win = True;
break;
default: default:
usage(); usage();
break; break;
@ -4060,7 +4114,6 @@ get_cfg(int argc, char *const *argv) {
static void static void
get_atoms(void) { get_atoms(void) {
extents_atom = XInternAtom(dpy, "_NET_FRAME_EXTENTS", False);
opacity_atom = XInternAtom(dpy, "_NET_WM_WINDOW_OPACITY", False); opacity_atom = XInternAtom(dpy, "_NET_WM_WINDOW_OPACITY", False);
frame_extents_atom = XInternAtom(dpy, "_NET_FRAME_EXTENTS", False); frame_extents_atom = XInternAtom(dpy, "_NET_FRAME_EXTENTS", False);
client_atom = XInternAtom(dpy, "WM_STATE", False); client_atom = XInternAtom(dpy, "WM_STATE", False);
@ -4068,6 +4121,7 @@ get_atoms(void) {
name_ewmh_atom = XInternAtom(dpy, "_NET_WM_NAME", False); name_ewmh_atom = XInternAtom(dpy, "_NET_WM_NAME", False);
class_atom = XA_WM_CLASS; class_atom = XA_WM_CLASS;
transient_atom = XA_WM_TRANSIENT_FOR; transient_atom = XA_WM_TRANSIENT_FOR;
ewmh_active_win_atom = XInternAtom(dpy, "_NET_ACTIVE_WINDOW", False);
win_type_atom = XInternAtom(dpy, win_type_atom = XInternAtom(dpy,
"_NET_WM_WINDOW_TYPE", False); "_NET_WM_WINDOW_TYPE", False);

71
compton.h
Unescape View File

@ -169,6 +169,11 @@ typedef enum {
WINDOW_ARGB WINDOW_ARGB
} winmode; } winmode;
typedef struct {
unsigned char *data;
int nitems;
} winattr_t;
typedef struct _ignore { typedef struct _ignore {
struct _ignore *next; struct _ignore *next;
unsigned long sequence; unsigned long sequence;
@ -385,6 +390,8 @@ typedef struct _options {
double inactive_dim; double inactive_dim;
/// Step for pregenerating alpha pictures. 0.01 - 1.0. /// Step for pregenerating alpha pictures. 0.01 - 1.0.
double alpha_step; double alpha_step;
/// Whether to use EWMH _NET_ACTIVE_WINDOW to find active window.
Bool use_ewmh_active_win;
// Calculated // Calculated
/// Whether compton needs to track focus changes. /// Whether compton needs to track focus changes.
@ -811,6 +818,63 @@ wid_has_attr(Display *dpy, Window w, Atom atom) {
return False; return False;
} }
/**
* Get a specific attribute of a window.
*
* Returns a blank structure if the returned type and format does not
* match the requested type and format.
*
* @param dpy Display to use
* @param w window
* @param atom atom of attribute to fetch
* @param length length to read
* @param rtype atom of the requested type
* @param rformat requested format
* @return a <code>winattr_t</code> structure containing the attribute
* and number of items. A blank one on failure.
*/
static winattr_t
wid_get_attr(Display *dpy, Window w, Atom atom, long length,
Atom rtype, int rformat) {
Atom type = None;
int format = 0;
unsigned long nitems = 0, after = 0;
unsigned char *data = NULL;
// Use two if statements to deal with the sequence point issue.
if (Success == XGetWindowProperty(dpy, w, atom, 0L, length, False,
rtype, &type, &format, &nitems, &after, &data)) {
if (type == rtype && format == rformat) {
return (winattr_t) {
.data = data,
.nitems = nitems
};
}
}
XFree(data);
return (winattr_t) {
.data = NULL,
.nitems = 0
};
}
/**
* Free a <code>winattr_t</code>.
*
* @param pattr pointer to the <code>winattr_t</code> to free.
*/
static inline void
free_winattr(winattr_t *pattr) {
// Empty the whole structure to avoid possible issues
if (pattr->data) {
XFree(pattr->data);
pattr->data = NULL;
}
pattr->nitems = 0;
}
/** /**
* Get the children of a window. * Get the children of a window.
* *
@ -881,10 +945,10 @@ static long
determine_evmask(Display *dpy, Window wid, win_evmode_t mode); determine_evmask(Display *dpy, Window wid, win_evmode_t mode);
static win * static win *
find_win(Display *dpy, Window id); find_win(Window id);
static win * static win *
find_toplevel(Display *dpy, Window id); find_toplevel(Window id);
static win * static win *
find_toplevel2(Display *dpy, Window wid); find_toplevel2(Display *dpy, Window wid);
@ -1072,6 +1136,9 @@ ev_circulate_notify(XCirculateEvent *ev);
inline static void inline static void
ev_expose(XExposeEvent *ev); ev_expose(XExposeEvent *ev);
static void
update_ewmh_active_win(Display *dpy);
inline static void inline static void
ev_property_notify(XPropertyEvent *ev); ev_property_notify(XPropertyEvent *ev);

Write Preview
Loading…
Cancel
Save