Re: [dwm] no () around ISTILE

2007-09-25 Thread Jesus Galan
On Tue, Sep 25, 2007 at 11:35:05PM +0200, Jesus Galan wrote: > On Tue, Sep 25, 2007 at 09:29:36PM +0200, Szabolcs Nagy wrote: > > #define ISTILEisarrange(tile) /* || isarrange() */ > > > > can be misleading with custom layouts since in zoom() it is used like > > > > if(.. !ISTILED ..) return;

Re: [dwm] no () around ISTILE

2007-09-25 Thread David Tweed
On 9/25/07, Jesus Galan <[EMAIL PROTECTED]> wrote: > On Tue, Sep 25, 2007 at 09:29:36PM +0200, Szabolcs Nagy wrote: > > #define ISTILEisarrange(tile) /* || isarrange() */ > > > > can be misleading with custom layouts since in zoom() it is used like > > > > if(.. !ISTILED ..) return; > > > > so

Re: [dwm] no () around ISTILE

2007-09-25 Thread Jesus Galan
On Tue, Sep 25, 2007 at 09:29:36PM +0200, Szabolcs Nagy wrote: > #define ISTILEisarrange(tile) /* || isarrange() */ > > can be misleading with custom layouts since in zoom() it is used like > > if(.. !ISTILED ..) return; > > so either there should be a comment about parenthesis in config.def

[dwm] no () around ISTILE

2007-09-25 Thread Szabolcs Nagy
#define ISTILEisarrange(tile) /* || isarrange() */ can be misleading with custom layouts since in zoom() it is used like if(.. !ISTILED ..) return; so either there should be a comment about parenthesis in config.def.h or zoom should use it like 'if(.. !(ISTILED) ..)'