Re: [dwm] cleanup patch for r874

From: pancake <pancake_AT_youterm.com>
Date: Wed, 16 May 2007 09:39:30 +0200 (CEST)

Oops, sorry for the noise O:)

replying mails from the mobile after getting some beers is not a good idea.

--pancake

> I think this patch is not necessary because cflags can be changed by
> providing our desired value as an argument for make:
>
> $ make CFLAGS=-w
>
> btw the use of ?= in cflags is recommended.
>
>> Thank you, I applied all your changes.
>>
>> Regards,
>> Anselm
>>
>> On Sat, May 12, 2007 at 05:37:35PM +0200, dfenze_AT_gmail.com wrote:
>>> Hi,
>>> created simply patch that somewhat cleanup r874 (actual tip).
>>>
>>> URL:
>>> http://na.srck.net/dwm/dwm-r874-cleanup.patch
>>>
>>> Stats:
>>> client.c | 14 ++++++++++----
>>> dwm.h | 1 +
>>> event.c | 3 +--
>>> layout.c | 15 ++++-----------
>>> 4 files changed, 16 insertions(+), 17 deletions(-)
>>>
>>> It creates new function:
>>> +focustopvisible(void) {
>>> + Client *c;
>>> +
>>> + for(c = stack; c && !isvisible(c); c = c->snext);
>>> + focus(c);
>>>
>>> and replaces all ocurrances of that selection&focus loops in dwm code
>>>
>>> and second thing cleaned up ;) was removing of code in layout.c/zoom
>>> which simply do nothing:
>>> - for(n = 0, c = nexttiled(clients); c; c = nexttiled(c->next))
>>> - n++;
>>>
>>> but n was never used in func. zoom.
>>>
>>> --anydot
>>>
>>
>> --
>> Anselm R. Garbe >< http://www.suckless.org/ >< GPG key: 0D73F361
>>
>>
>
>
>
>
Received on Wed May 16 2007 - 09:39:26 UTC

This archive was generated by hypermail 2.2.0 : Sun Jul 13 2008 - 14:41:48 UTC