On Tue, Sep 25, 2007 at 09:29:36PM +0200, Szabolcs Nagy wrote:
> #define ISTILE    isarrange(tile) /* || isarrange(<custom>) */
> 
> 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) ..)'

I see your point, but I think it is implicit that you should use
parenthesis in complex expressions, in the same way that you have
to use quotation marks in strings, for example (IMVHO).

--
yiyu||JGL


Reply via email to