2013/1/10 Behdad Esfahbod <beh...@behdad.org>

> It looks to me that David rewrote ttsbit, the code loading SFNT embedded
> bitmaps, to be more "memory-optimized", but then had to keep the old code
> for
> when "old internals" config option is enabled.  That means the
> functionality
> is duplicated now, in ttsbit.c and ttsbit0.c.  At least by default, "old
> internals" is enabled, which means that most probably most users are not
> using
> the "memory-optimized" version.
>
> I have a hard time imagining that the non-memory-optimized version has any
> measurable performance difference compared to the memory-optimized one in
> any
> real-world scenario.  As such, I like to suggest that the memory-optimized
> version be removed from the code base.
>
> Thoughts?
>
> It's been a long time, but I think it's really time we completely remove
the old-internals hack and corresponding code. It was only needed
transitively and all client code should have been modified to not require
it.

Also, sorry, but the Android build of FreeType uses the memory-optimized
version, I don't see any reason to remove it :-)


> --
> behdad
> http://behdad.org/
>
> _______________________________________________
> Freetype-devel mailing list
> Freetype-devel@nongnu.org
> https://lists.nongnu.org/mailman/listinfo/freetype-devel
>
_______________________________________________
Freetype-devel mailing list
Freetype-devel@nongnu.org
https://lists.nongnu.org/mailman/listinfo/freetype-devel

Reply via email to