simplified configurerequest

client.c

@@ -69,19 +69,20 @@ xerrordummy(Display *dsply, XErrorEvent *ee) {
 
 void
 configure(Client *c) {
-	XEvent synev;
+	XConfigureEvent ce;
 
-	synev.type = ConfigureNotify;
+	ce.type = ConfigureNotify;
-	synev.xconfigure.display = dpy;
+	ce.display = dpy;
-	synev.xconfigure.event = c->win;
+	ce.event = c->win;
-	synev.xconfigure.window = c->win;
+	ce.window = c->win;
-	synev.xconfigure.x = c->x;
+	ce.x = c->x;
-	synev.xconfigure.y = c->y;
+	ce.y = c->y;
-	synev.xconfigure.width = c->w;
+	ce.width = c->w;
-	synev.xconfigure.height = c->h;
+	ce.height = c->h;
-	synev.xconfigure.border_width = c->border;
+	ce.border_width = c->border;
-	synev.xconfigure.above = None;
+	ce.above = None;
-	XSendEvent(dpy, c->win, True, NoEventMask, &synev);
+	ce.override_redirect = False;
+	XSendEvent(dpy, c->win, False, StructureNotifyMask, (XEvent *)&ce);
 }
 
 void

event.c

@@ -166,52 +166,33 @@ buttonpress(XEvent *e) {
 
 static void
 configurerequest(XEvent *e) {
-	unsigned long newmask;
 	Client *c;
 	XConfigureRequestEvent *ev = &e->xconfigurerequest;
 	XWindowChanges wc;
 
+	wc.x = ev->x;
+	wc.y = ev->y;
+	wc.width = ev->width;
+	wc.height = ev->height;
+	wc.border_width = ev->border_width;
+	wc.sibling = ev->above;
+	wc.stack_mode = ev->detail;
 	if((c = getclient(ev->window))) {
 		c->ismax = False;
-		if(ev->value_mask & CWX)
-			c->x = ev->x;
-		if(ev->value_mask & CWY)
-			c->y = ev->y;
-		if(ev->value_mask & CWWidth)
-			c->w = ev->width;
-		if(ev->value_mask & CWHeight)
-			c->h = ev->height;
 		if(ev->value_mask & CWBorderWidth)
 			c->border = ev->border_width;
-		wc.x = c->x;
+		if((!c->isfloat && (arrange != dofloat))
-		wc.y = c->y;
+			|| ((ev->value_mask & (CWX|CWY)) && !(ev->value_mask & (CWWidth|CWHeight))))
-		wc.width = c->w;
+		{
-		wc.height = c->h;
-		newmask = ev->value_mask & (~(CWSibling | CWStackMode | CWBorderWidth));
-		if(newmask)
-			XConfigureWindow(dpy, c->win, newmask, &wc);
-		else
 			configure(c);
-		XSync(dpy, False);
+			XSync(dpy, False);
-		if(c->isfloat) {
+			return;
-			resize(c, False);
-			if(!isvisible(c))
-				XMoveWindow(dpy, c->win, c->x + 2 * sw, c->y);
 		}
-		else
-			arrange();
-	}
-	else {
-		wc.x = ev->x;
-		wc.y = ev->y;
-		wc.width = ev->width;
-		wc.height = ev->height;
-		wc.border_width = ev->border_width;
-		wc.sibling = ev->above;
-		wc.stack_mode = ev->detail;
-		XConfigureWindow(dpy, ev->window, ev->value_mask, &wc);
-		XSync(dpy, False);
 	}
+	XConfigureWindow(dpy, ev->window, ev->value_mask, &wc);
+	if(c && !isvisible(c))
+		XMoveWindow(dpy, c->win, c->x + 2 * sw, c->y);
+	XSync(dpy, False);
 }
 
 static void