Re: [hackers] [dwm] applied Jukkas patch

From: Sander van Dijk <a.h.vandijk_AT_gmail.com>
Date: Wed, 30 May 2007 19:54:19 +0200

Hi,

May I ask what this patch is for? I guess the second part is to keep
gcc from whining, but the first part is unclear to me; as far as I
know, it now does the same thing as it did before the patch. Btw. note
that I'm not particularly against the patch or anything, just
wondering what it's for...

Greetings, Sander.

On Wed May 30 08:57:44 2007 +0200, Anselm R. Garbe <arg_AT_suckless.org> wrote:
> changeset: 908:2f27c51332ed
> tag: tip
> user: Anselm R. Garbe <arg_AT_suckless.org>
> date: Wed May 30 08:57:44 2007 +0200
> summary: applied Jukkas patch
>
> diff -r a78ef26f7ae9 -r 2f27c51332ed client.c
> --- a/client.c Tue May 29 13:53:08 2007 +0200
> +++ b/client.c Wed May 30 08:57:44 2007 +0200
> @@ -131,7 +131,7 @@ detach(Client *c) {
>
> void
> focus(Client *c) {
> - if( !c && selscreen || c && !isvisible(c))
> + if((!c && selscreen)|| (c && !isvisible(c)))
> for(c = stack; c && !isvisible(c); c = c->snext);
> if(sel && sel != c) {
> grabbuttons(sel, False);
> diff -r a78ef26f7ae9 -r 2f27c51332ed event.c
> --- a/event.c Tue May 29 13:53:08 2007 +0200
> +++ b/event.c Wed May 30 08:57:44 2007 +0200
> @@ -244,7 +244,7 @@ enternotify(XEvent *e) {
>
> if(ev->mode != NotifyNormal || ev->detail == NotifyInferior)
> return;
> - if(c = getclient(ev->window))
> + if((c = getclient(ev->window)))
> focus(c);
> else if(ev->window == root) {
> selscreen = True;
>
>
Received on Wed May 30 2007 - 19:54:33 UTC

This archive was generated by hypermail 2.2.0 : Sun Jul 13 2008 - 15:56:53 UTC