Re: [dwm] [patch] Minor code cleanup, part 2

From: Anselm R Garbe <>
Date: Mon, 18 Aug 2008 10:06:58 +0100

2008/8/14 Szabolcs Nagy <>:
> On 8/14/08, Martin Hurton <> wrote:
>> Can you apply the following patch too? According to spec the
>> XGetTransientForHint() function returns non zero value on success
>> while the Success value is defined as zero (see /usr/include/X11/X.h).
> I'm not quite sure about this change.
> eg. X reference manual never states explicitly that GrabSuccess is 0
> (even though if one reads the X protocol specification he can *guess*
> that it should be 0) so the programmer shouldn't rely on this.
> but if it's changed then change consistently:
> $ grep Success dwm.c |wc -l
> 6

At least in XGetTransientForHint(3) the following is mentioned:

       The XGetTransientForHint function returns the WM_TRANSIENT_FOR
property for
       the specified window. It returns a nonzero status on success;
otherwise, it
       returns a zero status.

So everything is ok.

Kind regards,
Received on Mon Aug 18 2008 - 09:06:58 UTC

This archive was generated by hypermail 2.2.0 : Mon Aug 18 2008 - 09:12:05 UTC