[PATCH libX11] XFree will accept NULL as argument

Hans de Goede hdegoede at redhat.com
Fri Aug 26 09:08:02 UTC 2016


Hi,

On 15-08-16 19:18, 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

Thanks, applied.

Can you next time please add a S-o-b, i.e.:

Signed-off-by: Walter Harms <wharms at bfs.de>

As last line of the commit msg ?

Regards,

Hans


>
> ---
>  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,
>  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
>


More information about the xorg-devel mailing list