Twas brillig at 20:57:37 29.11.2010 UTC-08 when [email protected] 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 <[email protected]>

-- 
  http://fossarchy.blogspot.com/

Attachment: pgpV6VmA50iTi.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

Reply via email to