Hi,
On 12-08-16 16:09, walter harms wrote:
Am 12.08.2016 12:11, schrieb Hans de Goede:
Hi,
On 28-07-16 19:31, walter harms wrote:
janitorial patch: remove some unneeded if() before free()
This is not free() but Xfree() and "man Xfree" states:
"A NULL pointer cannot be passed to this function."
this is wrong.
Xfree is a define for free()
Xlibint.h:#define Xfree(ptr) free((ptr))
I've pushed the 2 original patches now.
Note next time if you drop an if for a construct
like this:
if (foo)
Xfree(foo);
Then please also re-indent the Xfree and do not just
drop the if (as you've done in 2/2 this time). I've
fixed this up before pushing.
Regards,
Hans
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: https://lists.x.org/mailman/listinfo/xorg-devel
