On Mon, 10 May 2010 09:21:53 -0700, Keith Packard <[email protected]> wrote: > On Mon, 10 May 2010 14:30:12 +0700, Mikhail Gusarov <[email protected]> > 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/.... -- [email protected]
pgpk4mVoCcIuj.pgp
Description: PGP signature
_______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
