[Xcb] [PATCH xpyb 01/10] conn.c: Delete explicit comparison with NULL

[Alan Coopersmith]

On 10/12/14 09:47 PM, Tapasweni Pathak wrote:
> if (pointer == NULL) is functionally equivalent to
> if (!pointer).
>
> Using the second will make it a bit obvious that there is a
> pointer involved and not an integer or something.

Really?  I find pointer == NULL to be obviously a pointer, and !value to
suggest a boolean.   Neither form suggests an integer or other type.

-- 
	-Alan Coopersmith-              alan.coopersmith at oracle.com
	 Oracle Solaris Engineering - http://blogs.oracle.com/alanc