Re: [dwm] cleanup patch for r874

From: Sergi Alvarez <pancake_AT_youterm.com>
Date: Wed, 16 May 2007 01:20:28 +0200 (CEST)

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 - 01:20:29 UTC

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