[PATCH libX11] XFree will accept NULL as argument

Eric Engestrom eric.engestrom at imgtec.com
Tue Aug 16 10:56:50 UTC 2016


On Mon, Aug 15, 2016 at 07:18:14PM +0200, walter harms wrote:
> since Xfree is a define for free()
>   Xlibint.h:#define Xfree(ptr) free((ptr))
> 
> Xfree will accept NULL and do nothing.
> 
> the patch is also available on github.com/xtforever/libX11.git
> 
> ---
>  man/XFree.man | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/man/XFree.man b/man/XFree.man
> index ffc513b..72674a5 100644
> --- a/man/XFree.man
> +++ b/man/XFree.man
> @@ -90,8 +90,9 @@ Specifies the data that are to be freed.
>  The
>  .ZN XFree
>  function is a general-purpose Xlib routine that frees the specified data.
> -You must use it to free any objects that were allocated by Xlib,
> +You must use it to free any objects that were allocated by Xlib,

What's the change here?

The rest is:
Reviewed-by: Eric Engestrom <eric.engestrom at imgtec.com>

>  unless an alternate function is explicitly specified for the object.
> -A NULL pointer cannot be passed to this function.
> +
> +If data is NULL, no operation is performed.
>  .SH "SEE ALSO"
>  \fI\*(xL\fP
> -- 
> 2.1.4
> 


More information about the xorg-devel mailing list