Re: [ft-devel] discussion of truetype glyph loading code

2005-08-09 Thread Werner LEMBERG
This patch does the changes (avoiding unnecessary array copying) I mentioned in the last post, along with a heavy cleanup of the truetype module. I just skimmed it, and it looks fine. Since the only good thing of the cleanup is to make the code more readible and it risks unstablizing the

Re: [ft-devel] discussion of truetype glyph loading code

2005-08-09 Thread George Williams
On Mon, 2005-08-08 at 22:50, Werner LEMBERG wrote: This snippet is in deciding the mac scale of component offsets. It is commented out because the result is bad. But in this page http://developer.apple.com/fonts/TTRefMan/RM06/Chap6glyf.html, the algorithm described is slightly different

Re: [ft-devel] discussion of truetype glyph loading code

2005-08-09 Thread Werner LEMBERG
I thought heavily clean up implies some changes are not listed :) This is a lame excuse :-) Seriously, your clean-ups are still comprehendable -- it's more a code shifting than a complete rewrite, isn't it? Werner ___ Freetype-devel mailing

Re: [ft-devel] discussion of truetype glyph loading code

2005-08-09 Thread Chia I Wu
On Tue, Aug 09, 2005 at 06:55:54PM +0200, Werner LEMBERG wrote: This is a lame excuse :-) Seriously, your clean-ups are still comprehendable -- it's more a code shifting than a complete rewrite, isn't it? Yes, I just moved the code around and gave it sensible names. The few practical changes