[PATCH libX11] XFree will accept NULL as argument

walter harms wharms at bfs.de
Fri Aug 26 09:14:29 UTC 2016



Am 26.08.2016 11:08, schrieb Hans de Goede:
> 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


Yes,
i really forgot about this detail this time.

re,
 wh

> 
>>
>> ---
>>  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
>>
> _______________________________________________
> xorg-devel at lists.x.org: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: https://lists.x.org/mailman/listinfo/xorg-devel


More information about the xorg-devel mailing list