2013/1/10 Behdad Esfahbod
<address@hidden>
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
address@hidden
https://lists.nongnu.org/mailman/listinfo/freetype-devel