making it more sure that transient checks will work in any case

[dwm.git] / client.c

diff --git a/client.c b/client.c
index 2b194d7..16678d3 100644 (file)
--- a/client.c
+++ b/client.c
@@ -9,13 +9,6 @@
 
 /* static */
 
-static void
-closestpt(float *rx, float *ry, float x, float y, float grad) {
-       float u = (x * grad + y) / (grad * grad + 1);
-       *rx = u * grad;
-       *ry = u;
-}
-
 static void
 detachstack(Client *c) {
        Client **tc;
@@ -76,19 +69,20 @@ xerrordummy(Display *dsply, XErrorEvent *ee) {
 
 void
 configure(Client *c) {
-       XEvent synev;
+       XConfigureEvent ce;
 
-       synev.type = ConfigureNotify;
-       synev.xconfigure.display = dpy;
-       synev.xconfigure.event = c->win;
-       synev.xconfigure.window = c->win;
-       synev.xconfigure.x = c->x;
-       synev.xconfigure.y = c->y;
-       synev.xconfigure.width = c->w;
-       synev.xconfigure.height = c->h;
-       synev.xconfigure.border_width = c->border;
-       synev.xconfigure.above = None;
-       XSendEvent(dpy, c->win, True, NoEventMask, &synev);
+       ce.type = ConfigureNotify;
+       ce.display = dpy;
+       ce.event = c->win;
+       ce.window = c->win;
+       ce.x = c->x;
+       ce.y = c->y;
+       ce.width = c->w;
+       ce.height = c->h;
+       ce.border_width = c->border;
+       ce.above = None;
+       ce.override_redirect = False;
+       XSendEvent(dpy, c->win, False, StructureNotifyMask, (XEvent *)&ce);
 }
 
 void
@@ -127,11 +121,26 @@ getclient(Window w) {
        return NULL;
 }
 
+Bool
+isprotodel(Client *c) {
+       int i, n;
+       Atom *protocols;
+       Bool ret = False;
+
+       if(XGetWMProtocols(dpy, c->win, &protocols, &n)) {
+               for(i = 0; !ret && i < n; i++)
+                       if(protocols[i] == wmatom[WMDelete])
+                               ret = True;
+               XFree(protocols);
+       }
+       return ret;
+}
+
 void
 killclient(Arg *arg) {
        if(!sel)
                return;
-       if(sel->proto & PROTODELWIN)
+       if(isprotodel(sel))
                sendevent(sel->win, wmatom[WMProtocols], wmatom[WMDelete]);
        else
                XKillClient(dpy, sel->win);
@@ -139,7 +148,7 @@ killclient(Arg *arg) {
 
 void
 manage(Window w, XWindowAttributes *wa) {
-       Client *c;
+       Client *c, *t;
        Window trans;
 
        c = emallocz(sizeof(Client));
@@ -166,16 +175,16 @@ manage(Window w, XWindowAttributes *wa) {
                        c->y = way;
        }
        updatesizehints(c);
-       c->proto = getproto(c->win);
        XSelectInput(dpy, c->win,
                StructureNotifyMask | PropertyChangeMask | EnterWindowMask);
        XGetTransientForHint(dpy, c->win, &trans);
        grabbuttons(c, False);
        XSetWindowBorder(dpy, c->win, dc.norm[ColBorder]);
        updatetitle(c);
-       settags(c, getclient(trans));
+       t = getclient(trans);
+       settags(c, t);
        if(!c->isfloat)
-               c->isfloat = trans || c->isfixed;
+               c->isfloat = (t != 0) || c->isfixed;
        if(clients)
                clients->prev = c;
        c->next = clients;
@@ -191,7 +200,7 @@ manage(Window w, XWindowAttributes *wa) {
 
 void
 resize(Client *c, Bool sizehints) {
-       float dx, dy, min, max, actual;
+       float actual, dx, dy, max, min;
        XWindowChanges wc;
 
        if(c->w <= 0 || c->h <= 0)
@@ -214,12 +223,14 @@ resize(Client *c, Bool sizehints) {
                        actual = dx / dy;
                        if(max > 0 && min > 0 && actual > 0) {
                                if(actual < min) {
-                                       closestpt(&dx, &dy, dx, dy, min);
+                                       dy = (dx * min + dy) / (min * min + 1);
+                                       dx = dy * min;
                                        c->w = (int)dx + c->basew;
                                        c->h = (int)dy + c->baseh;
                                }
                                else if(actual > max) {
-                                       closestpt(&dx, &dy, dx, dy, max);
+                                       dy = (dx * min + dy) / (max * max + 1);
+                                       dx = dy * min;
                                        c->w = (int)dx + c->basew;
                                        c->h = (int)dy + c->baseh;
                                }