[PATCH 0/8] Getting rid of xstrdup

Mikhail Gusarov dottedmag at dottedmag.net
Sun Jun 6 13:09:55 PDT 2010


Twas brillig at 03:04:05 07.06.2010 UTC+07 when dottedmag at dottedmag.net did gyre and gimble:

 AC>> For patch 8: While the change isn't wrong, I'll leave it up to one
 AC>> of the XKB maintainers to decide if introducing a FatalError()
 AC>> where there was none before (Xprintf -> XNFprintf) is the right
 AC>> thing to do.

 MG> There was segfault instead down the road if Xprintf fails.

Actually there was no segfault due to uninteresting set of checks and
side-effects of various functions which made the code of ddxLoad.c so
ugly and hard-to-read. I am preparing a better patch.

-- 
  http://fossarchy.blogspot.com/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 835 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20100607/93bf4a82/attachment.pgp>


More information about the xorg-devel mailing list