[PATCH 00/11] Xasprintf() and other string-handling cleanups

Mikhail Gusarov dottedmag at dottedmag.net
Tue Nov 30 04:08:36 PST 2010


Twas brillig at 20:57:37 29.11.2010 UTC-08 when alan.coopersmith at oracle.com did gyre and gimble:

 AC> The meat of this series is sandwiched right in the middle - patch
 AC> #6 replaces the nifty, but unique, Xprintf() API with a local
 AC> implementation of the asprintf() family that's becoming widely
 AC> adopted (found in recent versions of the GNU, FreeBSD, NetBSD,
 AC> OpenBSD, & Solaris libc's, but not yet in all the versions &
 AC> platforms we still support).

Is there particular reason why local implementation of
asprintf/vasprintf have to have X prefix? Why not define them as plain
asprintf/vasprintf from the beginning? (XNF has to stay, sigh).

If this issue is resolved (see also additional nitpick in patch 06)

Reviewed-by: Mikhail Gusarov <dottedmag at dottedmag.net>

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


More information about the xorg-devel mailing list