[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [ft-devel] Experimental: v38 interpreter with minimal backwards comp
From: |
Werner LEMBERG |
Subject: |
Re: [ft-devel] Experimental: v38 interpreter with minimal backwards compatibility mode and linear advance widths |
Date: |
Thu, 17 Mar 2016 09:25:02 +0100 (CET) |
> Another one.
Thanks.
There are a lot of repetitive texts in your patch, like
/* Given ClearType's virtual increase of resolution on the X-axis, rounding
* to the physical pixel grid there doesn't make much sense. Native ClearType
* fonts know this, the rest (where backwards compatibility applies) may or
* may not. Short out rounding if freedom vector is parallel to X-axis. */
This is hard to main in case there should be ever something to change.
Can you reduce this amount of duplication? For example, introduce a
comment section with a numbered list of remarks near the top of the
file, then refer to it in the code.
Werner
Re: [ft-devel] Experimental: v38 interpreter with minimal backwards compatibility mode and linear advance widths, Nikolaus Waxweiler, 2016/03/10
Re: [ft-devel] Experimental: v38 interpreter with minimal backwards compatibility mode and linear advance widths, Nikolaus Waxweiler, 2016/03/10
Re: [ft-devel] Experimental: v38 interpreter with minimal backwards compatibility mode and linear advance widths, Nikolaus Waxweiler, 2016/03/13