The patch below gets rid of two superfluous assignments and
moves the function's exit point at the end of its body.
Comments are welcome.

Cheers,
/Martin

diff -r 5be4e277ed7c dwm.c
--- a/dwm.c     Mon Aug 18 10:22:46 2008 +0100
+++ b/dwm.c     Mon Aug 18 17:45:07 2008 +0200
@@ -961,18 +961,15 @@
        if(!(c = sel))
                return;
        restack();
-       ocx = nx = c->x;
-       ocy = ny = c->y;
+       ocx = c->x;
+       ocy = c->y;
        if(XGrabPointer(dpy, root, False, MOUSEMASK, GrabModeAsync, 
GrabModeAsync,
        None, cursor[CurMove], CurrentTime) != GrabSuccess)
                return;
        XQueryPointer(dpy, root, &dummy, &dummy, &x, &y, &di, &di, &dui);
-       for(;;) {
+       do {
                XMaskEvent(dpy, 
MOUSEMASK|ExposureMask|SubstructureRedirectMask, &ev);
                switch (ev.type) {
-               case ButtonRelease:
-                       XUngrabPointer(dpy, CurrentTime);
-                       return;
                case ConfigureRequest:
                case Expose:
                case MapRequest:
@@ -999,7 +996,8 @@
                                resize(c, nx, ny, c->w, c->h, False);
                        break;
                }
-       }
+       } while(ev.type != ButtonRelease);
+       XUngrabPointer(dpy, CurrentTime);
 }
 
 Client *

Reply via email to