Re: [hackers] [dwm] applied Jukkas patch

From: Jukka Salmi <j+dwm_AT_2007.salmi.ch>
Date: Wed, 30 May 2007 20:58:15 +0200

Sander van Dijk --> hackers (2007-05-30 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...

Both parts are just for pleasing gcc. And yes, it does the same as
before.

Regards, Jukka

> 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;
>>
>>
>

-- 
bashian roulette:
$ ((RANDOM%6)) || rm -rf ~
Received on Wed May 30 2007 - 20:58:16 UTC

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