[Fontconfig] fontconfig: Branch 'master'
akira at tagoh.org
Wed May 30 19:12:17 PDT 2012
On Wed, May 30, 2012 at 11:59 PM, Alan Coopersmith
<alan.coopersmith at oracle.com> wrote:
> Sorry, I haven't been paying as much attention to this as I should, but
> I did notice this thread and wonder about this fix.
Please see git log. that should explains all what you want to know.
> Isn't this backwards? fontconfig/fontconfig.h has #define FcFalse 0, but at
> least the Solaris scandir man page says
Doh. right. just a typo.
> If this pointer is NULL, then all the directory entries are included.
> which suggests a simpler and more efficient fix of:
> It also looks like you could save a whole bunch of malloc calls by either using
> a fixed size buffer of PATH_MAX, or saving the f pointer and realloc'ing it to
> be larger if needed, instead of malloc and free every time, such as:
True. that makes sense.
> Both of those are just suggestions which I have not tested (or even tried to
Thanks for catching this up and kindly your suggestion :)
> -Alan Coopersmith- alan.coopersmith at oracle.com
> Oracle Solaris Engineering - http://blogs.oracle.com/alanc
More information about the Fontconfig