On 6/16/10 9:25 PM, Matt Turner wrote:
On Wed, Jun 16, 2010 at 2:28 PM, Tomas Carnecky<[email protected]> wrote:
Same for xfree/xrealloc/xcalloc. Also, the return value from
malloc/realloc/calloc doesn't require a cast, same for the
argument of free(). Also remove if()'s that check if the
pointer that is about to be passed to free() is NULL, since
free(NULL) is allowed.
Signed-off-by: Tomas Carnecky<[email protected]>
'Also' in commit messages is usually a sign that the commit needs to
be split into multiple patches.
Whatever the case,
Reviewed-by: Matt Turner<[email protected]>
Right. Maybe I shouldn't have written the 'Also'. I don't think the
cleanup I did around these functions would warrant a separate patch.
When I was renaming the functions, removing the cast on the same line
was pretty much a no-brainer for me.
tom
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel
