[PULL] getting rid of Xalloc/Xfree/... in server

Keith Packard keithp at keithp.com
Mon May 10 13:48:08 PDT 2010


On Mon, 10 May 2010 09:21:53 -0700, Keith Packard <keithp at keithp.com> wrote:
> On Mon, 10 May 2010 14:30:12 +0700, Mikhail Gusarov <dottedmag at dottedmag.net> wrote:
> 
> > Mikhail Gusarov (6):
> >       Clean {X,XNF}{alloc,calloc,realloc,free,strdup} from pre-C89 baggage
> >       Document allocation functions, noting deviations from C library
> 
> This documents that Xstrdup and XNFstrdup accept NULL, when the
> implementation does not check for that. I'm good with either fixing the
> docs or the code, but you need to pick one...

Sorry, I misread the patch -- the checks are still there, but 'hidden'
as they were the only unchanged lines.

Looks good to me. I'm going to merge this patch unless I get complaints
From someone who needs to replace the allocator and is running on a
system which cannot manage to override malloc/free/....

-- 
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20100510/1df29ff7/attachment.pgp>


More information about the xorg-devel mailing list