Re: [dwm] hg tip / upcoming 4.9

From: Nibble <nibble_AT_develsec.org>
Date: Tue, 25 Mar 2008 17:04:36 +0100

> No, your other fix is more correct. It should be ev->x >= bgw &&
> ev->x < x.

Then, the line 349:

        if((ev->x < x + blw) && ev->button == Button1)

would be more correct if it was:

        if((ev->x >= x && ev->x < x + blw) && ev->button == Button1)

Imho,

        if(ev->x < x)
        ...
        if((ev->x < x + blw) && ev->button == Button1)

are less redundant, due to function returns if the mouse event carries out the
previous condition. Anyway, I think both solutions are equally correct.

Kind regards,
Nibble
Received on Tue Mar 25 2008 - 17:05:00 UTC

This archive was generated by hypermail 2.2.0 : Sun Jul 13 2008 - 15:28:34 UTC