Re: [hackers] [dmenu] [PATCH] Set class name on menu window

From: Omar Sandoval <osandov_AT_osandov.com>
Date: Thu, 2 Nov 2017 10:29:15 -0700

On Thu, Nov 02, 2017 at 06:19:44PM +0100, Hiltjo Posthuma wrote:
> On Thu, Nov 02, 2017 at 02:10:35AM -0700, Omar Sandoval wrote:
> > This is useful for, e.g., compton.
> > ---
> > dmenu.c | 8 ++++++++
> > 1 file changed, 8 insertions(+)
> >
> > diff --git a/dmenu.c b/dmenu.c
> > index d605ab4..2a16ad8 100644
> > --- a/dmenu.c
> > +++ b/dmenu.c
> > _AT_@ -545,6 +545,7 @@ setup(void)
> > XIM xim;
> > Window w, dw, *dws;
> > XWindowAttributes wa;
> > + XClassHint *ch;
> > #ifdef XINERAMA
> > XineramaScreenInfo *info;
> > Window pw;
> > _AT_@ -613,6 +614,13 @@ setup(void)
> > win = XCreateWindow(dpy, parentwin, x, y, mw, mh, 0,
> > CopyFromParent, CopyFromParent, CopyFromParent,
> > CWOverrideRedirect | CWBackPixel | CWEventMask, &swa);
> > + ch = XAllocClassHint();
> > + if (ch) {
> > + ch->res_name = "dmenu";
> > + ch->res_class = "dmenu";
> > + XSetClassHint(dpy, win, ch);
> > + XFree(ch);
> > + }
> >
> > /* open input methods */
> > xim = XOpenIM(dpy, NULL, NULL, NULL);
> > --
> > 2.15.0
> >
> >
>
> Meh, can't you just use the process name?

No, because dmenu doesn't set _NET_WM_PID either, so there's no way to
map the window to a process :) I don't think compton and other similar
stuff supports keying on process name, anyways.
Received on Thu Nov 02 2017 - 18:29:15 CET